all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#50235] [PATCH] gnu: trash-cli: Update to 0.21.7.24.
@ 2021-08-28  7:48 Brice Waegeneire
  2021-09-24 11:56 ` Ludovic Courtès
  0 siblings, 1 reply; 3+ messages in thread
From: Brice Waegeneire @ 2021-08-28  7:48 UTC (permalink / raw)
  To: 50235

* gnu/packages/shellutils.scm (trash-cli): Update to 0.21.7.24.
  [source]: Use github repository instead of pypi.
  [arguments]<python>: Use python 3.
  <tests>: Enable tests.
  <phases>: Add "fix-setup.py" phase.
  [native-inputs]: Add python-pytest, python-mock and python-six.
  [propagated-inputs]: Add python-psutil.
---
 gnu/packages/shellutils.scm | 37 +++++++++++++++++++++++++++----------
 1 file changed, 27 insertions(+), 10 deletions(-)

The switch from PyPi to Github is to get the "test" directory to be able to
run the tests.

diff --git a/gnu/packages/shellutils.scm b/gnu/packages/shellutils.scm
index 94b5536df7..7c153cdf25 100644
--- a/gnu/packages/shellutils.scm
+++ b/gnu/packages/shellutils.scm
@@ -38,11 +38,13 @@
   #:use-module (gnu packages autotools)
   #:use-module (gnu packages base)
   #:use-module (gnu packages bison)
+  #:use-module (gnu packages check)
   #:use-module (gnu packages flex)
   #:use-module (gnu packages golang)
   #:use-module (gnu packages ncurses)
   #:use-module (gnu packages pkg-config)
   #:use-module (gnu packages python)
+  #:use-module (gnu packages python-xyz)
   #:use-module (gnu packages readline)
   #:use-module (gnu packages ruby)
   #:use-module (gnu packages shells)
@@ -203,19 +205,20 @@ between various shells or commands.")
 (define-public trash-cli
   (package
     (name "trash-cli")
-    (version "0.17.1.14")
+    (version "0.21.7.24")
     (source
      (origin
-       (method url-fetch)
-       (uri (pypi-uri "trash-cli" version))
-       (sha256
-        (base32
-         "01q0cl04ljf214z6s3g256gsxx3pqsgaf6ac1zh0vrq5bnhnr85h"))))
+      (method git-fetch)
+      (uri (git-reference
+            (url "https://github.com/andreafrancia/trash-cli")
+            (commit version)))
+      (file-name (git-file-name name version))
+      (sha256
+       (base32
+        "082mfl4mza4xkm3fdn5aka9rsbd8c149qj3cqqk62idrnszmgzg5"))))
     (build-system python-build-system)
     (arguments
-     `(#:python ,python-2
-       #:tests? #f ; no tests
-       #:phases
+     `(#:phases
        (modify-phases %standard-phases
          (add-before 'build 'patch-path-constants
            (lambda* (#:key inputs #:allow-other-keys)
@@ -225,8 +228,22 @@ between various shells or commands.")
                  (("\"/lib/libc.so.6\".*")
                   (string-append "\"" libc "/lib/libc.so.6\"\n"))
                  (("\"df\"")
-                  (string-append "\"" coreutils "/bin/df\"")))))))))
+                  (string-append "\"" coreutils "/bin/df\""))))))
+         (add-before 'build 'fix-setup.py
+           (lambda* (#:key outputs #:allow-other-keys)
+             (let* ((out (assoc-ref outputs "out"))
+                    (bin (string-append out "/bin")))
+               (mkdir-p bin)
+               (substitute* "setup.py"
+                 (("add_script\\('")
+                  (string-append "add_script('" bin "/" )))))))))
+    (native-inputs
+     `(("python-pytest" ,python-pytest)
+       ("python-mock" ,python-mock)
+       ("python-six" ,python-six)))
     (inputs `(("coreutils" ,coreutils)))
+    (propagated-inputs
+     `(("python-psutil" ,python-psutil)))
     (home-page "https://github.com/andreafrancia/trash-cli")
     (synopsis "Trash can management tool")
     (description

base-commit: 4716d544443f25a4cd1397d9f8deb856e28bb515
-- 
2.32.0





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

* [bug#50235] [PATCH] gnu: trash-cli: Update to 0.21.7.24.
  2021-08-28  7:48 [bug#50235] [PATCH] gnu: trash-cli: Update to 0.21.7.24 Brice Waegeneire
@ 2021-09-24 11:56 ` Ludovic Courtès
  2022-01-18  7:58   ` bug#50235: " Brice Waegeneire
  0 siblings, 1 reply; 3+ messages in thread
From: Ludovic Courtès @ 2021-09-24 11:56 UTC (permalink / raw)
  To: Brice Waegeneire; +Cc: 50235

Hi Brice,

Brice Waegeneire <brice@waegenei.re> skribis:

> * gnu/packages/shellutils.scm (trash-cli): Update to 0.21.7.24.
>   [source]: Use github repository instead of pypi.
>   [arguments]<python>: Use python 3.
>   <tests>: Enable tests.
>   <phases>: Add "fix-setup.py" phase.
>   [native-inputs]: Add python-pytest, python-mock and python-six.
>   [propagated-inputs]: Add python-psutil.

LGTM!

Ludo’.




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

* bug#50235: [PATCH] gnu: trash-cli: Update to 0.21.7.24.
  2021-09-24 11:56 ` Ludovic Courtès
@ 2022-01-18  7:58   ` Brice Waegeneire
  0 siblings, 0 replies; 3+ messages in thread
From: Brice Waegeneire @ 2022-01-18  7:58 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 50235-done

Hello Ludo’,

Ludovic Courtès <ludo@gnu.org> writes:

> Brice Waegeneire <brice@waegenei.re> skribis:
>
>> * gnu/packages/shellutils.scm (trash-cli): Update to 0.21.7.24.
>>   [source]: Use github repository instead of pypi.
>>   [arguments]<python>: Use python 3.
>>   <tests>: Enable tests.
>>   <phases>: Add "fix-setup.py" phase.
>>   [native-inputs]: Add python-pytest, python-mock and python-six.
>>   [propagated-inputs]: Add python-psutil.
>
> LGTM!

Thank for the review!

Pushed as 39510d5a5e4e0e38cd3d3bc944d5bf6d96242493, adjusted from the initial
patch to update to 0.21.10.24 and use new inputs style.

Cheers,
- Brice




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

end of thread, other threads:[~2022-01-18  8:14 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-28  7:48 [bug#50235] [PATCH] gnu: trash-cli: Update to 0.21.7.24 Brice Waegeneire
2021-09-24 11:56 ` Ludovic Courtès
2022-01-18  7:58   ` bug#50235: " Brice Waegeneire

Code repositories for project(s) associated with this external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.