unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#29419] [RFC PATCH] gnu: Add rename.
@ 2017-11-24  1:12 Marius Bakke
  2017-11-24 14:43 ` Leo Famulari
  2017-11-24 21:32 ` Ludovic Courtès
  0 siblings, 2 replies; 6+ messages in thread
From: Marius Bakke @ 2017-11-24  1:12 UTC (permalink / raw)
  To: 29419


[-- Attachment #1.1: Type: text/plain, Size: 388 bytes --]

Hello!

I packaged this to help rename a bunch of patches.

It installs an executable called 'rename' which conflicts with the
(limited) one in util-linux, due to the latter being part of
%base-packages.

Are there preferences to either:

* Install this one as 'perl-rename'; or
* Drop the `rename` executable from util-linux.

If there are no comments I'll go with the conservative #1.


[-- Attachment #1.2: 0001-gnu-Add-rename.patch --]
[-- Type: text/x-patch, Size: 2495 bytes --]

From 1065207246f5c5bd38ee0539e251ec6f4f65af22 Mon Sep 17 00:00:00 2001
From: Marius Bakke <mbakke@fastmail.com>
Date: Fri, 24 Nov 2017 01:27:18 +0100
Subject: [PATCH] gnu: Add rename.

* gnu/packages/admin.scm (rename): New public variable.
---
 gnu/packages/admin.scm | 29 +++++++++++++++++++++++++++++
 1 file changed, 29 insertions(+)

diff --git a/gnu/packages/admin.scm b/gnu/packages/admin.scm
index cd2b9a633..f604ff4d7 100644
--- a/gnu/packages/admin.scm
+++ b/gnu/packages/admin.scm
@@ -18,6 +18,7 @@
 ;;; Copyright © 2017 Ben Sturmfels <ben@sturm.com.au>
 ;;; Copyright © 2017 Ethan R. Jones <doubleplusgood23@gmail.com>
 ;;; Copyright © 2017 Christopher Allan Webber <cwebber@dustycloud.org>
+;;; Copyright © 2017 Marius Bakke <mbakke@fastmail.com>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -42,6 +43,7 @@
   #:use-module (guix git-download)
   #:use-module (guix build-system cmake)
   #:use-module (guix build-system gnu)
+  #:use-module (guix build-system perl)
   #:use-module (guix build-system python)
   #:use-module (guix build-system trivial)
   #:use-module (gnu packages)
@@ -755,6 +757,33 @@ console window to allow commands to be interactively run on multiple servers
 over ssh connections.")
     (license license:gpl2+)))
 
+(define-public rename
+  (package
+    (name "rename")
+    (version "0.20")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append
+                    "mirror://cpan/authors/id/R/RM/RMBARKER/File-Rename-"
+                    version ".tar.gz"))
+       (sha256
+        (base32
+         "1cf6xx2hiy1xalp35fh8g73j67r0w0g66jpcbc6971x9jbm7bvjy"))))
+    (build-system perl-build-system)
+    (native-inputs
+     `(("perl-module-build" ,perl-module-build)))
+    (propagated-inputs
+     `(("perl-test-pod" ,perl-test-pod)
+       ("perl-test-pod-coverage" ,perl-test-pod-coverage)))
+    (home-page "https://metacpan.org/pod/distribution/File-Rename/rename.PL")
+    (synopsis "Rename multiple files")
+    (description "@command{rename} renames the filenames supplied according
+to the Perl regular expression specified as the first argument. If a given
+filename is not modified by the expression, it will not be renamed.  If no
+filenames are given on the command line, filenames will be read via standard
+input.")
+    (license license:perl-license)))
+
 (define-public rottlog
   (package
     (name "rottlog")
-- 
2.15.0


[-- Attachment #1.3: Type: text/plain, Size: 16 bytes --]


Thanks,
Marius

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 487 bytes --]

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

* [bug#29419] [RFC PATCH] gnu: Add rename.
  2017-11-24  1:12 [bug#29419] [RFC PATCH] gnu: Add rename Marius Bakke
@ 2017-11-24 14:43 ` Leo Famulari
  2017-11-24 21:32 ` Ludovic Courtès
  1 sibling, 0 replies; 6+ messages in thread
From: Leo Famulari @ 2017-11-24 14:43 UTC (permalink / raw)
  To: Marius Bakke; +Cc: 29419

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

On Fri, Nov 24, 2017 at 02:12:51AM +0100, Marius Bakke wrote:
> Hello!
> 
> I packaged this to help rename a bunch of patches.
> 
> It installs an executable called 'rename' which conflicts with the
> (limited) one in util-linux, due to the latter being part of
> %base-packages.
> 
> Are there preferences to either:
> 
> * Install this one as 'perl-rename'; or
> * Drop the `rename` executable from util-linux.
> 
> If there are no comments I'll go with the conservative #1.

It looks like Debian uses this Perl rename instead of the one from
util-linux. So I'd be okay switching them. Do you have any idea if they
at all are compatible?

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

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

* [bug#29419] [RFC PATCH] gnu: Add rename.
  2017-11-24  1:12 [bug#29419] [RFC PATCH] gnu: Add rename Marius Bakke
  2017-11-24 14:43 ` Leo Famulari
@ 2017-11-24 21:32 ` Ludovic Courtès
  2017-11-25  0:12   ` Leo Famulari
  2018-01-07 13:42   ` Marius Bakke
  1 sibling, 2 replies; 6+ messages in thread
From: Ludovic Courtès @ 2017-11-24 21:32 UTC (permalink / raw)
  To: Marius Bakke; +Cc: 29419

Hello,

Marius Bakke <mbakke@fastmail.com> skribis:

> It installs an executable called 'rename' which conflicts with the
> (limited) one in util-linux, due to the latter being part of
> %base-packages.

Is this conflict a hindrance in practice?

> Are there preferences to either:
>
> * Install this one as 'perl-rename'; or
> * Drop the `rename` executable from util-linux.
>
> If there are no comments I'll go with the conservative #1.

Either way is fine with me.  (At some point I’d like to avoid Perl
dependencies in %base-packages if possible, though.)

The util-linux change would be for ‘core-updates’, which maybe doesn’t
address your immediate needs.

Ludo’.

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

* [bug#29419] [RFC PATCH] gnu: Add rename.
  2017-11-24 21:32 ` Ludovic Courtès
@ 2017-11-25  0:12   ` Leo Famulari
  2018-01-07 13:42   ` Marius Bakke
  1 sibling, 0 replies; 6+ messages in thread
From: Leo Famulari @ 2017-11-25  0:12 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 29419

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

On Fri, Nov 24, 2017 at 10:32:22PM +0100, Ludovic Courtès wrote:
> Either way is fine with me.  (At some point I’d like to avoid Perl
> dependencies in %base-packages if possible, though.)

util-linux also depends on Perl (not sure why).

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

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

* [bug#29419] [RFC PATCH] gnu: Add rename.
  2017-11-24 21:32 ` Ludovic Courtès
  2017-11-25  0:12   ` Leo Famulari
@ 2018-01-07 13:42   ` Marius Bakke
  2018-01-08 19:16     ` bug#29419: " Marius Bakke
  1 sibling, 1 reply; 6+ messages in thread
From: Marius Bakke @ 2018-01-07 13:42 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 29419

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

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

> Hello,
>
> Marius Bakke <mbakke@fastmail.com> skribis:
>
>> It installs an executable called 'rename' which conflicts with the
>> (limited) one in util-linux, due to the latter being part of
>> %base-packages.
>
> Is this conflict a hindrance in practice?

Only if you have util-linux in the same profile.  And there is always
`guix environment --pure`, so not really.

>> Are there preferences to either:
>>
>> * Install this one as 'perl-rename'; or
>> * Drop the `rename` executable from util-linux.
>>
>> If there are no comments I'll go with the conservative #1.
>
> Either way is fine with me.  (At some point I’d like to avoid Perl
> dependencies in %base-packages if possible, though.)

Actually, I changed my mind and would like to keep it as-is for now.
The main problem is that the man page is not easily accessible, but that
turned out to be a different bug: <https://bugs.gnu.org/30014>.

Will push this later when I figure out a synopsis that does not upset
`guix lint` :)

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 487 bytes --]

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

* bug#29419: [RFC PATCH] gnu: Add rename.
  2018-01-07 13:42   ` Marius Bakke
@ 2018-01-08 19:16     ` Marius Bakke
  0 siblings, 0 replies; 6+ messages in thread
From: Marius Bakke @ 2018-01-08 19:16 UTC (permalink / raw)
  To: 29419-done

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

Pushed as d1ce1125b80b0dfba4b665293a6bb0c611025584.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 487 bytes --]

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

end of thread, other threads:[~2018-01-08 19:17 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-24  1:12 [bug#29419] [RFC PATCH] gnu: Add rename Marius Bakke
2017-11-24 14:43 ` Leo Famulari
2017-11-24 21:32 ` Ludovic Courtès
2017-11-25  0:12   ` Leo Famulari
2018-01-07 13:42   ` Marius Bakke
2018-01-08 19:16     ` bug#29419: " Marius Bakke

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).