Bug 82 - [PATCH] Duplicate LC_MESSAGES in locale bn_BD
Summary: [PATCH] Duplicate LC_MESSAGES in locale bn_BD
Status: RESOLVED FIXED
Alias: None
Product: glibc
Classification: Unclassified
Component: localedata (show other bugs)
Version: 2.3.3
: P1 normal
Target Milestone: ---
Assignee: Petter Reinholdtsen
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-03-20 18:09 UTC by Petter Reinholdtsen
Modified: 2019-04-10 12:19 UTC (History)
2 users (show)

See Also:
Host:
Target:
Build:
Last reconfirmed:
fweimer: security-


Attachments
Patch for the locale bn_BD (676 bytes, patch)
2004-07-14 08:38 UTC, Jamil Ahmed
Details | Diff
Correct bn_BD locale (1023 bytes, patch)
2004-10-02 08:42 UTC, Petter Reinholdtsen
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Petter Reinholdtsen 2004-03-20 18:09:37 UTC
The locale bn_BD have two LC_MESSAGES sections.  It is the only
duplicate sections in the file. I'n not sure which one of them are
correct.  This is the current content:

  LC_MESSAGES
  yesexpr     "<U005B><U09B9><U09CD><U09AF><U09BE><U005D>"
  noexpr      "<U005B><U09A8><U09BE><U005D>"
  yesstr      "<U09B9><U09CD><U09AF><U09BE>"
  nostr        "<U09A8><U09BE>"
  END LC_MESSAGES

  LC_MESSAGES
  yesstr      "<U0561><U0575><U0578>"
  nostr       "<U0578><U0579>"
  yesexpr "<U005B><U0079><U0059><U0073><U0053><U0561><U0531><U005D><U002E><U002A>"
  noexpr      "<U005B><U006E><U004E><U0578><U0548><U005D><U002E><U002A>"
  END LC_MESSAGES
Comment 1 Taneem Ahmed 2004-03-22 19:54:40 UTC
The first part is valid. I'll add a patch in a day or two because one more line 
needs to be added to the file and the contact information needs to be updated.
Comment 2 Petter Reinholdtsen 2004-04-17 19:13:54 UTC
> The first part is valid. I'll add a patch in a day or two because
> one more line needs to be added to the file and the contact
> information needs to be updated.

I'm still waiting for the patch.
Comment 3 Petter Reinholdtsen 2004-05-15 07:44:37 UTC
The yesexpr/noexpr in the first block is missing 'yY' and 'nN'.
This make it hard to use the locale when translations are missing.
See bug #71 for more info on that issue.  The reason I mention this
is that I wounder if the first block is really correct.
Comment 4 Jamil Ahmed 2004-07-14 08:38:52 UTC
Created attachment 133 [details]
Patch for the locale bn_BD

Removed the additional LC_MESSAGES block.
Added <U0981> in yesexpr and yesstr.
Updated the contact information.
Comment 5 Petter Reinholdtsen 2004-07-14 19:46:57 UTC
I've sent an email to the original locale author, Taneem Ahmed, asking
him to verify that this patch is correct.  When his approval is
available, I'll forward the patch to the glibc maintainers.
Comment 6 Jamil Ahmed 2004-07-17 06:58:28 UTC
Might need to fix some more values.

LC_MONETARY - mon_grouping and LC_NUMERIC - grouping's value is not correct - i
guess.

Some values of LC_TIME - abday and abmon might need corrections too.

I am trying to be sure about those values and will generate another patch - if
required.
Comment 7 Taneem Ahmed 2004-08-09 14:57:43 UTC
For personal reasons I am/will be MIA for a while. Jamil is currently 
maintaining the bn_BD datafile, so please accept his patch.
Comment 8 Petter Reinholdtsen 2004-10-02 08:42:16 UTC
Created attachment 217 [details]
Correct bn_BD locale

Updated patch with changelog and minor corrections to the category
entries.  This patch was just sent to the glibc maintainers on the
libc-hacker mailing list.
Comment 9 cvs-commit@gcc.gnu.org 2004-12-19 20:49:06 UTC
Subject: Bug 82

CVSROOT:	/cvs/glibc
Module name:	libc
Changes by:	aj@sources.redhat.com	2004-12-19 20:49:00

Modified files:
	localedata/locales: bn_BD 

Log message:
	[BZ #82]
	* locales/bn_BD: Remove duplicate LC_MESSAGES block. Correct the
	remaining block.  Change contact info as this locale is now
	maintained by Jamil Ahmed.  Patch from Jamil Ahmed.

Patches:
http://sources.redhat.com/cgi-bin/cvsweb.cgi/libc/localedata/locales/bn_BD.diff?cvsroot=glibc&r1=1.4&r2=1.5

Comment 10 Andreas Jaeger 2004-12-19 20:50:38 UTC
Patch submitted to CVS.