[debbugs-tracker] bug#28061: closed ([PATCH] gnu: moreutils: Add more inputs to moreutils.)

[GNU bug Tracking System]

Your message dated Sat, 12 Aug 2017 22:11:27 +0100
with message-id <address@hidden>
and subject line Re: [bug#28061] [PATCH] gnu: moreutils: Add more inputs to 
moreutils.
has caused the debbugs.gnu.org bug report #28061,
regarding [PATCH] gnu: moreutils: Add more inputs to moreutils.
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
28061: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=28061
GNU Bug Tracking System
Contact address@hidden with problems
> --- Begin Message ---
>
>
>
> Subject: 
>
> [PATCH] gnu: moreutils: Add more inputs to moreutils.
>
>
>
>
> Date: 
>
> Sat, 12 Aug 2017 12:43:39 +0100
>
>
>
>
> 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
>
>
>
>
> --- End Message ---

--- Begin Message --- Subject: Re: [bug#28061] [PATCH] gnu: moreutils: Add more inputs to moreutils. Date: Sat, 12 Aug 2017 22:11:27 +0100

On Sat, 12 Aug 2017 14:18:26 +0200
Marius Bakke <address@hidden> wrote:

> Christopher Baines <address@hidden> 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.

pgpZovx9AP2Kj.pgp
Description: OpenPGP digital signature

--- End Message ---