Re: [PATCH 2/2] gnu: julia: Use unbundled release tarball.

[ludo@gnu.org (Ludovic Courtès)]

Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de> skribis:

> Efraim Flashner <efraim@flashner.co.il> writes:
>
>>> +       ;; FIXME: The following inputs are downloaded from upstream to allow us
>>> +       ;; to use the lightweight Julia release tarball.  Ideally, these inputs
>>> +       ;; would eventually be replaced with proper Guix packages.
>>> +       ("rmath-julia"
>>> +        ,(origin
>>> +           (method url-fetch)
>>> +           (uri "https://api.github.com/repos/JuliaLang/Rmath-julia/tarball/v0.1")
>>> +           ;;(file-name "rmath-julia-0.1.tar.gz")
>>              ^^^
>>         did this not like being renamed?
>
> Oops, this was not supposed to be commented.  There was a problem where
> the build system complained about the hash of the tarball, and I could
> only fix it by *not* overriding the name.  But this shouldn’t have been
> part of the commit.  I’ll play with this again to see if it can stay or
> needs to go.
>
>>> +           (sha256
>>> +            (base32
>>> +             "0ai5dhjc43zcvangz123ryxmlbm51s21rg13bllwyn98w67arhb4"))))
>>> +       ("suitesparse"
>>> +        ,(origin
>>> +           (method url-fetch)
>>> +           (uri "http://faculty.cse.tamu.edu/davis/SuiteSparse/SuiteSparse-4.4.2.tar.gz")
>>> +           (sha256
>>> +            (base32
>>> +             "1dg0qsv07n71nbn9cgcvn73933rgy1jnxw5bfqkwfq3bidk44cqc"))))
>>> +       ("objconv"
>>> +        ,(origin
>>> +           (method url-fetch)
>>> +           (uri "http://www.agner.org/optimize/objconv.zip")
>>> +           (sha256
>>> +            (base32
>>> +             "1fi7qa2sd9vb35dvkgripjf0fayzg2qmff215f8agfqfiwd1g8qs"))))
>>> +       ("dsfmt"
>>> +        ,(origin
>>> +           (method url-fetch)
>>> +           (uri (string-append
>>> +                 "http://www.math.sci.hiroshima-u.ac.jp/~m-mat/MT/"
>>> +                 "SFMT/dSFMT-src-2.2.3.tar.gz"))
>>> +           (sha256
>>> +            (base32
>>> +             "03kaqbjbi6viz0n33dk5jlf6ayxqlsq4804n7kwkndiga9s4hd42"))))
>>> +       ("virtualenv"
>>
>> we actually do have virtualenv packaged. I forgot when but at some point
>> they made a massive jump in version numbers.
>
> Yes, I’ve seen that we have it, but it looks like julia doesn’t make it
> easy to use our packaged version (and it insists on this old version).
> We should fix this eventually, but the goal of this patch was to make
> this dependency explicit — before that it would just quietly take the
> bundled “virtualenv” in the fat tarball.

This patch LGTM, but I agree that eventually it’d be nice if Julia could
use the packaged versions of these things, at least SuiteSparse and
virtualenv, but probably all of them.

Thanks!

Ludo’.