fix pa-IN problems in defines.inc and tabs.dtd

RESOLVED FIXED

Status

Mozilla Localizations
pa-IN / Punjabi
RESOLVED FIXED
12 years ago
11 years ago

People

(Reporter: Pike, Assigned: Pike)

Tracking

({fixed1.8.1.1, verified1.8.1.1})

unspecified
fixed1.8.1.1, verified1.8.1.1
Dependency tree / graph
Bug Flags:
blocking1.8.1.1 +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

(Assignee)

Description

12 years ago
There are XML parsing errors in the pa-IN build, which no tool caught without running.
In addition, I have been unable to create a working langpack, and haven't been able to understand that error.

The xml parsing error is real, though. It doesn't pull brand.dtd in tabs.dtd, but uses &bandShortName;, which makes the prefwindow look bad in the tabs pane.
(Assignee)

Comment 1

12 years ago
Getting this on the radar.

I have a fix in hand, but I still need to actually make the patch. The XML parsing error is fixed, though, and I'm confident I found the bug in the langpack.
Flags: blocking-firefox2?
Plussing to keep it on the radar. In addition to the problems that Axel mentions, there's also problems with the pa-IN help files. Depending on the timing of getting a patch together, we can either take a fix or pull the build from shipped-locales.
Flags: blocking-firefox2? → blocking-firefox2+
Summary: remove pa-IN from Firefox 2 shipped-locales → fix pa-IN problems or remove it from Firefox 2 shipped-locales
(Assignee)

Comment 3

12 years ago
Created attachment 240654 [details] [diff] [review]
fix defines.inc and tabs.dtd

This patch fixes defines.inc to actually be a proper defines.inc, and fixes tabs.dtd to include brand.dtd and no line numbers instead.

Requesting approval1.8.1 to get this landed.
Assignee: nobody → l10n
Status: NEW → ASSIGNED
Attachment #240654 - Flags: approval1.8.1?

Updated

12 years ago
Attachment #240654 - Flags: approval1.8.1? → approval1.8.1+
(Assignee)

Comment 4

12 years ago
Landed the two bustage fixes. Help is not in the best condition, but it offers some basic functionality.

Hrm. I could have CCed Amanpreet earlier, too.
Keywords: fixed1.8.1
Summary: fix pa-IN problems or remove it from Firefox 2 shipped-locales → fix pa-IN problems in defines.inc and tabs.dtd

Comment 5

12 years ago
Created attachment 240657 [details] [diff] [review]
Fix for help files

As far as I could see, help is in English and not uptodate. Simply copy-pasting en-US into pa-IN and making a diff. If this could help.
Attachment #240657 - Flags: approval-l10n?
(Assignee)

Comment 6

12 years ago
Comment on attachment 240657 [details] [diff] [review]
Fix for help files

Not going to attack help for 2.0
Attachment #240657 - Flags: approval-l10n? → approval-l10n-
(Assignee)

Comment 7

12 years ago
This is FIXED, too.
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
(Assignee)

Comment 8

11 years ago
I'm sooooo stupid, and I'm really sorry.

The embarrassment that is attachment 242846 [details] over in bug 353341 is mine, and the patch even shows.

Not FIXED, not fixed1.8.1, and stop-ship, IMHO.

Requesting blocking1.8.1.1 to get that fixed in the next minor release.

Note to self: slam your head against the wall.
Status: RESOLVED → REOPENED
Flags: blocking1.8.1.1?
Keywords: fixed1.8.1
Resolution: FIXED → ---
(Assignee)

Updated

11 years ago
Blocks: 357378
moving flags appropriately
Flags: blocking1.8.1.1?
Flags: blocking1.8.1.1+
Flags: blocking-firefox2+
Created attachment 244986 [details] [diff] [review]
Unbreak tabs.dtd

This patch replaces Pike's "=========" strings with the real Punjabi strings he deleted, but still keeps the tabs.dtd well-formed. :)
Attachment #244986 - Flags: review?(apbrar)
Attachment #244986 - Flags: approval1.8.1.1?
Component: Build Config → pa-IN / Punjabi
Flags: review?(apbrar)
Product: Firefox → Mozilla Localizations
Target Milestone: Firefox 2 → ---
Version: 2.0 Branch → unspecified
Comment on attachment 244986 [details] [diff] [review]
Unbreak tabs.dtd

please nominate for approval only after it's been reviewed
Attachment #244986 - Flags: approval1.8.1.1? → review?(apbrar)

Comment 12

11 years ago
yes this patch contains Punjabi translation by removing =====.
it is ok to include.

thanks
(Assignee)

Comment 13

11 years ago
Comment on attachment 244986 [details] [diff] [review]
Unbreak tabs.dtd

[pa-IN] approval to fix tabs.dtd for firefox 2.0.0.1 granted.

Amanpreet, please check in the fixed tabs.dtd on the MOZILLA_1_8_BRANCH with a check-in comment referencing this bug and my approval. Add the fixed1.8.1.1 keyword to this bug when the check-in is done, and change it to the verified1.8.1.1 keyword after successful testing.

Thanks and sorry again for not fixing this right myself the first time.
Attachment #244986 - Flags: review?(apbrar)
Attachment #244986 - Flags: review+
Attachment #244986 - Flags: approval1.8.1.1+
QA Contact: build.config → apbrar

Comment 14

11 years ago
Thanks Axel,
I have commited changes to CVS and added keyword.
Keywords: fixed1.8.1.1
(Assignee)

Comment 15

11 years ago
[mass-verified1.8.1.1 message, filter on this if you need to]

This bug has an approved patch, but not the verified1.8.1.1 keyword.

Please make sure that the approved patch is landed (add the fixed1.8.1.1 keyword then) and tested. If you verified that the bug in question is indeed fixed on the 1.8 branch nightlies, please add the verified1.8.1.1 keyword.

We need to get the QA activities around Firefox 2.0.0.1 up to speed so that we can feel good about releasing this update.

Comment 16

11 years ago
tested with latest release and it is fixed.

thanks
Status: REOPENED → RESOLVED
Last Resolved: 12 years ago11 years ago
Keywords: verified1.8.1.1
Resolution: --- → FIXED
(Assignee)

Updated

11 years ago
Blocks: 352123
You need to log in before you can comment on or make changes to this bug.