From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sergei Trofimovich Subject: Re: ld-wrapper is broken in core-updates due to guile-2.2 (host running guile-2.0) Date: Sat, 15 Apr 2017 14:43:44 +0100 Message-ID: <20170415144344.0fd40584@sf> References: <20170414215947.56e464b3@sf> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; boundary="Sig_/8mklT+/Sv/jtSN5hIrjyX8E"; protocol="application/pgp-signature" Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:58551) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1czNzd-0008Be-5h for guix-devel@gnu.org; Sat, 15 Apr 2017 09:44:02 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1czNza-0003Fy-2e for guix-devel@gnu.org; Sat, 15 Apr 2017 09:44:01 -0400 In-Reply-To: <20170414215947.56e464b3@sf> 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: guix-devel@gnu.org --Sig_/8mklT+/Sv/jtSN5hIrjyX8E Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable On Fri, 14 Apr 2017 21:59:47 +0100 Sergei Trofimovich wrote: > Efraim noticed core-updates breakage on package libgpg-error. >=20 > TL;DR: >=20 > ld-wrapper package embeds guile-2.2 interpreter > to interpret ld.go bytecode built by guile-2.0. > I have no idea how to fix it but still decided to > share my findings so far. Confirmed guile-2.2 is triggering failure: reverting commit 34d624cef51f0e4eb4e888622b60fc2befe31fa7 ("gnu: Build derivations with Guile 2.2 by default.") fixes libgpg-error bu= ild. Another less obvious problem caused by guile-2.2 is triggered in bash-stati= c: guix-core-updates $ ./pre-inst-env guix build bash-static The following derivation will be built: /gnu/store/hr3g5avmb9fmgr047990r9mc6gmnq674-bash-static-4.4.12.drv @ build-started /gnu/store/hr3g5avmb9fmgr047990r9mc6gmnq674-bash-static-4= .4.12.drv - x86_64-linux /var/log/guix/drvs/hr//3g5avmb9fmgr047990r9mc6gmnq= 674-bash-static-4.4.12.drv.bz2 ice-9/psyntax.scm:1534:32: In procedure expand-macro: ice-9/psyntax.scm:1534:32: Syntax error: /gnu/store/bc7pyl426zaa2489qp97bqaycv2ir2v1-bash-static-4.4.12-guile-buil= der:1:2300: source expression failed to match any pattern in form (%modify-= phases phases* (delete (quote move-development-files))) builder for `/gnu/store/hr3g5avmb9fmgr047990r9mc6gmnq674-bash-static-4.4.= 12.drv' failed with exit code 1 @ build-failed /gnu/store/hr3g5avmb9fmgr047990r9mc6gmnq674-bash-static-4.= 4.12.drv - 1 builder for `/gnu/store/hr3g5avmb9fmgr047990r9mc6gmnq674-bash-= static-4.4.12.drv' failed with exit code 1 guix build: error: build failed: build of `/gnu/store/hr3g5avmb9fmgr04799= 0r9mc6gmnq674-bash-static-4.4.12.drv' failed It looks like guile-2.2 has different syntax checking rules which is triggered by the following modify-phases call site: [ http://git.savannah.gnu.org/cgit/guix.git/tree/gnu/packages/bash.scm#n204= ] ... ((#:phases phases) `(modify-phases ,phases ;; No loadable modules. (delete 'move-development-files)))))))) [ http://git.savannah.gnu.org/cgit/guix.git/tree/guix/build/utils.scm#n550 ] ... (define-syntax-rule (modify-phases phases mod-spec ...) "Modify PHASES sequentially as per each MOD-SPEC, which may have one of= the following forms: (delete ) (replace ) (add-before ) (add-after ) Where every <*-phase-name> is an expression evaluating to a symbol, and an expression evaluating to a procedure." (let* ((phases* phases) (phases* (%modify-phases phases* mod-spec)) ...) phases*)) (define-syntax %modify-phases (syntax-rules (delete replace add-before add-after) ((_ phases (delete old-phase-name)) (alist-delete old-phase-name phases)) ((_ phases (replace old-phase-name new-phase)) (alist-replace old-phase-name new-phase phases)) ((_ phases (add-before old-phase-name new-phase-name new-phase)) (alist-cons-before old-phase-name new-phase-name new-phase phases)) ((_ phases (add-after old-phase-name new-phase-name new-phase)) (alist-cons-after old-phase-name new-phase-name new-phase phases)))) CCed Andy in case it's an obvious bug in guix or guile syntax rewriter. Reverting commit 34d624cef51f0e4eb4e888622b60fc2befe31fa7 ("gnu: Build derivations with Guile 2.2 by default.") fixes bash-static as = well. --=20 Sergei --Sig_/8mklT+/Sv/jtSN5hIrjyX8E Content-Type: application/pgp-signature Content-Description: Цифровая подпись OpenPGP -----BEGIN PGP SIGNATURE----- iF0EARECAB0WIQSZKa0VG5avZRlY01hxoe52YR/zqgUCWPIjkAAKCRBxoe52YR/z qmH6AJ9uZIbrhb2XlXf4PbFZtC8Ux6ivQgCfaMge4j5uxoWBvFFvopek1oJpPX8= =xjJo -----END PGP SIGNATURE----- --Sig_/8mklT+/Sv/jtSN5hIrjyX8E--