all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#28061] [PATCH] gnu: moreutils: Add more inputs to moreutils.
@ 2017-08-12 11:43 Christopher Baines
  2017-08-12 12:18 ` Marius Bakke
  0 siblings, 1 reply; 3+ messages in thread
From: Christopher Baines @ 2017-08-12 11:43 UTC (permalink / raw)
  To: 28061

ts requires the Time::Duration and Date::Parse perl modules for the -r option.

* gnu/packages/moreutils.scm (moreutils)[inputs]: Add perl-timedate and
  perl-time-duration.
  [arguments]: Wrap ts with PERL5LIB.
---
 gnu/packages/moreutils.scm | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/moreutils.scm b/gnu/packages/moreutils.scm
index 03ffa8f99..bb6228af7 100644
--- a/gnu/packages/moreutils.scm
+++ b/gnu/packages/moreutils.scm
@@ -52,10 +52,18 @@
        ("libxml2" ,libxml2)
        ("libxslt" ,libxslt)))
     (inputs
-     `(("perl" ,perl)))
+     `(("perl" ,perl)
+       ("perl-timedate" ,perl-timedate)
+       ("perl-time-duration" ,perl-time-duration)))
     (arguments
      `(#:phases
        (modify-phases %standard-phases
+         (add-after 'install 'wrap-program
+                    (lambda* (#:key outputs #:allow-other-keys)
+                      (let* ((out (assoc-ref outputs "out")))
+                        (wrap-program
+                            (string-append out "/bin/ts")
+                          `("PERL5LIB" ":" prefix (,(getenv "PERL5LIB")))))))
          (delete 'configure))           ; no configure script
        #:make-flags
        (list (string-append "PREFIX=" (assoc-ref %outputs "out"))
-- 
2.13.1

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

* [bug#28061] [PATCH] gnu: moreutils: Add more inputs to moreutils.
  2017-08-12 11:43 [bug#28061] [PATCH] gnu: moreutils: Add more inputs to moreutils Christopher Baines
@ 2017-08-12 12:18 ` Marius Bakke
  2017-08-12 21:11   ` bug#28061: " Christopher Baines
  0 siblings, 1 reply; 3+ messages in thread
From: Marius Bakke @ 2017-08-12 12:18 UTC (permalink / raw)
  To: Christopher Baines, 28061

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

Christopher Baines <mail@cbaines.net> writes:

> ts requires the Time::Duration and Date::Parse perl modules for the -r option.
>
> * gnu/packages/moreutils.scm (moreutils)[inputs]: Add perl-timedate and
>   perl-time-duration.
>   [arguments]: Wrap ts with PERL5LIB.

LGTM. Hopefully we can get #27003 et.al in with the next core-updates
cycle so the wrapping is done automatically.

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

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

* bug#28061: [PATCH] gnu: moreutils: Add more inputs to moreutils.
  2017-08-12 12:18 ` Marius Bakke
@ 2017-08-12 21:11   ` Christopher Baines
  0 siblings, 0 replies; 3+ messages in thread
From: Christopher Baines @ 2017-08-12 21:11 UTC (permalink / raw)
  To: Marius Bakke; +Cc: 28061-done

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

On Sat, 12 Aug 2017 14:18:26 +0200
Marius Bakke <mbakke@fastmail.com> wrote:

> Christopher Baines <mail@cbaines.net> writes:
> 
> > ts requires the Time::Duration and Date::Parse perl modules for the
> > -r option.
> >
> > * gnu/packages/moreutils.scm (moreutils)[inputs]: Add perl-timedate
> > and perl-time-duration.
> >   [arguments]: Wrap ts with PERL5LIB.  
> 
> LGTM. Hopefully we can get #27003 et.al in with the next core-updates
> cycle so the wrapping is done automatically.

Thanks for your review Marius :) I've now pushed this.

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 963 bytes --]

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

end of thread, other threads:[~2017-08-12 21:12 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-08-12 11:43 [bug#28061] [PATCH] gnu: moreutils: Add more inputs to moreutils Christopher Baines
2017-08-12 12:18 ` Marius Bakke
2017-08-12 21:11   ` bug#28061: " Christopher Baines

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.