GNU bug report logs -
#46391
[PATCH 0/1] Canonicalize-path load-path option
Previous Next
Reported by: zimoun <zimon.toutoune <at> gmail.com>
Date: Tue, 9 Feb 2021 00:41:01 UTC
Severity: normal
Tags: patch
Done: Ludovic Courtès <ludo <at> gnu.org>
Bug is archived. No further changes may be made.
To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 46391 in the body.
You can then email your comments to 46391 AT debbugs.gnu.org in the normal way.
Toggle the display of automated, internal messages from the tracker.
Report forwarded
to
guix-patches <at> gnu.org
:
bug#46391
; Package
guix-patches
.
(Tue, 09 Feb 2021 00:41:02 GMT)
Full text and
rfc822 format available.
Acknowledgement sent
to
zimoun <zimon.toutoune <at> gmail.com>
:
New bug report received and forwarded. Copy sent to
guix-patches <at> gnu.org
.
(Tue, 09 Feb 2021 00:41:02 GMT)
Full text and
rfc822 format available.
Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):
Hi,
This fixes <https://bugs.gnu.org/46390>.
The test suite does not check relative path for the load-path option.
I do not know what is the best to add a test about that.
Last, I remember discussions about absolute/relative path and I do not know
if this 'canonicalize-path' addition is the best fix here.
All the best,
simon
zimoun (1):
scripts: build: Fix relative load-path.
guix/scripts/build.scm | 16 +++++++++-------
1 file changed, 9 insertions(+), 7 deletions(-)
base-commit: 4cd8bab7d4b345c5538b8d039d84a957df7a235f
--
2.29.2
Reply sent
to
Ludovic Courtès <ludo <at> gnu.org>
:
You have taken responsibility.
(Wed, 24 Feb 2021 22:48:02 GMT)
Full text and
rfc822 format available.
Notification sent
to
zimoun <zimon.toutoune <at> gmail.com>
:
bug acknowledged by developer.
(Wed, 24 Feb 2021 22:48:02 GMT)
Full text and
rfc822 format available.
Message #10 received at 46391-done <at> debbugs.gnu.org (full text, mbox):
Hi!
zimoun <zimon.toutoune <at> gmail.com> skribis:
> This fixes <https://bugs.gnu.org/46390>.
>
> The test suite does not check relative path for the load-path option.
> I do not know what is the best to add a test about that.
>
> Last, I remember discussions about absolute/relative path and I do not know
> if this 'canonicalize-path' addition is the best fix here.
>
>
> All the best,
> simon
>
>
> zimoun (1):
> scripts: build: Fix relative load-path.
>
> guix/scripts/build.scm | 16 +++++++++-------
> 1 file changed, 9 insertions(+), 7 deletions(-)
I’m discovering this after having independently addreses #46390, sorry
about that! (Perhaps we should always send patches that fix bugs to the
corresponding bug report rather than/in addition to guix-patches?)
Let me know what you think.
Thanks,
Ludo’.
Information forwarded
to
guix-patches <at> gnu.org
:
bug#46391
; Package
guix-patches
.
(Wed, 24 Feb 2021 23:13:02 GMT)
Full text and
rfc822 format available.
Message #13 received at 46391-done <at> debbugs.gnu.org (full text, mbox):
Hi Ludo,
On Wed, 24 Feb 2021 at 23:46, Ludovic Courtès <ludo <at> gnu.org> wrote:
> I’m discovering this after having independently addreses #46390, sorry
> about that! (Perhaps we should always send patches that fix bugs to the
> corresponding bug report rather than/in addition to guix-patches?)
Anyway, because your patch is better than mine. :-)
In general, I try to at least mention in bug-guix the number from
guix-patches. Sometimes, I directly send the patch to bug-guix. It
depends on my mood I guess. :-)
Well, I do not have an opinion.
Maybe tiny patch (like this) that fixes bug should be sent to the
corresponding bug report. And larger patches should be sent to
guix-patches; it eases to keep separated the discussion about the bug
and about the patches, IMHO.
Cheers,
simon
Information forwarded
to
guix-patches <at> gnu.org
:
bug#46391
; Package
guix-patches
.
(Wed, 03 Mar 2021 13:31:04 GMT)
Full text and
rfc822 format available.
Message #16 received at 46391-done <at> debbugs.gnu.org (full text, mbox):
Hi,
On Tue, 09 Feb 2021 at 01:40, zimoun <zimon.toutoune <at> gmail.com> wrote:
> This fixes <https://bugs.gnu.org/46390>.
Fixed by 9a38bed2cf32e9462badfa43e74cdd4580e804fc.
> The test suite does not check relative path for the load-path option.
> I do not know what is the best to add a test about that.
>
> Last, I remember discussions about absolute/relative path and I do not know
> if this 'canonicalize-path' addition is the best fix here.
[...]
> zimoun (1):
> scripts: build: Fix relative load-path.
I even forgot to send the patch. :-)
Anyway, closing!
All the best,
simon
bug archived.
Request was from
Debbugs Internal Request <help-debbugs <at> gnu.org>
to
internal_control <at> debbugs.gnu.org
.
(Thu, 01 Apr 2021 11:24:07 GMT)
Full text and
rfc822 format available.
This bug report was last modified 3 years and 24 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.