emacs-bug-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#51048: closed (No license in crate - guix import)


From: GNU bug Tracking System
Subject: bug#51048: closed (No license in crate - guix import)
Date: Thu, 07 Oct 2021 08:53:01 +0000

Your message dated Thu, 07 Oct 2021 10:52:12 +0200
with message-id <87ily9fc8z.fsf@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@gnu.org.)


-- 
51048: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=51048
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems
--- Begin Message --- 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@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@ordenagailua ~ $ guix import crate libpulse-sys@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



--- End Message ---
--- Begin Message --- Subject: Re: bug#51048: No license in crate - guix import Date: Thu, 07 Oct 2021 10:52:12 +0200 User-agent: Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux)
Hi,

Maxime Devos <maximedevos@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@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’.


--- End Message ---

reply via email to

[Prev in Thread] Current Thread [Next in Thread]