From mboxrd@z Thu Jan  1 00:00:00 1970
From: Mark H Weaver <mhw@netris.org>
Subject: Re: [PATCH] utils: Allow wrap-program to be called multiple times.
Date: Sun, 14 Sep 2014 11:58:05 -0400
Message-ID: <87ppeyw5wy.fsf@yeeloong.lan>
References: <871trk2yis.fsf@member.fsf.org> <8738byti9p.fsf@gnu.org>
	<87sijw122v.fsf@gmail.com> <87lhpmyhh2.fsf@netris.org>
	<877g16b7lv.fsf@gnu.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
Return-path: <guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org>
Received: from eggs.gnu.org ([2001:4830:134:3::10]:55449)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <mhw@netris.org>) id 1XTCDS-0004yA-9y
	for guix-devel@gnu.org; Sun, 14 Sep 2014 11:59:59 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <mhw@netris.org>) id 1XTCDN-0006MK-0S
	for guix-devel@gnu.org; Sun, 14 Sep 2014 11:59:54 -0400
In-Reply-To: <877g16b7lv.fsf@gnu.org> ("Ludovic
	\=\?utf-8\?Q\?Court\=C3\=A8s\=22'\?\=
	\=\?utf-8\?Q\?s\?\= message of "Sun, 14 Sep 2014 16:27:08 +0200")
List-Id: "Development of GNU Guix and the GNU System distribution."
	<guix-devel.gnu.org>
List-Unsubscribe: <https://lists.gnu.org/mailman/options/guix-devel>,
	<mailto:guix-devel-request@gnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/archive/html/guix-devel>
List-Post: <mailto:guix-devel@gnu.org>
List-Help: <mailto:guix-devel-request@gnu.org?subject=help>
List-Subscribe: <https://lists.gnu.org/mailman/listinfo/guix-devel>,
	<mailto:guix-devel-request@gnu.org?subject=subscribe>
Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org
Sender: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org
To: Ludovic =?utf-8?Q?Court=C3=A8s?= <ludo@gnu.org>
Cc: guix-devel@gnu.org

ludo@gnu.org (Ludovic Court=C3=A8s) writes:

> mhw@netris.org skribis:
>
>> When compiling core-updates, I now see the following warning:
>>
>> LC_ALL=3DC							\
>> ./pre-inst-env					\
>> /home/mhw/.guix-profile/bin/guild compile -L "." -L "."	\
>>   -Wformat -Wunbound-variable -Warity-mismatch			\
>>   --target=3D"i686-pc-linux-gnu"						\
>>   -o "guix/build/utils.go" "guix/build/utils.scm"
>> guix/build/utils.scm:718:4: warning: "~a/.~a-wrap-~2'0d": unsupported fo=
rmat option ~2, use (ice-9 format) instead
>> wrote `guix/build/utils.go'
>
> Oops, this needs to be fixed, indeed.

Okay, I pushed a fix to core-updates.

>> Have you verified that this call to 'format' works as expected in all
>> the relevant cases?
>
> It could be that it works in practice, because (ice-9 format) does this
> crazy thing:
>
>   (module-set! the-root-module 'format format)
>
> (-Wformat is purposefully stricter, because this crazy thing should
> vanish eventually.)

Indeed, maybe we should consider fixing that for 2.2.

    Thanks,
      Mark