unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Marius Bakke <mbakke@fastmail.com>
To: Brendan Tildesley <brendan.tildesley@openmailbox.org>,
	guix-devel@gnu.org
Subject: Re: [PATCH] gnu: Add libicns
Date: Sat, 03 Sep 2016 21:04:19 +0100	[thread overview]
Message-ID: <87vaycpx6k.fsf@ike.i-did-not-set--mail-host-address--so-tickle-me> (raw)
In-Reply-To: <2f6589d3-5cba-0e2b-5576-d2d6f8ee3d2e@openmailbox.org>

[-- Attachment #1: Type: text/plain, Size: 1439 bytes --]

Brendan Tildesley <brendan.tildesley@openmailbox.org> writes:

> I just notice the configure script wasn't looking for any newer version.

[...]

> checking libpng/png.h usability... no
> checking libpng/png.h presence... no
> checking for libpng/png.h... no
> checking libpng10/png.h usability... no
> checking libpng10/png.h presence... no
> checking for libpng10/png.h... no
> checking libpng12/png.h usability... yes
> checking libpng12/png.h presence... yes
> checking for libpng12/png.h... yes

You cut off the important lines! Here is how it looks with the latest
libpng in inputs:

checking for png_create_read_struct in -lpng... yes
checking png.h usability... yes                      <---
checking png.h presence... yes                       <---
checking for png.h... yes                            <---
checking libpng/png.h usability... no
checking libpng/png.h presence... no
checking for libpng/png.h... no
checking libpng10/png.h usability... no
checking libpng10/png.h presence... no
checking for libpng10/png.h... no
checking libpng12/png.h usability... no
checking libpng12/png.h presence... no
checking for libpng12/png.h... no

>>
>> The rest of the patch looks good to me.
>>
>> Thanks again!
>> Marius
>
> Thanks. If I posted a fix, I'd just end up making some new mistakes, so
> I'll just leave it to be added!

I did the mentioned adjustments and attached a new patch.

Hope I did not scare you off! :)

~marius


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-Add-libicns.patch --]
[-- Type: text/x-patch, Size: 2287 bytes --]

From 0efb2e7cb5c3344b49e30af57efcdd5b1136ac7f Mon Sep 17 00:00:00 2001
From: Brendan Tildesley <brendan.tildesley@openmailbox.org>
Date: Sat, 3 Sep 2016 05:53:56 +1000
Subject: [PATCH] gnu: Add libicns.

* gnu/packages/image.scm (libicns): New variable.
---
 gnu/packages/image.scm | 38 ++++++++++++++++++++++++++++++++++++++
 1 file changed, 38 insertions(+)

diff --git a/gnu/packages/image.scm b/gnu/packages/image.scm
index 4fdc4ae..c1b79f3 100644
--- a/gnu/packages/image.scm
+++ b/gnu/packages/image.scm
@@ -147,6 +147,44 @@ maximum quality factor.")
    (license license:gpl2+)
    (home-page "http://www.kokkonen.net/tjko/projects.html#jpegoptim")))
 
+(define-public libicns
+  (package
+    (name "libicns")
+    (version "0.8.1")
+    (source (origin
+             (method url-fetch)
+             (uri (string-append
+                   "mirror://sourceforge/icns/libicns-" version ".tar.gz"))
+             (sha256 (base32
+                      "1hjm8lwap7bjyyxsyi94fh5817xzqhk4kb5y0b7mb6675xw10prk"))))
+    (build-system gnu-build-system)
+    (inputs
+     `(("libpng" ,libpng)
+       ("jasper" ,jasper)))
+    (native-inputs
+     `(("autoconf" ,autoconf)
+       ("automake" ,automake)
+       ("libtool" ,libtool)))
+    (arguments
+     `(#:tests? #t ; No tests.
+       #:phases
+       (modify-phases %standard-phases
+         (add-before 'configure 'autoreconf
+           (lambda _
+             (zero? (system* "autoreconf" "-vfi")))))))
+    (home-page "http://icns.sourceforge.net/")
+    (synopsis "Library for handling Mac OS icns resource files")
+    (description
+     "Libicns is a library for the manipulation of Mac OS IconFamily resource
+type files (ICNS).  @command{icns2png} and @command{png2icns} are provided to
+convert between PNG and ICNS. @command{icns2png} will extract image files from
+ICNS files under names like \"Foo_48x48x32.png\" useful for installing for use
+with .desktop files.  Additionally, @command{icontainer2png} is provided for
+extracting icontainer icon files.")
+    (license (list license:lgpl2.1+     ; libicns
+                   license:lgpl2.0+     ; src/apidocs.*
+                   license:gpl2+))))    ; icns2png, png2icns, icontainer2png
+
 (define-public libtiff
   (package
    (name "libtiff")
-- 
2.9.3


  reply	other threads:[~2016-09-03 20:04 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-02 20:31 [PATCH] gnu: Add libicns Brendan Tildesley
2016-09-03 10:27 ` Marius Bakke
2016-09-03 12:38   ` Marius Bakke
2016-09-03 13:16     ` Brendan Tildesley
2016-09-03 18:59       ` Marius Bakke
2016-09-03 19:38         ` Brendan Tildesley
2016-09-03 20:04           ` Marius Bakke [this message]
2016-09-04 11:05             ` David Craven
  -- strict thread matches above, loose matches on Subject: below --
2016-09-02 20:33 Brendan Tildesley

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87vaycpx6k.fsf@ike.i-did-not-set--mail-host-address--so-tickle-me \
    --to=mbakke@fastmail.com \
    --cc=brendan.tildesley@openmailbox.org \
    --cc=guix-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).