[Bug 2400] New: pt_BR.UTF-8 support recently added to 6.8.2 branch, was untested, and broken

[bugzilla-daemon@freedesktop.org]

Please do not reply to this email: if you want to comment on the bug, go to            
the URL shown below and enter yourcomments there.     
   
https://bugs.freedesktop.org/show_bug.cgi?id=2400          
     
           Summary: pt_BR.UTF-8 support recently added to 6.8.2 branch, was
                    untested, and broken
           Product: xorg
           Version: unspecified
          Platform: PC
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: I18N
        AssignedTo: xorg-bugzilla-noise@freedesktop.org
        ReportedBy: mharris@www.linux.org.uk


Recently, broken pt_BR.UTF-8 support was added to 6.8.1.903.


rm -f pt_BR.UTF-8.lt
cpp -traditional -Dlinux -D__i386__ -D_POSIX_C_SOURCE=199309L                  
        -D_POSIX_SOURCE -D_XOPEN_SOURCE                                
-D_BSD_SOURCE -D_SVID_
SOURCE                           -D_GNU_SOURCE                            
-DWCHAR32=1 <pt_BR.UTF-8 | sed -e '/^#  *[0-9][0-9]*  *.*$/d'                  
     -e '/^#line  *
[0-9][0-9]*  *.*$/d'                    -e '/^[         ]*XCOMM$/s/XCOMM/#/'   
                -e '/^[         ]*XCOMM[^a-zA-Z0-9_]/s/XCOMM/#/'               
        -e '/^
[       ]*XHASH/s/XHASH/#/'                     -e '/\@\@$/s/\@\@$/\\/'
>pt_BR.UTF-8.lt
<stdin>:1: invalid preprocessing directive #XLocale
<stdin>:3: invalid preprocessing directive #Based
<stdin>:7: invalid preprocessing directive #XLC_FONTSET
<stdin>:15: invalid preprocessing directive #fs0
<stdin>:25: invalid preprocessing directive #fs1
<stdin>:34: invalid preprocessing directive #fs2
<stdin>:43: invalid preprocessing directive #fs3
<stdin>:52: invalid preprocessing directive #fs4
<stdin>:61: invalid preprocessing directive #fs5
<stdin>:71: invalid preprocessing directive #ISO10646
<stdin>:72: invalid preprocessing directive #http
<stdin>:84: invalid preprocessing directive #XLC_XLOCALE
<stdin>:92: invalid preprocessing directive #cs0
<stdin>:99: invalid preprocessing directive #cs1
<stdin>:106: invalid preprocessing directive #cs2
<stdin>:114: invalid preprocessing directive #cs3
<stdin>:122: invalid preprocessing directive #cs4
<stdin>:129: invalid preprocessing directive #cs5
<stdin>:136: invalid preprocessing directive #cs6


It was a debian patch that was aparently created from copying a
preprocessed system locale file, and modifying it for pt_BR.UTF-8,
rather than working from source.  It hadn't been submitted by
Debian upstream because it was not yet fit for upstream.

Branden Robinson (credited in the Xorg changelog for the patch)
just informed me he did not submit this, and it wasn't ready yet
for inclusion:



<mharris>     Add support for pt_BR.UTF-8 locale.
<mharris>     Patch by Gustavo Noronha Silva, Branden Robinson, Julien Lafon.
<ajax> .a is just one or more .o files glommed together
<mharris> <stdin>:1: invalid preprocessing directive #XLocale
<mharris> <stdin>:3: invalid preprocessing directive #Based
<mharris> <stdin>:7: invalid preprocessing directive #XLC_FONTSET
<mharris> ...
<Overfiend> well, yeah, in the real world.
<Overfiend> mharris: it's buggy
<Overfiend> mharris: I didn't submit it.
<mharris> why did it get into 6.8.2?
<Overfiend> mharris: someone forgot to work from source
<mharris> Someone forgot to _test_ things who commited them
<Overfiend> they took a post-processed en_US.UTF-8 and went from there
<mharris> ugh
<Overfiend> mharris: surprise!


Should be fixed or reverted, so 6.8.2 is clean.          
     
     
--           
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email         
     
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.