From mboxrd@z Thu Jan 1 00:00:00 1970 From: Martin Becze Subject: Re: [PATCH] WIP patches for the rust importer Date: Mon, 02 Dec 2019 15:10:06 -0800 Message-ID: <785501a89e81d30db021047529e7262e@riseup.net> References: <20191126120408.GL1124@E5400> <1E01C051-2EE5-4050-B826-C60E858AAC2B@flashner.co.il> <0b2db52d687fc2acf34dc1e00618dae7.squirrel@sm.riseup.net> <20191128122255.GT1124@E5400> <8981451ac5d914dd5f53fa928741b846@riseup.net> <20191201085941.GB14869@E5400> <322B85FF-6EA4-492E-B96A-1F326ADD136E@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:470:142:3::10]:48615) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1ibuqb-0003Pt-H1 for guix-devel@gnu.org; Mon, 02 Dec 2019 18:11:19 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ibuqZ-0004l7-IE for guix-devel@gnu.org; Mon, 02 Dec 2019 18:11:17 -0500 Received: from mx1.riseup.net ([198.252.153.129]:47716) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1ibuqX-0004jS-If for guix-devel@gnu.org; Mon, 02 Dec 2019 18:11:14 -0500 In-Reply-To: <322B85FF-6EA4-492E-B96A-1F326ADD136E@gmail.com> List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: "Guix-devel" To: Ivan Petkov Cc: guix-devel@gnu.org, 38408@debbugs.gnu.org On 2019-12-02 04:01, Ivan Petkov wrote: > Hi Martin! >=20 >> On Dec 1, 2019, at 7:17 PM, Martin Becze wrote: >>=20 >> oh to be more clear. Even with "#:skip-build? #t" on all the rust >> libs, >> things fail if I omit one optional dependency from somewhere. I'm >> just >> testing on hello-cli but hello-cli inculdes clap which has alot of >> optional deps. In ./guix/build-systems/cargo.scm on line 186 it says >>=20 >>=20 >> "Cargo requires all transitive crate dependencies' sources to be >> available >> in its index, even if they are optional (this is so it can generate >> deterministic Cargo.lock files regardless of the target platform or >> enabled >> features). Thus we need all transitive crate dependencies for any >> cargo >> dev-dependencies, but this is only needed when building/testing a >> crate >> directly >> (i.e. we will never need transitive dev-dependencies for any >> dependency >> crates)." >>=20 >> I haven't really dug into the cargo build-system yet but my guess is >> that the problem lies there. Allllsooo i totally could have missed >> something simple.. idk >=20 > Yes, this is the primary limitation of packaging rust crates into guix > without manually needing to edit the Cargo.toml definition. >=20 > My opinion is that the easiest (to maintain) method for incorporating > rust packages into guix is to bite the bullet and perform a (source) > import of all transitive dependencies that a package might need. > Manually keeping up with tweaking the Cargo.toml file for every single > potential package, across every single published version will make > you go mad :) >=20 > Then the challenge would be how to automate the incremental importing > from crates.io [1] (for example, some crates require a specific x.y.z > version of > a dependency, some are willing to work with x.y.*, some have ranges, > etc.) > and making sure that all packaged crates point to the appropriate > dependencies > as they get updated in guix. >=20 > =E2=80=94Ivan >=20 > Links: > ------ > [1] http://crates.io Hi Ivan, > My opinion is that the easiest (to maintain) method for incorporating > rust packages into guix is to bite the bullet and perform a (source) > import of all transitive dependencies that a package might need. When you say source import of the transitive dependencies, do you mean that all the rust libs should just be source only or do you mean the top level package should have to declare all the transitive dependencies? If former I agree but if it is the latter then I would consider rust packaging to be broken. > Then the challenge would be how to automate the incremental importing > from crates.io [1] (for example, some crates require a specific x.y.z > version of > a dependency, some are willing to work with x.y.*, some have ranges, > etc.) Yes that is what this patch (https://issues.guix.gnu.org/issue/38408) does. It uses semantic versioning to select the correct package form crate.io or if available from the alread packaged rust packages. -Martin