GNU bug report logs - #26157
[PATCH 1/2] gnu: Add fmt.

Previous Next

Full log

Message #31 received at 26157-done <at> debbugs.gnu.org (full text, mbox):

From: Marius Bakke <mbakke <at> fastmail.com>
To: Leo Famulari <leo <at> famulari.name>
Cc: 26157-done <at> debbugs.gnu.org
Subject: Re: bug#26157: [PATCH 2/2] gnu: kodi: Update to 18.0_alpha-0-4912f8f.
Date: Mon, 20 Mar 2017 23:41:51 +0100

[Message part 1 (text/plain, inline)]

Leo Famulari <leo <at> famulari.name> writes:

> On Mon, Mar 20, 2017 at 05:39:05PM +0100, Marius Bakke wrote:
>> Leo Famulari <leo <at> famulari.name> writes:
>> >> +         (add-after 'bootstrap-bundled-software 'patch-stuff
>> >>             (lambda _
>> >> +             ;; Prevent the build scripts from calling autoreconf in
>> >> +             ;; the build stage.  Otherwise, it would undo the careful
>> >> +             ;; bootstrapping that we've worked so hard for.
>> >> +             (substitute* "cmake/modules/FindLibDvd.cmake"
>> >> +               ;; Let's also use this opportunity to make the freshly
>> >> +               ;; unpackaged source directory writable.
>> >> +               (("autoreconf -vif") "chmod -R u+w ."))
>> >
>> > Clever, but perhaps too clever :) I think we should use the Scheme
>> > (chmod) procedure to do this separately, if there is a point where the
>> > files are unpacked between build phases.
>> 
>> Unfortunately, this is the only good opportunity to do this. They are
>> "downloaded", unpacked and bootstrapped from that cmake file, which is
>> invoked in the 'build' phase, and my attempts to wrangle this in a
>> separate phase were unsuccessful (hence the top-level bootstrapped
>> variables). I will make it clearer why we have to do this.
>
> I thought it might be something like this, but I figured I'd suggest
> "the right way" anyways :)

Pushed with a couple of other minor improvements and removed inputs :)

[signature.asc (application/pgp-signature, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.