Bug 2310720 - Review Request: juliaup - Julia installer and version multiplexer
Summary: Review Request: juliaup - Julia installer and version multiplexer
Keywords:
Status: NEW
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL: https://github.com/julialang/juliaup
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2024-09-08 15:34 UTC by Fabio Valentini
Modified: 2024-09-19 10:48 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed:
Type: ---
Embargoed:


Attachments (Terms of Use)
The .spec file difference from Copr build 7997618 to 8034125 (329 bytes, patch)
2024-09-19 10:48 UTC, Fedora Review Service
no flags Details | Diff

Description Fabio Valentini 2024-09-08 15:34:14 UTC
Spec URL: https://decathorpe.fedorapeople.org/juliaup.spec
SRPM URL: https://decathorpe.fedorapeople.org/juliaup-1.17.4-1.fc40.src.rpm

Description:
Julia installer and version multiplexer.

Fedora Account System Username: decathorpe

Comment 1 Fabio Valentini 2024-09-08 15:34:48 UTC
koji scratch build for rawhide:
https://koji.fedoraproject.org/koji/taskinfo?taskID=123105692

Comment 2 Fedora Review Service 2024-09-08 15:35:59 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/7997618
(failed)

Build log:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2310720-juliaup/fedora-rawhide-x86_64/07997618-juliaup/builder-live.log.gz

Please make sure the package builds successfully at least for Fedora Rawhide.

- If the build failed for unrelated reasons (e.g. temporary network
  unavailability), please ignore it.
- If the build failed because of missing BuildRequires, please make sure they
  are listed in the "Depends On" field


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 3 Ben Beasley 2024-09-18 15:50:46 UTC
The binary /usr/bin/julia conflicts with the julia package. Per [1], *something* needs to be done about this – it can’t just be an implicit conflict.

I don’t think I understand this well enough to make a recommendation. I see that upstream recommends uninstalling all other Julias before using juliaup[2], and strongly recommends installing via "curl | bash" rather than using package managers[3] due to unspecified “drawbacks (that we hope to lift in the future).” I’m not sure what to think about all that.

[1] https://docs.fedoraproject.org/en-US/packaging-guidelines/Conflicts/#_binary_name_conflicts
[2] https://github.com/julialang/juliaup?tab=readme-ov-file#installation
[3] https://github.com/julialang/juliaup?tab=readme-ov-file#software-repositories

Comment 4 Fabio Valentini 2024-09-18 16:04:27 UTC
You are correct, making the Conflicts explicit is probably a good idea.

There's been a few threads about packaged julia in the past.

This bug has some context for why packaging it (without bundling the world) is basically impossible:
https://bugzilla.redhat.com/show_bug.cgi?id=2274270

There was also a discussion on the "devel" list:
https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org/thread/J423CXSMKN525MFGQHHNTNPTSJOKOI4E/

Comment 5 Fabio Valentini 2024-09-19 10:33:14 UTC
I've added explicit "Conflicts: julia".

[fedora-review-service-build]

Comment 6 Fedora Review Service 2024-09-19 10:48:51 UTC
Created attachment 2047557 [details]
The .spec file difference from Copr build 7997618 to 8034125

Comment 7 Fedora Review Service 2024-09-19 10:48:54 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/8034125
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2310720-juliaup/fedora-rawhide-x86_64/08034125-juliaup/fedora-review/review.txt

Found issues:

- No gcc, gcc-c++ or clang found in BuildRequires
  Read more: https://docs.fedoraproject.org/en-US/packaging-guidelines/C_and_C++/

Please know that there can be false-positives.

---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.


Note You need to log in before you can comment on or make changes to this bug.