After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 780324 - Cargo is required by libgepub
Cargo is required by libgepub
Status: RESOLVED FIXED
Product: jhbuild
Classification: Infrastructure
Component: module sets
unspecified
Other Linux
: Normal blocker
: ---
Assigned To: Jhbuild maintainers
Jhbuild QA
Depends on:
Blocks:
 
 
Reported: 2017-03-20 19:42 UTC by Rafael Fontenelle
Modified: 2017-03-21 12:35 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Build libgepub from git.gnome.org and from pre-rust branch (942 bytes, patch)
2017-03-20 22:17 UTC, Michael Catanzaro
committed Details | Review

Description Rafael Fontenelle 2017-03-20 19:42:19 UTC
libepub depends on cargo/rust since: https://git.gnome.org/browse/libgepub/commit/?id=2461ac86dc90f29fb787655747a1f8908983567d

AFAIK, jhbuild doesn't list them in its modulesets.

Error output:

checking for cargo... no
configure: error: cargo is required.  Please install the Rust toolchain from https://www.rust-lang.org/
Comment 1 Michael Catanzaro 2017-03-20 21:50:55 UTC
We have decided not to depend on cargo for now, so libgepub will have to be branched.
Comment 2 Michael Catanzaro 2017-03-20 21:52:15 UTC
Hey Daniel, is it possible to find a way to make Rust happen using traditional pkg-config for dependency management? cargo is not an acceptable dependency for GNOME. You don't have any branches currently that I can fall back to so I'm just going to create one, if you don't mind....
Comment 3 Michael Catanzaro 2017-03-20 22:17:11 UTC
The following fix has been pushed:
0cf376a Build libgepub from git.gnome.org and from pre-rust branch
Comment 4 Michael Catanzaro 2017-03-20 22:17:15 UTC
Created attachment 348364 [details] [review]
Build libgepub from git.gnome.org and from pre-rust branch
Comment 5 Javier Jardón (IRC: jjardon) 2017-03-20 22:20:37 UTC
Michael; when it was decided cargo is not an acceptable dependency for GNOME? Can you share a link for my own reference?
Comment 6 Michael Catanzaro 2017-03-21 00:35:23 UTC
See the discussion following https://mail.gnome.org/archives/desktop-devel-list/2017-January/msg00002.html

In particular:

https://mail.gnome.org/archives/desktop-devel-list/2017-January/msg00011.html

I think we discussed this in #release-team recently and agreed to ask the folks attending the Rust hackfest to try to find a path forward.
Comment 7 Michael Catanzaro 2017-03-21 01:05:53 UTC
Maybe I should qualify that: "cargo is not an acceptable dependency for GNOME *at this time*." Further discussion is required on how to make this work for our distributors. We don't want to get into a situation where our Rust software does not get shipped.
Comment 8 Daniel Garcia 2017-03-21 07:18:51 UTC
I've updated the pre-rust branch to the 0.4 tag. It was in a commit before, the code doesn't change, only the NEWS file, but it the last commit before the introduction of Rust and there is a tag, so I think it's better to be there.
Comment 9 Michael Catanzaro 2017-03-21 12:35:57 UTC
OK, thanks. We should also revert https://git.gnome.org/browse/releng/commit/?id=afb8d3af4b82b87ce8abeb99beba18dbbb4b3088 once we've figured this out.