Patchwork new package: iso-codes

login
register
mail settings
Submitter Stefan Fröberg
Date Feb. 17, 2013, 4:34 p.m.
Message ID <1361118896-2263-1-git-send-email-stefan.froberg@petroprogram.com>
Download mbox | patch
Permalink /patch/221076/
State Rejected
Headers show

Comments

Stefan Fröberg - Feb. 17, 2013, 4:34 p.m.
iso-codes

This package provides the ISO-639 Language code list, the ISO-4217 
Currency code list, the ISO-3166 Territory code list, and ISO-3166-2
sub-territory lists, and all their translations in gettext .po form.

Signed-off-by: Stefan Fröberg <stefan.froberg@petroprogram.com>
---
Note: This package is needed by network-manager-applet (coming soon)

 package/Config.in              |    1 +
 package/iso-codes/Config.in    |   10 ++++++++++
 package/iso-codes/iso-codes.mk |   13 +++++++++++++
 3 files changed, 24 insertions(+), 0 deletions(-)
 create mode 100644 package/iso-codes/Config.in
 create mode 100644 package/iso-codes/iso-codes.mk
Thomas Petazzoni - Feb. 17, 2013, 5:30 p.m.
Dear Stefan Fröberg,

On Sun, 17 Feb 2013 18:34:56 +0200, Stefan Fröberg wrote:

> +	select BR2_PACKAGE_GETTEXT

Are you sure it really needs gettext as a build dependency?

Also, this seems to install a big bunch of files. Maybe
network-manager-applet needs only some of the ISOxyz files, and
therefore having suboptions to select which ISOxyz set of files should
be installed would be useful?

Thanks,

Thomas
Stefan Fröberg - Feb. 17, 2013, 5:55 p.m.
17.2.2013 19:30, Thomas Petazzoni kirjoitti:
> Dear Stefan Fröberg,
>
> On Sun, 17 Feb 2013 18:34:56 +0200, Stefan Fröberg wrote:
>
>> +	select BR2_PACKAGE_GETTEXT
> Are you sure it really needs gettext as a build dependency?

Well, I did not try to build it without gettext installed but that's
what the gentoo ebuild file
lists as its hard dependency

> Also, this seems to install a big bunch of files. Maybe
> network-manager-applet needs only some of the ISOxyz files, and
> therefore having suboptions to select which ISOxyz set of files should
> be installed would be useful?
>

I have to check it.

Stefan
Thomas Petazzoni - Feb. 17, 2013, 6:11 p.m.
Dear Stefan Fröberg,

On Sun, 17 Feb 2013 19:55:11 +0200, Stefan Fröberg wrote:

> >> +	select BR2_PACKAGE_GETTEXT
> > Are you sure it really needs gettext as a build dependency?
> 
> Well, I did not try to build it without gettext installed but that's
> what the gentoo ebuild file
> lists as its hard dependency

We need to figure out what is needed exactly. gettext support in the C
library (in which case it depends on whether you're using glibc or
uClibc, see
http://buildroot.org/downloads/manual/manual.html#_gettext_integration_and_interaction_with_packages.
Or instead the host gettext tools to generate some stuff.

> > Also, this seems to install a big bunch of files. Maybe
> > network-manager-applet needs only some of the ISOxyz files, and
> > therefore having suboptions to select which ISOxyz set of files should
> > be installed would be useful?
> >
> 
> I have to check it.

Thanks.

Thomas

Patch

diff --git a/package/Config.in b/package/Config.in
index e1788c2..cf4b456 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -571,6 +571,7 @@  endmenu
 
 menu "Text and terminal handling"
 source "package/enchant/Config.in"
+source "package/iso-codes/Config.in"
 source "package/libfribidi/Config.in"
 source "package/icu/Config.in"
 source "package/libiconv/Config.in"
diff --git a/package/iso-codes/Config.in b/package/iso-codes/Config.in
new file mode 100644
index 0000000..ee3ac49
--- /dev/null
+++ b/package/iso-codes/Config.in
@@ -0,0 +1,10 @@ 
+config BR2_PACKAGE_ISO_CODES
+	bool "iso-codes"
+	select BR2_PACKAGE_GETTEXT
+	help
+	  This package provides the ISO-639 Language code list, the ISO-3166
+	  Territory code list, and ISO-3166-2 sub-territory lists, and all
+	  their translations in gettext .po form.
+
+	  http://alioth.debian.org/projects/pkg-isocodes
+
diff --git a/package/iso-codes/iso-codes.mk b/package/iso-codes/iso-codes.mk
new file mode 100644
index 0000000..ac40f9c
--- /dev/null
+++ b/package/iso-codes/iso-codes.mk
@@ -0,0 +1,13 @@ 
+#############################################################
+#
+# iso-codes
+#
+#############################################################
+
+ISO_CODES_VERSION = 3.40
+ISO_CODES_SOURCE = iso-codes-$(ISO_CODES_VERSION).tar.xz
+ISO_CODES_SITE = http://pkg-isocodes.alioth.debian.org/downloads
+ISO_CODES_INSTALL_STAGING = YES
+ISO_CODES_DEPENDENCIES = gettext
+
+$(eval $(autotools-package))