bug#26009: libpng-apng

[ng0]

Kei Kebreau transcribed 1.3K bytes:
> address@hidden (Ludovic Courtès) writes:
> 
> > Good morning!
> >
> 
> Howdy! :-)
> 
> > Kei Kebreau <address@hidden> skribis:
> >
> >> From 76e4bec9f8313adc6a78d3ae571ccea489c97848 Mon Sep 17 00:00:00 2001
> >> From: ng0 <address@hidden>
> >> Date: Tue, 10 Jan 2017 20:49:30 +0000
> >> Subject: [PATCH] gnu: Add libpng-apng.
> >> To: address@hidden
> >>
> >> * gnu/packages/image.scm (libpng-apng): New variable.
> >>
> >> Signed-off-by: Kei Kebreau <address@hidden>
> >
> > [...]
> >
> >> +         (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 :)

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