From patchwork Sat Sep 19 13:36:04 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Pierre Langlois X-Patchwork-Id: 24222 Return-Path: X-Original-To: patchwork@mira.cbaines.net Delivered-To: patchwork@mira.cbaines.net Received: by mira.cbaines.net (Postfix, from userid 113) id DECE427BBE8; Sat, 19 Sep 2020 14:38:45 +0100 (BST) X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on mira.cbaines.net X-Spam-Level: X-Spam-Status: No, score=-2.8 required=5.0 tests=BAYES_00,DKIM_SIGNED, FREEMAIL_FROM,MAILING_LIST_MULTI,RCVD_IN_MSPIKE_H4,RCVD_IN_MSPIKE_WL, T_DKIM_INVALID,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.2 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mira.cbaines.net (Postfix) with ESMTPS id 2FC8227BBE7 for ; Sat, 19 Sep 2020 14:38:45 +0100 (BST) Received: from localhost ([::1]:40896 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1kJd4e-0002H5-6x for patchwork@mira.cbaines.net; Sat, 19 Sep 2020 09:38:44 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:51198) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1kJd30-0001SY-Do for guix-patches@gnu.org; Sat, 19 Sep 2020 09:37:02 -0400 Received: from debbugs.gnu.org ([209.51.188.43]:34287) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1kJd30-0005DW-2w for guix-patches@gnu.org; Sat, 19 Sep 2020 09:37:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1kJd2z-0001cf-WB for guix-patches@gnu.org; Sat, 19 Sep 2020 09:37:02 -0400 X-Loop: help-debbugs@gnu.org Subject: [bug#42888] [PATCH] gnu: rust: Fix install phase for non-x86_64 platforms. Resent-From: Pierre Langlois Original-Sender: "Debbugs-submit" Resent-CC: guix-patches@gnu.org Resent-Date: Sat, 19 Sep 2020 13:37:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 42888 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: patch To: 42888@debbugs.gnu.org Cc: Pierre Langlois Received: via spool by 42888-submit@debbugs.gnu.org id=B42888.16005225866191 (code B ref 42888); Sat, 19 Sep 2020 13:37:01 +0000 Received: (at 42888) by debbugs.gnu.org; 19 Sep 2020 13:36:26 +0000 Received: from localhost ([127.0.0.1]:45833 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1kJd2P-0001bn-VN for submit@debbugs.gnu.org; Sat, 19 Sep 2020 09:36:26 -0400 Received: from mout.gmx.net ([212.227.15.15]:53309) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1kJd2K-0001bW-Kl for 42888@debbugs.gnu.org; Sat, 19 Sep 2020 09:36:24 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gmx.net; s=badeba3b8450; t=1600522574; bh=hdAXU1sDTa9PU1xfSaIS5qUu03cSxMRPFKFTKQ2okho=; h=X-UI-Sender-Class:References:From:To:Cc:Subject:In-reply-to:Date; b=Kyjm3YCmEKt1uF14jTZv1X7CwDgcEznElyRi23IyuhrY0JGf9FuMq62RYyjmzxjme n3GCkjuBDnzc5kCcaMq+aXcmOEq1aWMtJPQV2cf7p6nmBKeUwFuTbex/B6Vznj/4c3 2ml9geqBVsCjSo5Nimc0kNeqNTMc9nga7iM00fkY= X-UI-Sender-Class: 01bb95c1-4bf8-414a-932a-4f6e2808ef9c Received: from labiere ([82.69.64.142]) by mail.gmx.com (mrgmx005 [212.227.17.184]) with ESMTPSA (Nemesis) id 1MZkpR-1jwNfG47Z1-00Wq3l; Sat, 19 Sep 2020 15:36:14 +0200 References: <87d03qptmv.fsf@gmx.com> <875z9ips81.fsf@gmx.com> <20200819215552.stnpvcfaj5xpqkn7@gravity> <874koy47g8.fsf@gmx.com> User-agent: mu4e 1.4.13; emacs 27.1 From: Pierre Langlois In-reply-to: <874koy47g8.fsf@gmx.com> Date: Sat, 19 Sep 2020 14:36:04 +0100 Message-ID: <87zh5l6g8r.fsf@gmx.com> MIME-Version: 1.0 X-Provags-ID: V03:K1:KgwDdQTO+5UzG04AqXuNqXPuY1BJBB/NzUwGr4JzLZjqI26/ZGV GdZJLF5OigKb8RTy8laIGet1ZHAYS3lnGVIbtEe/deL7w1DYSFebPBn7qdGhkZOxhi8Riv5 15o3qeGjAtqO8jFHTlLRcewBIU3L3eYu2SC5KsJ9CX+0rtafjVRnt0bDoKGOOWfBaFq/uED +lKItZO/MGAzgM2Tvo3wQ== X-UI-Out-Filterresults: notjunk:1;V03:K0:wId73NC8z4Q=:YH8XHrDIfXzOWLU4o/NulS 2larg4deNlGNkjAxGy7JGAm5U0nZrKOj+vNbuzRh12b0aQ0d5AFdFKP5+NGx1cnFeQaRF5OBc V1K/we8WJB5HkkV9zJTsFjaEJzaxuV31FFcKZShv7U1HKyGKpNCoNKZlAIJ03x334SfsPo9L7 5sLSDMsIv8ZxRHJv14PS5q6YiqAkZLzvmZBjrhpuraKAmgNZNRa5wGKbKKmuFmNyxPWmyjbbF vXiMzrU5yABBQ1Jf6t0IPwuOPVAzxCozMTbd7avh1Ji30bnIAWqwaaN7nHlLRhUq3odWm15on zDJ9klbRk0o5moowLocZQE5lmSSrVtfiyyDguHJPyXuydjVtzjmLkJ9mwvzaLlxlvzhG64/Y/ NucO0Sq2vYhTiiuj0OgQ6RF9HevVNwH7RYfoONind801asG0m4dqo/9yS+Q4R8ZQ00/yAhxMs wZcfhbza9kSnIy4kYL7zT21YTGwa6R/TcOfXkUoH8dGeDu/GFYbPPEgLc0K4MtZ0oJ7nXI5YS Jbv+FjroKDJHdSQI+HK+iEBQJXr484Krlo0ZiHHji++D0lpdWDFGQ75TMr2s1mOGpe4Xvoymj sJkRc4cSzv9yiJurKRnN2TYttYrpAS98jTMqvYsmUUo2yvO7MWtmCyZhww5xgzHdSpVyUQSl5 0M9L0ZYJWKdvKbzKp3yIIfU88N8NlLyVS41GMF+YKE93E0WdeaKiZ/40B2Cm3Rp3+b+Lh+Pkq 6uMT7hd+cDuZm+ay5LzcFjZKPJXMvG7+pFSY3Kfy0+/7jQ/XcYYDh4wo9bx9+5+vJmBvPq7pe 76/VKVEr7XXlDWxWeZjK03twNFQZcmZMPWeHul0StfWmPfoUaqEDsN32AXhVs/OVGQHAr7BE5 Dlu7zCKkYvBF+wSyTwUq2nFDzAdN2ubu77VYjrX/go1ax+ZGTIU2hRNPW9MBUVzGrcTnXZ1WK Fx8XhVP72gIpPGYDKTLPQ07mWgSFreyLXtlPDWY3di7/OyCDXlqosLRATHYNrp8OlB6zPpkM3 1WPFUPqHQTXvoensieX5bcnKtFvDHWzij8GqX35FR7+tXfsowhLdBgPlZ45ce2Vr2Hl4aV/vP YUME0G6SrcYrZGcJhAmmJObNFF228Abl0KXssanMKuMtAMW2pFHsIceKXhWXCi6ZQHtNbbu3S /FBLFQnOLqy7JoMPhutYCI3z7SXaTzloo+XMkgXZaAi46lRFjbT2/Bi+p76dtoC6cPLeLq8i4 BKSkiG8sXf1mX6b5qyqfd4MQYBOoBEaJf6uKwOw== X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: guix-patches@gnu.org List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+patchwork=mira.cbaines.net@gnu.org Sender: "Guix-patches" X-getmail-retrieved-from-mailbox: Patches Hello Guix! Pierre Langlois writes: > Hi Jakub, > > Jakub Kądziołka writes: > (snip) >> Pierre, >> >> thanks for your patch! I was working on a similar change before, but >> when I tried it, it failed even earlier in the bootstrap chain. It >> might've been QEMU weirdness, though, a la #42448. >> >>> @@ -612,9 +613,10 @@ jemalloc = \"" jemalloc "/lib/libjemalloc_pic.a" "\" >>> (cargo-out (assoc-ref outputs "cargo"))) >>> (for-each >>> (lambda (file) (delete-manifest-file out file)) >>> - '("install.log" >>> + `("install.log" >>> "manifest-rust-docs" >>> - "manifest-rust-std-x86_64-unknown-linux-gnu" >>> + ,,(string-append "manifest-rust-std-" >>> + (nix-system->gnu-triplet-for-rust)) >>> "manifest-rustc")) >>> (for-each >>> (lambda (file) (delete-manifest-file cargo-out file)) >> >> If I understand the code correctly, this quasiquote is unnecessary, as >> the host-side code will evaluate to a string that can be inserted as-is, >> without another unquote on the build side. >> >> Fixing this would mean that the patch can go on master, since it would >> now only trigger rebuilds on architectures that are already broken. > > Ooooh yeah, that'll be much better, I've attached a patch that does just > that. I can confirm it doesn't trigger a full rebuild, nice! > I've just rebased this patch and fixed a little formatting issue. I've made sure, it doesn't trigger any rust rebuilds on x86. Is this OK to commit? Thanks, Pierre From 207071445a0a637075138fb678bdbcdcfbaa038d Mon Sep 17 00:00:00 2001 From: Pierre Langlois Date: Sun, 16 Aug 2020 12:17:18 +0100 Subject: [PATCH] gnu: rust: Fix install phase for non-x86_64 platforms. * gnu/packages/rust.scm (rust-1.20)[arguments]: Replace hardcoded x86_64 triplet with nix-system->gnu-triplet-for-rust in 'delete-install-logs phase. --- gnu/packages/rust.scm | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/gnu/packages/rust.scm b/gnu/packages/rust.scm index d5914f012b..1690ad96c5 100644 --- a/gnu/packages/rust.scm +++ b/gnu/packages/rust.scm @@ -9,6 +9,7 @@ ;;; Copyright © 2018 Danny Milosavljevic ;;; Copyright © 2019 Ivan Petkov ;;; Copyright © 2020 Jakub Kądziołka +;;; Copyright © 2020 Pierre Langlois ;;; ;;; This file is part of GNU Guix. ;;; @@ -614,7 +615,8 @@ jemalloc = \"" jemalloc "/lib/libjemalloc_pic.a" "\" (lambda (file) (delete-manifest-file out file)) '("install.log" "manifest-rust-docs" - "manifest-rust-std-x86_64-unknown-linux-gnu" + ,(string-append "manifest-rust-std-" + (nix-system->gnu-triplet-for-rust)) "manifest-rustc")) (for-each (lambda (file) (delete-manifest-file cargo-out file)) -- 2.28.0