Closed Bug 626517 Opened 15 years ago Closed 14 years ago

Extra bytes at the beginning of the /chrome/pt-PT/locale/branding/brand.dtd file in European Portuguese

Categories

(Mozilla Localizations :: pt-PT / Portuguese, defect)

x86
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: david.planella, Assigned: joao.neves)

References

Details

User-Agent: Mozilla/5.0 (X11; U; Linux i686; ca; rv:1.9.2.13) Gecko/20101206 Ubuntu/10.10 (maverick) Firefox/3.6.13 Build Identifier: Mozilla/5.0 (X11; U; Linux i686; ca; rv:1.9.2.13) Gecko/20101206 Ubuntu/10.10 (maverick) Firefox/3.6.13 While importing the Firefox translations in Ubuntu to be included in language packs, we noticed that there is an error in the Brazilian Portuguese translation that prevents the import. Here is the error message in the Launchpad XPI checker: chrome/pt-PT/locale/branding/brand.dtd: Illegal construct This seems to do with some extra bytes that appear at the beginning of the file, which can be seen with a hex editor. It would be great if the Brazilian Portuguese team could have a look at it, which would allow to import and ship the translation in Ubuntu. Thanks! More details: https://translations.launchpad.net/ubuntu/natty/+source/firefox/+imports Reproducible: Always
Assignee: nobody → joao.neves
Component: General → pt-PT / Portuguese
Product: Firefox → Mozilla Localizations
QA Contact: general → joao.neves
That looks like the UTF-8 BOM mark at the beginning of the file
Component: pt-PT / Portuguese → General
Product: Mozilla Localizations → Firefox
Component: General → pt-PT / Portuguese
Product: Firefox → Mozilla Localizations
Btw, I meant to say the Portuguese team on the initial comment (pt-PT), not the Brazilian team (pt-BR), but it seems it got assigned to the right team anyway.
Is this still an issue? With which version? I'm not seeing that char in any of the brand.dtd files...
That'd be on 1.9.2, same on l10n-central. wokbok:~ axelhecht$ cd src/central/releases/l10n-mozilla-1.9.2/pt-PT/ wokbok:pt-PT axelhecht$ file other-licenses/branding/firefox/brand.dtd other-licenses/branding/firefox/brand.dtd: UTF-8 Unicode (with BOM) text
Thanks a lot Axel. I was looking at the files not seeing it. Corrected in changeset 557:98831c3ea194 for brand.dtd. I've found other files in the localization and will correct those next.
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Ok, just pushed changesets in l10n-1.9.2 and l10n-central removing all Byte-Order-Markers. It should be effective for the next versions of mozilla applications. David, sorry for the delay.
Summary: Extra bytes at the beginning of the /chrome/pt-PT/locale/branding/brand.dtd file in Brazilian Portuguese → Extra bytes at the beginning of the /chrome/pt-PT/locale/branding/brand.dtd file in European Portuguese
David Planella added the following comment to Launchpad bug report 704210: Thanks a lot for taking care of this, João! -- http://launchpad.net/bugs/704210
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.