From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark H Weaver Subject: Re: 01/01: gnu: totem: Enable parallel build. Date: Sun, 12 Nov 2017 23:14:28 -0500 Message-ID: <87mv3qyfvf.fsf@netris.org> References: <20171109014628.6934.66587@vcs0.savannah.gnu.org> <20171109014630.1CC31201B8@vcs0.savannah.gnu.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:41876) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1eE69D-0002dr-Lp for guix-devel@gnu.org; Sun, 12 Nov 2017 23:15:03 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1eE698-0005Gy-SS for guix-devel@gnu.org; Sun, 12 Nov 2017 23:14:59 -0500 Received: from world.peace.net ([50.252.239.5]:33352) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1eE698-0005EZ-Ni for guix-devel@gnu.org; Sun, 12 Nov 2017 23:14:54 -0500 In-Reply-To: <20171109014630.1CC31201B8@vcs0.savannah.gnu.org> (Kei Kebreau's message of "Wed, 8 Nov 2017 20:46:29 -0500 (EST)") 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: Kei Kebreau Cc: guix-devel@gnu.org Hi, kkebreau@posteo.net (Kei Kebreau) writes: > kkebreau pushed a commit to branch master > in repository guix. > > commit e5f748a6c0c774265f52c7246ee7b80bec6d3522 > Author: Kei Kebreau > Date: Wed Nov 8 20:42:20 2017 -0500 > > gnu: totem: Enable parallel build. >=20=20=20=20=20 > * gnu/packages/gnome.scm (totem)[arguments]: Set #:parallel-build? to= #t > implicitly. Your commit reverted the preceding commit made to our 'totem' package, namely: > commit a97bfa46cdb782f705204a50a10bca9fa98c56b2 > Author: Adam Van Ymeren > Date: Fri Oct 13 12:40:50 2017 -0400 >=20 > gnu: totem: Disable parallel build due to http://debbugs.gnu.org/28813 >=20=20=20=20=20 > Fixes . >=20=20=20=20=20 > * gnu/packages/gnome.scm (totem)[arguments]: Add #:parallel-build?. >=20=20=20=20=20 > Signed-off-by: Ludovic Court=C3=A8s Unsurprisingly, this caused bug #28813 to re-appear. Since reverting this fix, one of the totem builds failed on Hydra with the same error message described in : https://hydra.gnu.org/build/2352092 namely: --8<---------------cut here---------------start------------->8--- [124/150] Compiling Vala source ../totem-3.26.0/src/plugins/sample-vala/tot= em-sample-vala-plugin.vala. FAILED: src/plugins/sample-vala/sample-vala@sha/totem-sample-vala-plugin.c = src/plugins/sample-vala/sample-vala.h src/plugins/sample-vala/sample-vala.v= api=20 valac -C --pkg libpeas-1.0 --pkg gtk+-3.0 -d src/plugins/sample-vala/sample= -vala@sha --library=3Dsample-vala -H src/plugins/sample-vala/sample-vala.h = --vapi ../sample-vala.vapi --girdir=3D/tmp/guix-build-totem-3.26.0.drv-0/bu= ild/src --pkg=3DTotem-1.0 ../totem-3.26.0/src/plugins/sample-vala/totem-sam= ple-vala-plugin.vala error: Package `Totem-1.0' not found in specified Vala API directories or G= Object-Introspection GIR directories Compilation failed: 1 error(s), 0 warning(s) [125/150] Linking target src/plugins/ontop/libontop.so. [126/150] Linking target src/totem. [127/150] Generating Totem-1.0.gir with a custom command. ninja: build stopped: subcommand failed. phase `build' failed after 15.8 seconds builder for `/gnu/store/l6pv9v3kvvbj4dfnmqj2irmqzwa9hw4c-totem-3.26.0.drv' = failed with exit code 1 @ build-failed /gnu/store/l6pv9v3kvvbj4dfnmqj2irmqzwa9hw4c-totem-3.26.0.drv= - 1 builder for `/gnu/store/l6pv9v3kvvbj4dfnmqj2irmqzwa9hw4c-totem-3.26.0.= drv' failed with exit code 1 --8<---------------cut here---------------end--------------->8--- I'd like to understand what led you to believe that removing this fix was appropriate. In general, we don't add #:parallel-build? #f without a good reason. The code even included a comment referencing bug #28813. I assume that you saw the comment, since you removed it. So, what made you think that this fix was no longer needed? Can you help me understand how this happened? Mark