GNU bug report logs - #26009
libpng-apng

Previous Next

Full log

View this message in rfc822 format

From: Kei Kebreau <kei <at> openmailbox.org>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 26009 <at> debbugs.gnu.org
Subject: bug#26009: libpng-apng
Date: Fri, 17 Mar 2017 11:37:59 -0400

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

ng0 <contact.ng0 <at> cryptolab.net> writes:

> Kei Kebreau transcribed 1.3K bytes:
>> ludo <at> gnu.org (Ludovic Courtès) writes:
>> 
>> > Good morning!
>> >
>> 
>> Howdy! :-)
>> 
>> > Kei Kebreau <kei <at> openmailbox.org> skribis:
>> >
>> >> From 76e4bec9f8313adc6a78d3ae571ccea489c97848 Mon Sep 17 00:00:00 2001
>> >> From: ng0 <ng0 <at> libertad.pw>
>> >> Date: Tue, 10 Jan 2017 20:49:30 +0000
>> >> Subject: [PATCH] gnu: Add libpng-apng.
>> >> To: guix-patches <at> gnu.org
>> >>
>> >> * gnu/packages/image.scm (libpng-apng): New variable.
>> >>
>> >> Signed-off-by: Kei Kebreau <kei <at> openmailbox.org>
>> >
>> > [...]
>> >
>> >> +         (add-before 'configure 'no-checks
>> >> +           (lambda _
>> >> +             (substitute* "Makefile.in"
>> >> +               (("^scripts/symbols.chk") "")
>> >> +               (("check: scripts/symbols.chk") ""))
>> >> +             (zero? (system* "libtool"))
>> >> +             #t)))))
>> >
>> > Do you really want to run ‘libtool’ here?  I think that doesn’t have any
>> > effect:
>> >
>> > $ libtool
>> > Usage: /home/ludo/.guix-profile/bin/libtool [OPTION]... [MODE-ARG]...
>> > Try 'libtool --help' for more information.
>> > libtool:   error: you must specify a MODE
>> >
>> 
>> ng0 had some adjustments to make to the patch for sending it upstream,
>> so I'm waiting for a response. Perhaps removing this libtool execution
>> is one of them.
>
> On the one hand I wanted to check on this, but on the other hand it
> looks like you have already solved most or all of the problems?
> I decided to take an almost-vacation for 3 weeks, just plan and document
> and maybe add some thoughts to patches which are open, minimal work.
> If it looks good for you and it works, please gp ahead (and add the
> Co-authored-by line).
>
> Thanks for your fixes on this :)
>

You're welcome. Thanks to Ludovic as well for pointing out the
replacement issue.

>> > If there’s nothing else blocking this patch, I think you can go ahead.
>> >
>> > Thanks,
>> > Ludo’.
>> 
>> Thank you. I'll push it when I get the okay from ng0.

Pushed to master!

[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.