[Bug 1040517] Review Request: julia - High-level, high-performance dynamic language for technical computing

[bugzilla at redhat.com]

https://bugzilla.redhat.com/show_bug.cgi?id=1040517



--- Comment #42 from Baurzhan Muftakhidinov <baurthefirst at gmail.com> ---
(In reply to Milan Bouchet-Valat from comment #41)
> Yes, I've considered EPEL, but as a first step I wanted to get the package
> working for Fedora. But thanks for looking at it, I can make sure the
> package is clean for EPEL.
> 
> (In reply to Baurzhan Muftakhidinov from comment #40)
> > (In reply to Milan Bouchet-Valat from comment #38)
> > > Could anybody on 32-bit check whether Julia starts correctly with the
> > > following package on F20? What's changed is that it should now run on all
> > > i386 CPUs, not only on recent ones (but checking that it works somewhere is
> > > a good start ;-). Thanks!
> > > 
> > > http://copr-be.cloud.fedoraproject.org/results/nalimilan/julia/fedora-20-
> > > i386/julia-0.3.0-2.fc20/
> > 
> > 1) In your spec file of julia, line 41, you have
> > 
> > %if 0%{fedora} < 20
> > 
> > which gives error when building on epel7, I changed it to
> > 
> > %if 0%{?fedora} < 20
> Thanks, I fixed that.
> 
> > 2) Due to LLVM being shipped as llvm3.3 package, I changed that line again,
> > to
> > 
> > %if 0%{?fedora} < 20 && 0%{?rhel} < 6
> Done.

Hi,
I put rhel version there to ensure that build passes. Otherwise, without
second check, %if 0%{?fedora} < 20 was true for rhel7 and build searched for
llvm-3.3 and not for llvm33.
Correct check should be like
%if 0%{?fedora} < 20 AND NOT 0%{?rhel} but I don't know how to write it
correctly.

Regards,

> > 3) Line 33
> > BuildRequires:  double-conversion-devel >= 1.1.1
> > 
> > gave build error, changed to 
> > 
> > BuildRequires:  double-conversion-devel >= 2.0.0
> I don't get this. We only need 1.1.1, and 2.0.0 is clearly greater than
> 1.1.1, so it should work.
> 
> > 4) Required packages built for epel7. What is the policy of adding new
> > packages to epel? Having openlibm, utf8proc, openspecfun, openblas, patchelf
> > there will be very useful.
> > 
> > Finally, julia built successfully, you may want to see the results at
> > https://copr.fedoraproject.org/coprs/baurzhanm/julia-epel7/builds/
> Good to know, I've also added these targets to my Copr, waiting for LLVM to
> finish the build.
> 
> > IMHO, because of RHEL7 (Centos 7) having a long-term support, it serves as a
> > good candidate to have julia too (educational purposes, etc).
> Sure.
> 
> New package is at:
> http://nalimilan.perso.neuf.fr/transfert/julia.spec
> http://nalimilan.perso.neuf.fr/transfert/julia-0.3.0-3.fc20.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component