From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:35714) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gPpeH-0004D3-Sb for guix-patches@gnu.org; Thu, 22 Nov 2018 09:08:07 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gPpeD-0007bg-U2 for guix-patches@gnu.org; Thu, 22 Nov 2018 09:08:05 -0500 Received: from debbugs.gnu.org ([208.118.235.43]:36419) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1gPpeD-0007bD-Q4 for guix-patches@gnu.org; Thu, 22 Nov 2018 09:08:01 -0500 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1gPpeD-0006TW-L8 for guix-patches@gnu.org; Thu, 22 Nov 2018 09:08:01 -0500 Subject: [bug#33465] [PATCH] gnu: rust: Don't depend on 'git'. Resent-Message-ID: Received: from eggs.gnu.org ([2001:4830:134:3::10]:35295) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gPpdB-0003dx-5P for guix-patches@gnu.org; Thu, 22 Nov 2018 09:06:58 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gPpd7-0005Wv-6p for guix-patches@gnu.org; Thu, 22 Nov 2018 09:06:57 -0500 Received: from lepiller.eu ([2a00:5884:8208::1]:36822) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1gPpd6-0005SW-Ue for guix-patches@gnu.org; Thu, 22 Nov 2018 09:06:53 -0500 Received: from webmail.lepiller.eu (static-176-182-42-79.ncc.abo.bbox.fr [176.182.42.79]) by lepiller.eu (OpenSMTPD) with ESMTPSA id ace3508a (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256:NO) for ; Thu, 22 Nov 2018 14:04:30 +0000 (UTC) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Date: Thu, 22 Nov 2018 15:06:48 +0100 From: Julien Lepiller In-Reply-To: <20181122135830.18927-1-mbakke@fastmail.com> References: <20181122135830.18927-1-mbakke@fastmail.com> Message-ID: <1c2849407fb4914810f3c367fc16b6cf@lepiller.eu> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org Sender: "Guix-patches" To: 33465@debbugs.gnu.org Le 2018-11-22 14:58, Marius Bakke a écrit : > * gnu/packages/rust.scm (rust-1.19)[inputs]: Remove GIT. > (rust-1.20)[arguments]: Disable Cargo tests that require git. > (rust-1.26)[arguments]: Likewise. > --- > > Notes: > Guix, > > The Rust toolchain is very expensive to build and needs less > volatility. > > So far I have only built up to Rust 1.23 with this patch. I > suggest > applying this on 'core-updates', and giving Rust in 'master' a > git-2.19.1 input to the previous substitutes are valid again. > > WDYT? Hi, I don't really know how our rust packages are built, but I wonder if they have more than the bare minimum optional features for our purposes (apart from 1.24 used by icecat and the latest version, we probably don't need to build everything)? Maybe it's worth investigating if that can speed up the build of the whole chain. It seems we're two versions behind current rust too. Is anyone working on packaging them? > > gnu/packages/rust.scm | 15 +++++++++++++-- > 1 file changed, 13 insertions(+), 2 deletions(-) > > diff --git a/gnu/packages/rust.scm b/gnu/packages/rust.scm > index a56faad07..ae41a7dd3 100644 > --- a/gnu/packages/rust.scm > +++ b/gnu/packages/rust.scm > @@ -41,7 +41,6 @@ > #:use-module (gnu packages python) > #:use-module (gnu packages ssh) > #:use-module (gnu packages tls) > - #:use-module (gnu packages version-control) > #:use-module (gnu packages) > #:use-module (guix build-system cargo) > #:use-module (guix build-system gnu) > @@ -387,7 +386,6 @@ test = { path = \"../libtest\" } > ("cmake" ,cmake) > ("flex" ,flex) ; For the tests > ("gdb" ,gdb) ; For the tests > - ("git" ,git) > ("procps" ,procps) ; For the tests > ("python-2" ,python-2) > ("rustc-bootstrap" ,mrustc) > @@ -446,6 +444,13 @@ safety and thread safety guarantees.") > ;; i686-linux. > (substitute* "src/tools/cargo/tests/test.rs" > (("fn cargo_test_env") "#[ignore]\nfn > cargo_test_env")) > + > + ;; These tests pull in a dependency on "git", which > changes > + ;; too frequently take part in the Rust toolchain. > + (substitute* "src/tools/cargo/tests/new.rs" > + (("fn author_prefers_cargo") "#[ignore]\nfn > author_prefers_cargo") > + (("fn finds_author_git") "#[ignore]\nfn > finds_author_git") > + (("fn finds_local_author_git") "#[ignore]\nfn > finds_local_author_git")) > #t)) > (add-after 'patch-cargo-tests > 'ignore-glibc-2.27-incompatible-test > ;; https://github.com/rust-lang/rust/issues/47863 > @@ -678,6 +683,12 @@ jemalloc = \"" jemalloc "/lib/libjemalloc_pic.a" > "\" > ;; i686-linux. > (substitute* > "src/tools/cargo/tests/testsuite/test.rs" > (("fn cargo_test_env") "#[ignore]\nfn > cargo_test_env")) > + > + ;; Avoid dependency on "git". > + (substitute* "src/tools/cargo/tests/new.rs" > + (("fn author_prefers_cargo") "#[ignore]\nfn > author_prefers_cargo") > + (("fn finds_author_git") "#[ignore]\nfn > finds_author_git") > + (("fn finds_local_author_git") "#[ignore]\nfn > finds_local_author_git")) > #t)) > (add-after 'patch-cargo-tests > 'disable-cargo-test-for-nightly-channel > (lambda* _