[bug#28061] [PATCH] gnu: moreutils: Add more inputs to moreutils.

[bug#28061] [PATCH] gnu: moreutils: Add more inputs to moreutils.

[Christopher Baines]

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

[bug#28061] [PATCH] gnu: moreutils: Add more inputs to moreutils.

[Marius Bakke]

[-- 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 --]

bug#28061: [PATCH] gnu: moreutils: Add more inputs to moreutils.

[Christopher Baines]

[-- 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 --]