unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#55233] [PATCH 0/3] Add python-madbg.
@ 2022-05-02 21:56 jgart via Guix-patches via
  2022-05-02 22:00 ` [bug#55233] [PATCH 1/3] gnu: Add python-pyinjector jgart via Guix-patches via
                   ` (2 more replies)
  0 siblings, 3 replies; 10+ messages in thread
From: jgart via Guix-patches via @ 2022-05-02 21:56 UTC (permalink / raw)
  To: 55233

Hi Guixers,

https://github.com/kmaork/madbg

all best,

jgart

ps 

above patches currently available as a pre-release from guixrus




^ permalink raw reply	[flat|nested] 10+ messages in thread

* [bug#55233] [PATCH 1/3] gnu: Add python-pyinjector.
  2022-05-02 21:56 [bug#55233] [PATCH 0/3] Add python-madbg jgart via Guix-patches via
@ 2022-05-02 22:00 ` jgart via Guix-patches via
  2022-05-02 22:00   ` [bug#55233] [PATCH 2/3] gnu: Add python-hypno jgart via Guix-patches via
                     ` (2 more replies)
  2022-05-02 22:08 ` [bug#55233] linter warning jgart via Guix-patches via
  2023-10-07  5:53 ` bug#55233: Closing jgart via Guix-patches via
  2 siblings, 3 replies; 10+ messages in thread
From: jgart via Guix-patches via @ 2022-05-02 22:00 UTC (permalink / raw)
  To: 55233; +Cc: jgart

* gnu/packages/python-xyz.scm (python-pyinjector): New variable.
---
 gnu/packages/python-xyz.scm | 20 ++++++++++++++++++++
 1 file changed, 20 insertions(+)

diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
index 5b3a742cb7..453080a4b7 100644
--- a/gnu/packages/python-xyz.scm
+++ b/gnu/packages/python-xyz.scm
@@ -28981,6 +28981,26 @@ (define-public pudb
 keyboard-friendly package.")
     (license license:expat)))
 
+(define-public python-pyinjector
+  (package
+    (name "python-pyinjector")
+    (version "1.0.6")
+    (source
+      (origin
+        (method url-fetch)
+        (uri (pypi-uri "pyinjector" version ".zip"))
+        (sha256
+          (base32 "126n70qmrlqd25xmxwaf800abfm1k250zx01kx3dha2p6pk3vx89"))))
+    (build-system python-build-system)
+    (native-inputs (list unzip))
+    (home-page "https://github.com/kmaork/pyinjector")
+    (synopsis
+      "A tool/library allowing dynamic library injection into running processes")
+    (description
+      "This package provides a tool/library allowing dynamic library injection into
+running processes")
+    (license license:expat)))
+
 (define-public python-iwlib
   (package
     (name "python-iwlib")
-- 
2.35.1





^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [bug#55233] [PATCH 2/3] gnu: Add python-hypno.
  2022-05-02 22:00 ` [bug#55233] [PATCH 1/3] gnu: Add python-pyinjector jgart via Guix-patches via
@ 2022-05-02 22:00   ` jgart via Guix-patches via
  2022-06-24 15:23     ` [bug#55233] [PATCH 0/3] Add python-madbg Maxim Cournoyer
  2022-05-02 22:00   ` [bug#55233] [PATCH 3/3] gnu: " jgart via Guix-patches via
  2022-06-24 15:21   ` Maxim Cournoyer
  2 siblings, 1 reply; 10+ messages in thread
From: jgart via Guix-patches via @ 2022-05-02 22:00 UTC (permalink / raw)
  To: 55233; +Cc: jgart

* gnu/packages/python-xyz.scm (python-hypno): New variable.
---
 gnu/packages/python-xyz.scm | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
index 453080a4b7..61c36699c7 100644
--- a/gnu/packages/python-xyz.scm
+++ b/gnu/packages/python-xyz.scm
@@ -29001,6 +29001,27 @@ (define-public python-pyinjector
 running processes")
     (license license:expat)))
 
+(define-public python-hypno
+  (package
+    (name "python-hypno")
+    (version "0.1.4")
+    (source
+      (origin
+        (method url-fetch)
+        (uri (pypi-uri "hypno" version ".zip"))
+        (sha256
+          (base32 "1703c1k1g404sl2yvh5wylk7prnfgm7c83cpx2gv3n50025d6910"))))
+    (build-system python-build-system)
+    (propagated-inputs (list python-pyinjector))
+    (native-inputs (list unzip))
+    (home-page "https://github.com/kmaork/hypno")
+    (synopsis
+      "A tool/library allowing to inject python code into a running python process.")
+    (description
+      "This package provides a tool/library allowing to inject python code into a
+running python process.")
+    (license license:expat)))
+
 (define-public python-iwlib
   (package
     (name "python-iwlib")
-- 
2.35.1





^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [bug#55233] [PATCH 3/3] gnu: Add python-madbg.
  2022-05-02 22:00 ` [bug#55233] [PATCH 1/3] gnu: Add python-pyinjector jgart via Guix-patches via
  2022-05-02 22:00   ` [bug#55233] [PATCH 2/3] gnu: Add python-hypno jgart via Guix-patches via
@ 2022-05-02 22:00   ` jgart via Guix-patches via
  2022-06-24 15:25     ` [bug#55233] [PATCH 0/3] " Maxim Cournoyer
  2022-06-24 15:21   ` Maxim Cournoyer
  2 siblings, 1 reply; 10+ messages in thread
From: jgart via Guix-patches via @ 2022-05-02 22:00 UTC (permalink / raw)
  To: 55233; +Cc: jgart

* gnu/packages/python-xyz.scm (python-madbg): New variable.
---
 gnu/packages/python-xyz.scm | 20 ++++++++++++++++++++
 1 file changed, 20 insertions(+)

diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
index 61c36699c7..551544b08e 100644
--- a/gnu/packages/python-xyz.scm
+++ b/gnu/packages/python-xyz.scm
@@ -29022,6 +29022,26 @@ (define-public python-hypno
 running python process.")
     (license license:expat)))
 
+(define-public python-madbg
+  (package
+    (name "python-madbg")
+    (version "1.2.0")
+    (source
+      (origin
+        (method url-fetch)
+        (uri (pypi-uri "madbg" version ".zip"))
+        (sha256
+          (base32 "19cll917w3zshxrcf2v5bdwkvbam3nslyik2hp4wpn5z4xkzsw5v"))))
+    (build-system python-build-system)
+    (propagated-inputs
+      (list python-click python-hypno python-ipython python-prompt-toolkit))
+    (native-inputs (list unzip))
+    (home-page "https://github.com/kmaork/madbg")
+    (synopsis "A fully-featured remote debugger for python")
+    (description
+      "This package provides a fully-featured remote debugger for python")
+    (license license:expat)))
+
 (define-public python-iwlib
   (package
     (name "python-iwlib")
-- 
2.35.1





^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [bug#55233] linter warning
  2022-05-02 21:56 [bug#55233] [PATCH 0/3] Add python-madbg jgart via Guix-patches via
  2022-05-02 22:00 ` [bug#55233] [PATCH 1/3] gnu: Add python-pyinjector jgart via Guix-patches via
@ 2022-05-02 22:08 ` jgart via Guix-patches via
  2022-05-03  8:56   ` Maxime Devos
  2023-10-07  5:53 ` bug#55233: Closing jgart via Guix-patches via
  2 siblings, 1 reply; 10+ messages in thread
From: jgart via Guix-patches via @ 2022-05-02 22:08 UTC (permalink / raw)
  To: 55233

Hi,

What's should I do to fix this one?

```
/home/jgart/973f231e-fd8e-47ce-bf3d-5f3baf22e411-guix/gnu/packages/python-xyz.scm:29026:2: python-madbg@1.2.0: propagated inputs python-prompt-toolkit@2.0.7 and python-prompt-toolkit@3.0.18 collide
```

all best,

jgart




^ permalink raw reply	[flat|nested] 10+ messages in thread

* [bug#55233] linter warning
  2022-05-02 22:08 ` [bug#55233] linter warning jgart via Guix-patches via
@ 2022-05-03  8:56   ` Maxime Devos
  0 siblings, 0 replies; 10+ messages in thread
From: Maxime Devos @ 2022-05-03  8:56 UTC (permalink / raw)
  To: jgart, 55233

[-- Attachment #1: Type: text/plain, Size: 859 bytes --]

jgart via Guix-patches via schreef op ma 02-05-2022 om 22:08 [+0000]:
> Hi,
> 
> What's should I do to fix this one?
> 
> ```
> /home/jgart/973f231e-fd8e-47ce-bf3d-5f3baf22e411-guix/gnu/packages/python-xyz.scm:29026:2: python-madbg@1.2.0: propagated inputs python-prompt-toolkit@2.0.7 and python-prompt-toolkit@3.0.18 collide

Determine which package has python-prompt-toolkit@2.0.7 and try
replacing it with python-prompt-toolkit@3.0.18 somehow
(for compatibility reasons, it might be necessary to update the package
using python-pprompt-toolkit@2.0.7).

These packages appear to be

  * python-ipython
  * python-jupyter-console

55104 has patches for updating ipythonand changes python-prompt-
toolkit@2 to python-prompt-toolkit@3, so it seems that the propagation
issue will be resolved by that patch series?

Greetings,
Maxime.

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 260 bytes --]

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [bug#55233] [PATCH 0/3] Add python-madbg.
  2022-05-02 22:00 ` [bug#55233] [PATCH 1/3] gnu: Add python-pyinjector jgart via Guix-patches via
  2022-05-02 22:00   ` [bug#55233] [PATCH 2/3] gnu: Add python-hypno jgart via Guix-patches via
  2022-05-02 22:00   ` [bug#55233] [PATCH 3/3] gnu: " jgart via Guix-patches via
@ 2022-06-24 15:21   ` Maxim Cournoyer
  2 siblings, 0 replies; 10+ messages in thread
From: Maxim Cournoyer @ 2022-06-24 15:21 UTC (permalink / raw)
  To: jgart; +Cc: 55233

Hi jgart,

jgart <jgart@dismail.de> writes:

> * gnu/packages/python-xyz.scm (python-pyinjector): New variable.
> ---
>  gnu/packages/python-xyz.scm | 20 ++++++++++++++++++++
>  1 file changed, 20 insertions(+)
>
> diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
> index 5b3a742cb7..453080a4b7 100644
> --- a/gnu/packages/python-xyz.scm
> +++ b/gnu/packages/python-xyz.scm
> @@ -28981,6 +28981,26 @@ (define-public pudb
>  keyboard-friendly package.")
>      (license license:expat)))
>  
> +(define-public python-pyinjector
> +  (package
> +    (name "python-pyinjector")
> +    (version "1.0.6")
> +    (source
> +      (origin
> +        (method url-fetch)
> +        (uri (pypi-uri "pyinjector" version ".zip"))
> +        (sha256
> +          (base32 "126n70qmrlqd25xmxwaf800abfm1k250zx01kx3dha2p6pk3vx89"))))
> +    (build-system python-build-system)
> +    (native-inputs (list unzip))
> +    (home-page "https://github.com/kmaork/pyinjector")
> +    (synopsis
> +      "A tool/library allowing dynamic library injection into running processes")
> +    (description
> +      "This package provides a tool/library allowing dynamic library injection into
> +running processes")
> +    (license license:expat)))
> +

Please run this through 'guix lint', and fix any problems reported (the
synopsis will be flagged, at least).  Your description should end with a
period (i.e., be a complete sentence).

I assume the test suite is not run; you may have to fetch from git to do
so, but there is a test suite:
https://github.com/kmaork/pyinjector/blob/master/setup.py.

Thanks,

Maxim




^ permalink raw reply	[flat|nested] 10+ messages in thread

* [bug#55233] [PATCH 0/3] Add python-madbg.
  2022-05-02 22:00   ` [bug#55233] [PATCH 2/3] gnu: Add python-hypno jgart via Guix-patches via
@ 2022-06-24 15:23     ` Maxim Cournoyer
  0 siblings, 0 replies; 10+ messages in thread
From: Maxim Cournoyer @ 2022-06-24 15:23 UTC (permalink / raw)
  To: jgart; +Cc: 55233

Hi,

jgart <jgart@dismail.de> writes:

> * gnu/packages/python-xyz.scm (python-hypno): New variable.
> ---
>  gnu/packages/python-xyz.scm | 21 +++++++++++++++++++++
>  1 file changed, 21 insertions(+)
>
> diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
> index 453080a4b7..61c36699c7 100644
> --- a/gnu/packages/python-xyz.scm
> +++ b/gnu/packages/python-xyz.scm
> @@ -29001,6 +29001,27 @@ (define-public python-pyinjector
>  running processes")
>      (license license:expat)))
>  
> +(define-public python-hypno
> +  (package
> +    (name "python-hypno")
> +    (version "0.1.4")
> +    (source
> +      (origin
> +        (method url-fetch)
> +        (uri (pypi-uri "hypno" version ".zip"))
> +        (sha256
> +          (base32 "1703c1k1g404sl2yvh5wylk7prnfgm7c83cpx2gv3n50025d6910"))))
> +    (build-system python-build-system)
> +    (propagated-inputs (list python-pyinjector))
> +    (native-inputs (list unzip))
> +    (home-page "https://github.com/kmaork/hypno")
> +    (synopsis
> +      "A tool/library allowing to inject python code into a running python process.")
> +    (description
> +      "This package provides a tool/library allowing to inject python code into a
> +running python process.")
> +    (license license:expat)))

The package synopsis/description is the same as for pyinjector.  Perhaps
it should say in the description that is based on it.

Otherwise, same comments as for python-pyinjector apply: run 'guix lint'
and fix any problem, and make sure the test suite is run [0].

[0]  https://github.com/kmaork/hypno/tree/master/tests

Thanks,

Maxim




^ permalink raw reply	[flat|nested] 10+ messages in thread

* [bug#55233] [PATCH 0/3] Add python-madbg.
  2022-05-02 22:00   ` [bug#55233] [PATCH 3/3] gnu: " jgart via Guix-patches via
@ 2022-06-24 15:25     ` Maxim Cournoyer
  0 siblings, 0 replies; 10+ messages in thread
From: Maxim Cournoyer @ 2022-06-24 15:25 UTC (permalink / raw)
  To: jgart; +Cc: 55233

jgart <jgart@dismail.de> writes:

> * gnu/packages/python-xyz.scm (python-madbg): New variable.
> ---
>  gnu/packages/python-xyz.scm | 20 ++++++++++++++++++++
>  1 file changed, 20 insertions(+)
>
> diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
> index 61c36699c7..551544b08e 100644
> --- a/gnu/packages/python-xyz.scm
> +++ b/gnu/packages/python-xyz.scm
> @@ -29022,6 +29022,26 @@ (define-public python-hypno
>  running python process.")
>      (license license:expat)))
>  
> +(define-public python-madbg
> +  (package
> +    (name "python-madbg")
> +    (version "1.2.0")
> +    (source
> +      (origin
> +        (method url-fetch)
> +        (uri (pypi-uri "madbg" version ".zip"))
> +        (sha256
> +          (base32 "19cll917w3zshxrcf2v5bdwkvbam3nslyik2hp4wpn5z4xkzsw5v"))))
> +    (build-system python-build-system)
> +    (propagated-inputs
> +      (list python-click python-hypno python-ipython python-prompt-toolkit))
> +    (native-inputs (list unzip))
> +    (home-page "https://github.com/kmaork/madbg")
> +    (synopsis "A fully-featured remote debugger for python")
> +    (description
> +      "This package provides a fully-featured remote debugger for python")
> +    (license license:expat)))

Synopsis shouldn't start with determinant (run 'guix lint' to find out).
Description should end with period.  I'd also capitalize "Python", since
it's a proper name.

Make sure the test suite is run [0].

Thanks!

[0]  https://github.com/kmaork/madbg/tree/master/tests




^ permalink raw reply	[flat|nested] 10+ messages in thread

* bug#55233: Closing
  2022-05-02 21:56 [bug#55233] [PATCH 0/3] Add python-madbg jgart via Guix-patches via
  2022-05-02 22:00 ` [bug#55233] [PATCH 1/3] gnu: Add python-pyinjector jgart via Guix-patches via
  2022-05-02 22:08 ` [bug#55233] linter warning jgart via Guix-patches via
@ 2023-10-07  5:53 ` jgart via Guix-patches via
  2 siblings, 0 replies; 10+ messages in thread
From: jgart via Guix-patches via @ 2023-10-07  5:53 UTC (permalink / raw)
  To: 55233-done

Hi,

I'm going to close this patch series I previously sent as won't do.

Feel free to re-open if anyone else wants to work on it.

all best,

jgart




^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2023-10-07  6:02 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-05-02 21:56 [bug#55233] [PATCH 0/3] Add python-madbg jgart via Guix-patches via
2022-05-02 22:00 ` [bug#55233] [PATCH 1/3] gnu: Add python-pyinjector jgart via Guix-patches via
2022-05-02 22:00   ` [bug#55233] [PATCH 2/3] gnu: Add python-hypno jgart via Guix-patches via
2022-06-24 15:23     ` [bug#55233] [PATCH 0/3] Add python-madbg Maxim Cournoyer
2022-05-02 22:00   ` [bug#55233] [PATCH 3/3] gnu: " jgart via Guix-patches via
2022-06-24 15:25     ` [bug#55233] [PATCH 0/3] " Maxim Cournoyer
2022-06-24 15:21   ` Maxim Cournoyer
2022-05-02 22:08 ` [bug#55233] linter warning jgart via Guix-patches via
2022-05-03  8:56   ` Maxime Devos
2023-10-07  5:53 ` bug#55233: Closing jgart via Guix-patches via

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).