--- Begin Message ---
|
Subject: |
libpng-apng |
|
Date: |
Tue, 7 Mar 2017 12:19:31 +0000 |
This adds `libpng-apng'. I had this in the dependency graph of palemoon,
a webbrowser I stopped to work on because of license/trademark/legal
insecurities.
libpng-apng can be used to unbundle it from icecat. Despite the comment
in gnu/packages/gnuzilla.scm: This is a real, maintained software which
bases on top of libpng and tracks its releases.
When libpng is updated, this package needs to be updated aswell as soon
as the updated release of the patch is out.
I have appended the last version I packaged as a git format-patch file.
In the description the 2 occurences of APGN (Animated Portable Network Graphics)
could be written with the @defn{} of texinfo I think,
and the synopsis could be changed to just "Animated Portable Network
Graphics library". I think back then I just copied upstrema in text.
Should anyone be interested to finish the work on the webbrowser, the
repository with the branch is available at request.
0001-gnu-Add-libpng-apng.patch
Description: Text document
--- End Message ---
--- Begin Message ---
|
Subject: |
Re: bug#26009: libpng-apng |
|
Date: |
Thu, 23 Mar 2017 11:36:05 +0000 |
Kei Kebreau transcribed 3.1K bytes:
> ng0 <address@hidden> writes:
>
> > 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 :)
> >
>
> 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!
Thanks. closed
--- End Message ---