From mboxrd@z Thu Jan 1 00:00:00 1970 From: Roel Janssen Subject: Re: [PATCH] Add libotf. Date: Fri, 19 Feb 2016 12:36:16 +0100 Message-ID: <87egc9exu7.fsf@gnu.org> References: <87fuwpf23g.fsf@gnu.org> <87y4ah6li8.fsf@member.fsf.org> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="=-=-=" Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:54663) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aWjMN-0001iW-Bn for guix-devel@gnu.org; Fri, 19 Feb 2016 06:36:32 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aWjMI-0006Il-A3 for guix-devel@gnu.org; Fri, 19 Feb 2016 06:36:31 -0500 In-reply-to: <87y4ah6li8.fsf@member.fsf.org> List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-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: =?utf-8?B?5a6L5paH5q2m?= Cc: guix-devel --=-=-= Content-Type: text/x-patch Content-Disposition: attachment; filename=0001-gnu-Add-libotf-v2.patch >From 96b52535da247b6c461612e3789cd22d8c6dd66a Mon Sep 17 00:00:00 2001 From: Roel Janssen Date: Fri, 19 Feb 2016 12:32:15 +0100 Subject: [PATCH] gnu: Add libotf. * gnu/packages/fontutils.scm (libotf): New variable. --- gnu/packages/fontutils.scm | 22 ++++++++++++++++++++++ 1 file changed, 22 insertions(+) diff --git a/gnu/packages/fontutils.scm b/gnu/packages/fontutils.scm index 7b5b330..cf46dbf 100644 --- a/gnu/packages/fontutils.scm +++ b/gnu/packages/fontutils.scm @@ -259,6 +259,28 @@ resolution.") (license license:gpl2+) (home-page "http://potrace.sourceforge.net/"))) +(define-public libotf + (package + (name "libotf") + (version "0.9.13") + (source (origin + (method url-fetch) + (uri (string-append + "http://download.savannah.gnu.org/releases/m17n/libotf-" + version ".tar.gz")) + (sha256 + (base32 "0239zvfan56w7vrppriwy77fzb10ag9llaz15nsraps2a2x6di3v")))) + (build-system gnu-build-system) + (propagated-inputs + `(("freetype" ,freetype))) + (home-page "http://www.nongnu.org/m17n/") + (synopsis "Library for handling OpenType Font") + (description "This library can read Open Type Layout Tables from an OTF +file. Currently these tables are supported; head, name, cmap, GDEF, GSUB, and +GPOS. It can convert a Unicode character sequence to a glyph code sequence by +using the above tables.") + (license license:lgpl2.0+))) + (define-public libspiro (package (name "libspiro") -- 2.5.0 --=-=-= Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Thank you for your quick reply. 宋文武 writes: > Roel Janssen writes: > >> From 64cfdae64cfe9af4ff162935edd4e208a0f5afc1 Mon Sep 17 00:00:00 2001 >> From: Roel Janssen >> Date: Fri, 19 Feb 2016 11:00:06 +0100 >> Subject: [PATCH] gnu: Add libotf. >> >> * gnu/packages/fontutils.scm (libotf): New variable. >> --- >> gnu/packages/fontutils.scm | 21 +++++++++++++++++++++ >> 1 file changed, 21 insertions(+) >> >> diff --git a/gnu/packages/fontutils.scm b/gnu/packages/fontutils.scm >> index 7b5b330..5cd7c56 100644 >> --- a/gnu/packages/fontutils.scm >> +++ b/gnu/packages/fontutils.scm >> @@ -259,6 +259,27 @@ resolution.") >> (license license:gpl2+) >> (home-page "http://potrace.sourceforge.net/"))) >> >> +(define-public libotf >> + (package >> + (name "libotf") >> + (version "0.9.13") >> + (source (origin >> + (method url-fetch) >> + (uri (string-append "http://download.savannah.gnu.org/releases/m17n/libotf-" > this line is too long, shouldn’t more than 80 chars. >> + version ".tar.gz")) >> + (sha256 >> + (base32 "0239zvfan56w7vrppriwy77fzb10ag9llaz15nsraps2a2x6di3v")))) >> + (build-system gnu-build-system) >> + (native-inputs >> + `(("freetype" ,freetype))) > No need to be ‘native-inputs’. And does ‘freetype’ is in the ‘libotf.pc’ > file? If so, this should be in ‘propagate-inputs’. > > > for the details. Thanks for pointing this out. I thought 'native-inputs' was for dependencies at compile time.. I changed it to propagated-inputs. >> + (home-page "http://www.nongnu.org/m17n/") >> + (synopsis "A Library for handling OpenType Font") >> + (description "The library can read Open Type Layout Tables from OTF file. >> +Currently these tables are supported; head, name, cmap, GDEF, GSUB, and GPOS. >> +It can convert a Unicode character sequence to a glyph code sequence by using >> +the above tables.") > The ‘synopsis’ shouldn’t start with “A” or “An”, expect for GNU > packages. And I think “This library” is better than “The library” > for ‘description’. Fixed. >> + (license license:lgpl2.0+))) >> + >> (define-public libspiro >> (package >> (name "libspiro") --=-=-=--