GNU bug report logs - #51048
No license in crate - guix import

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#51048: closed (No license in crate - guix import)
Date: Thu, 07 Oct 2021 08:53:01 +0000

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

Your message dated Thu, 07 Oct 2021 10:52:12 +0200
with message-id <87ily9fc8z.fsf <at> gnu.org>
and subject line Re: bug#51048: No license in crate - guix import
has caused the debbugs.gnu.org bug report #51048,
regarding No license in crate - guix import
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
51048: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=51048
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Michael Zappa <me <at> michzappa.com>
To: bug-guix <at> gnu.org
Subject: No license in crate - guix import
Date: Tue, 05 Oct 2021 18:31:08 -0400

Hello all,
I have been playing around with the 'guix import' tools to
see how easily I can get some package definitions. In the process of
trying to package https://github.com/Spotifyd/spotifyd with 'guix import
crate spotifyd -r' I found that one of the nested dependencies,
libpulse-sys <at> 0.0.0 did not work with the automatic importer because it
does not have a license in its crate
https://crates.io/crates/libpulse-sys/0.0.0.

Obviously it would be ideal to get whoever is using this out-of-date
library in their package to update their dependencies so this is
entirely avoided, but short of that has there ever been discussion on
how to handle 'license-less' packages? I haven't seen any in my short
time lurking on this list. It seems to be a rigid requirement for the
crate importer.

Backtrace:
michael <at> ordenagailua ~ $ guix import crate libpulse-sys <at> 0.0.0 -r
./pre-inst-env: line 33: cd: @abs_top_srcdir@: No such file or directory
./pre-inst-env: line 34: cd: @abs_top_builddir@: No such file or directory
Backtrace:
          10 (primitive-load "/home/michael/.config/guix/current/bin…")
In guix/ui.scm:
   2185:7  9 (run-guix . _)
  2148:10  8 (run-guix-command _ . _)
In guix/scripts/import.scm:
   124:11  7 (guix-import . _)
In guix/import/utils.scm:
   495:27  6 (recursive-import _ #:repo->guix-package _ #:guix-name _ …)
   485:33  5 (lookup-node "libpulse-sys" "0.0.0")
In guix/memoization.scm:
     98:0  4 (mproc "libpulse-sys" #:version "0.0.0" #:repo #f # #t)
In unknown file:
           3 (_ #<procedure 7fcb23655e80 at guix/memoization.scm:17…> …)
In guix/import/crate.scm:
   316:37  2 (crate->guix-package "libpulse-sys" #:version _ # _ # _)
   213:14  1 (string->license _)
In unknown file:
           0 (string-split null #<charset {#\space #\/}>)

ERROR: In procedure string-split:
In procedure string-split: Wrong type argument in position 1 (expecting string): null


Thank you,
Michael

[Message part 3 (message/rfc822, inline)]

From: Ludovic Courtès <ludo <at> gnu.org>
To: Maxime Devos <maximedevos <at> telenet.be>
Cc: Michael Zappa <me <at> michzappa.com>, 51048-done <at> debbugs.gnu.org
Subject: Re: bug#51048: No license in crate - guix import
Date: Thu, 07 Oct 2021 10:52:12 +0200

Hi,

Maxime Devos <maximedevos <at> telenet.be> skribis:

> Michael Zappa schreef op di 05-10-2021 om 18:31 [-0400]:
>> Hello all,
>> I have been playing around with the 'guix import' tools to
>> see how easily I can get some package definitions. In the process of
>> trying to package https://github.com/Spotifyd/spotifyd with 'guix import
>> crate spotifyd -r' I found that one of the nested dependencies,
>> libpulse-sys <at> 0.0.0 did not work with the automatic importer because it
>> does not have a license in its crate
>> https://crates.io/crates/libpulse-sys/0.0.0.
>> 
>> Obviously it would be ideal to get whoever is using this out-of-date
>> library in their package to update their dependencies so this is
>> entirely avoided, but short of that has there ever been discussion on
>> how to handle 'license-less' packages? I haven't seen any in my short
>> time lurking on this list. It seems to be a rigid requirement for the
>> crate importer.
>
> I don't now if there has been a discussion,
> but other importers (at least the minetest importer) set the license
> field to #f if no license information was unavailable.
>
> Modifying <crate-version> such that 'license' is set to #f if it has 
> 'null' as value in the JSON might be sufficient I think?

Indeed.  Fixed in 1327ec822fa6dd396e979efd8d4e1f7479f1d5b3.

Thanks!

Ludo’.

Previous Next

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