GNU bug report logs - #26747
[PATCH] gnu: Avoid circular dependencies by perl license

Previous Next

Package: guix-patches;

Reported by: Petter <petter <at> mykolab.ch>

Date: Tue, 2 May 2017 15:49:02 UTC

Severity: normal

Tags: patch

Done: ludo <at> gnu.org (Ludovic Courtès)

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 26747 in the body.
You can then email your comments to 26747 AT debbugs.gnu.org in the normal way.

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

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to guix-patches <at> gnu.org:
bug#26747; Package guix-patches. (Tue, 02 May 2017 15:49:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to Petter <petter <at> mykolab.ch>:
New bug report received and forwarded. Copy sent to guix-patches <at> gnu.org. (Tue, 02 May 2017 15:49:02 GMT) Full text and rfc822 format available.

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

From: Petter <petter <at> mykolab.ch>
To: guix-patches <at> gnu.org
Subject: [PATCH] gnu: Avoid circular dependencies by perl license
Date: Tue, 2 May 2017 17:46:48 +0200
[Message part 1 (text/plain, inline)]
Hi Guix,

I've never been able to add modules to perl.scm. Today civodul showed me that
it's because "(package-license perl)" in at least a few other files were causing
circular dependencies with perl.scm. I don't know how to figure out exactly
which were causing this for perl.scm, so I've changed this in every file with
this (except perl.scm) to have "license:perl-license" instead. (Fixing just
what I'd have needed seems like leaving a problem for the next person anyway.)

With this patch I can add modules to perl.scm :)

Best,
Petter


P.S.  The error I got was the same as I get with most of my errors: "unknown
package". At this point I'm beginning to interpret "unknown package" as
"something failed". I find it that unhelpful, but more misleading. Maybe we
should discuss this.
[0001-gnu-Avoid-circular-dependencies-by-perl-license.patch (text/x-patch, attachment)]

Reply sent to ludo <at> gnu.org (Ludovic Courtès):
You have taken responsibility. (Tue, 02 May 2017 21:30:03 GMT) Full text and rfc822 format available.

Notification sent to Petter <petter <at> mykolab.ch>:
bug acknowledged by developer. (Tue, 02 May 2017 21:30:03 GMT) Full text and rfc822 format available.

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

From: ludo <at> gnu.org (Ludovic Courtès)
To: Petter <petter <at> mykolab.ch>
Cc: 26747-done <at> debbugs.gnu.org
Subject: Re: bug#26747: [PATCH] gnu: Avoid circular dependencies by perl
 license
Date: Tue, 02 May 2017 23:29:22 +0200
Hello Petter,

Petter <petter <at> mykolab.ch> skribis:

> I've never been able to add modules to perl.scm. Today civodul showed me that
> it's because "(package-license perl)" in at least a few other files were causing
> circular dependencies with perl.scm. I don't know how to figure out exactly
> which were causing this for perl.scm, so I've changed this in every file with
> this (except perl.scm) to have "license:perl-license" instead. (Fixing just
> what I'd have needed seems like leaving a problem for the next person anyway.)
>
> With this patch I can add modules to perl.scm :)

Nice.  :-)

> P.S.  The error I got was the same as I get with most of my errors: "unknown
> package". At this point I'm beginning to interpret "unknown package" as
> "something failed". I find it that unhelpful, but more misleading. Maybe we
> should discuss this.

Yeah, it’s tricky.  In this case, what happened is that it failed to
load (gnu packages perl).  Thus, the best it could do would be to print
the infamous “no code for module (gnu packages perl)”, which wouldn’t
help.

What would help would be detection circular dependencies due to
top-level references in the first place.

> From 7c39ae8fe3b956eb291fd1bc5ba86297cefb67b8 Mon Sep 17 00:00:00 2001
> From: Petter <petter <at> mykolab.ch>
> Date: Tue, 2 May 2017 16:57:52 +0200
> Subject: [PATCH] gnu: Avoid circular dependencies by perl license
>
> * guix/licenses.scm: New variable.
> * gnu/packages/bioinformatics.scm: Change (package-license perl) to perl-license.
> * gnu/packages/compression.scm: Same.
> * gnu/packages/databases.scm: Same.
> * gnu/packages/gd.scm: Same.
> * gnu/packages/language.scm: Same.
> * gnu/packages/libevent.scm: Same.
> * gnu/packages/mail.scm: Same.
> * gnu/packages/messaging.scm: Same.
> * gnu/packages/ncurses.scm: Same.
> * gnu/packages/networking.scm: Same.
> * gnu/packages/photo.scm: Same.
> * gnu/packages/tcl.scm: Same.
> * gnu/packages/tex.scm: Same.
> * gnu/packages/tls.scm: Same.
> * gnu/packages/web.scm: Same.
> * gnu/packages/wm.scm: Same.
> * gnu/packages/xml.scm: Same.
> * gnu/packages/xorg.scm: Same.
> * gnu/packages/zip.scm: Same.

[...]

> +(define perl-license gpl1+)

I added a comment here and committed.

Thank you!

Ludo’.




bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Wed, 31 May 2017 11:24:04 GMT) Full text and rfc822 format available.

This bug report was last modified 8 years and 20 days ago.

Previous Next


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