GNU bug report logs - #31041
[PATCH] gnu: Add perl-pathtools.

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 31041 in the body.
You can then email your comments to 31041 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Roel Janssen <roel <at> gnu.org>
To: guix-patches <guix-patches <at> gnu.org>
Subject: [PATCH] gnu: Add perl-pathtools.
Date: Tue, 03 Apr 2018 15:09:17 +0200

[Message part 1 (text/plain, inline)]

Dear Guix,

The attached patch would add perl-pathtools.  This package searches for
the “pwd” program in specific paths (ignoring the PATH environment
variable).  Because “pwd” is part of “coreutils”, it has a funny
dependency.

In the path I substitute the entry for “/bin/pwd” to the store path of
coreutils's “pwd”.

Is this the right way of packaging perl-pathtools?

Kind regards,
Roel Janssen

[0001-gnu-Add-perl-pathtools.patch (text/x-patch, attachment)]

Message #8 received at 31041 <at> debbugs.gnu.org (full text, mbox):

From: ludo <at> gnu.org (Ludovic Courtès)
To: Roel Janssen <roel <at> gnu.org>
Cc: 31041 <at> debbugs.gnu.org
Subject: Re: [bug#31041] [PATCH] gnu: Add perl-pathtools.
Date: Wed, 04 Apr 2018 13:52:50 +0200

Hello,

Roel Janssen <roel <at> gnu.org> skribis:

> From a9adb8d30747afb47284baaca898261a012a1d03 Mon Sep 17 00:00:00 2001
> From: Roel Janssen <roel <at> gnu.org>
> Date: Tue, 3 Apr 2018 15:00:15 +0200
> Subject: [PATCH] gnu: Add perl-pathtools.
>
> * gnu/packages/perl.scm (perl-pathtools): New variable.

[...]

> +    (license (package-license perl))))

Please change this as discussed before.

Otherwise LGTM!

Thanks,
Ludo’.

Message #13 received at 31041-done <at> debbugs.gnu.org (full text, mbox):

From: Roel Janssen <roel <at> gnu.org>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 31041-done <at> debbugs.gnu.org
Subject: Re: [bug#31041] [PATCH] gnu: Add perl-pathtools.
Date: Wed, 04 Apr 2018 14:55:55 +0200

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

> Hello,
>
> Roel Janssen <roel <at> gnu.org> skribis:
>
>> From a9adb8d30747afb47284baaca898261a012a1d03 Mon Sep 17 00:00:00 2001
>> From: Roel Janssen <roel <at> gnu.org>
>> Date: Tue, 3 Apr 2018 15:00:15 +0200
>> Subject: [PATCH] gnu: Add perl-pathtools.
>>
>> * gnu/packages/perl.scm (perl-pathtools): New variable.
>
> [...]
>
>> +    (license (package-license perl))))
>
> Please change this as discussed before.
>
> Otherwise LGTM!

I adjusted the license field, and pushed in b028a9830.

Thanks!

Kind regards,
Roel Janssen

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.