Revert removal of XDR currency from locale/iso-4217.def
diff mbox

Message ID 20140324035239.GG21819@0c3.net
State New
Headers show

Commit Message

Adam Conrad March 24, 2014, 3:52 a.m. UTC
In 7447ccd98ee3944a95247ae23284dfac1de6c2aa, the XDR currency was
removed from locale/iso-4217.def, despite the fact that it's both
still a part of the standard, according to the official table:

http://www.currency-iso.org/dam/downloads/table_a1.xml

... and, more importantly, is referenced from localedata/i18n, so
any quick-and-dirty locale definition that uses "copy i18n" for
LC_MONETARY won't work anymore.

Okay to commit?

---
---

Comments

Adam Conrad March 24, 2014, 11:02 p.m. UTC | #1
On Sun, Mar 23, 2014 at 09:52:39PM -0600, Adam Conrad wrote:
> In 7447ccd98ee3944a95247ae23284dfac1de6c2aa, the XDR currency was
> removed from locale/iso-4217.def, despite the fact that it's both
> still a part of the standard, according to the official table:
> 
> http://www.currency-iso.org/dam/downloads/table_a1.xml
> 
> ... and, more importantly, is referenced from localedata/i18n, so
> any quick-and-dirty locale definition that uses "copy i18n" for
> LC_MONETARY won't work anymore.

Given the trivial nature of this patch, I'll commit in 24h if I don't
get any objections.

... Adam
Ondřej Bílka April 8, 2014, 3:34 p.m. UTC | #2
On Sun, Mar 23, 2014 at 09:52:39PM -0600, Adam Conrad wrote:
> In 7447ccd98ee3944a95247ae23284dfac1de6c2aa, the XDR currency was
> removed from locale/iso-4217.def, despite the fact that it's both
> still a part of the standard, according to the official table:
> 
> http://www.currency-iso.org/dam/downloads/table_a1.xml
> 
> ... and, more importantly, is referenced from localedata/i18n, so
> any quick-and-dirty locale definition that uses "copy i18n" for
> LC_MONETARY won't work anymore.
> 
> Okay to commit?
> 
ok
Adam Conrad May 4, 2014, 5:49 a.m. UTC | #3
On Mon, Mar 24, 2014 at 05:02:09PM -0600, Adam Conrad wrote:
> 
> Given the trivial nature of this patch, I'll commit in 24h if I don't
> get any objections.

Oops, forgot about this one.  Committed now.

Patch
diff mbox

diff --git a/ChangeLog b/ChangeLog
index 720183f..46ca8ec 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@ 
+2014-03-23  Adam Conrad  <adconrad@0c3.net>
+
+	* locale/iso-4217.def: Reintroduce XDR currency.
+
 2014-03-21  Joseph Myers  <joseph@codesourcery.com>
 
 	* sysdeps/mips/bits/wordsize.h: Include <sgidefs.h>.
diff --git a/locale/iso-4217.def b/locale/iso-4217.def
index bcc170b..414f69b 100644
--- a/locale/iso-4217.def
+++ b/locale/iso-4217.def
@@ -162,6 +162,7 @@  DEFINE_INT_CURR("VUV")		/* Vanuatu Vatu  */
 DEFINE_INT_CURR("WST")		/* Samoa Tala  */
 DEFINE_INT_CURR("XAF")		/* Central African Franc (United Republic of Cameroon, Central African Republic, Chad, Congo, Gabon)  */
 DEFINE_INT_CURR("XCD")		/* East Caribbean Dollar (Antiqua, Dominica, Grenada, Montserrat, St. Kitts-Nevis-Anguilla, Saint Lucia, Saint Vincent and the Grenadines)  */
+DEFINE_INT_CURR("XDR")		/* International Monetary Fund  */
 DEFINE_INT_CURR("XOF")		/* West African Franc (Benin, Ivory Coast, Niger, Senegal, Togo, Upper Volta)  */
 DEFINE_INT_CURR("XPF")		/* French polynesia, New Caledonia, Wallis and Futuna Islands  */
 DEFINE_INT_CURR("YER")		/* Yemeni Rial  */