Parsing errors in it localization preventing Android native nightly from building

RESOLVED FIXED

Status

Mozilla Localizations
it / Italian
--
blocker
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: WG9s, Assigned: flod)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

7 years ago
The following parsing errors occurred in the 20111215 Android native nightly build.

03:52:44    ERROR -  res/values-it/strings.xml:131: error: Apostrophe not preceded by \ (in Per attivare selezionare “Collega un dispositivo” sull'altro dispositivo.)
03:52:44    ERROR -  res/values-it/strings.xml:141: error: Apostrophe not preceded by \ (in Inserire le informazioni sull'account Firefox Sync)
03:52:44    ERROR -  res/values-it/strings.xml:158: error: Apostrophe not preceded by \ (in L'account Firefox Sync è già configurato. Per gestire l'account andare in Impostazioni.)
(Reporter)

Updated

7 years ago
Blocks: 713442
(Reporter)

Comment 1

7 years ago
(In reply to Bill Gianopoulos from comment #0)
> The following parsing errors occurred in the 20111215 Android native nightly
                                               ^^^^^^^^
                                               20111225
> build.
(Assignee)

Comment 2

7 years ago
I'm going to fix this ASAP replacing ' with typographic apostrophes, but this should be fixed on the build side once for all (see old bug 622429).

Where can I check if my next commit breaks something? Is there a tinderbox-like page for Android?
(Reporter)

Comment 3

7 years ago
I agree this should be handled better.  Android builds show up on the Firefox page so just https://tbpl.mozilla.org/

Problem is not much being checked in to trigger a new build due to the Holiday and it also being Sunday.  IT does not seem that changes to l10n-central trigger a new build/test sequence.
(Reporter)

Comment 4

7 years ago
Also, I have a strange suspicion that the tinderbox builds might be en-US only.
(Assignee)

Comment 5

7 years ago
Pushed
http://hg.mozilla.org/l10n-central/it/rev/cf789ec87c4a

Just to be on the safe side I replaced all apostrophes in android/chrome and android/base (those 3 errors are from base/sync_strings.dtd).
(Assignee)

Updated

7 years ago
Assignee: l10n-it → francesco.lodolo
(Reporter)

Comment 6

7 years ago
(In reply to flod (Francesco Lodolo) from comment #2)
> Where can I check if my next commit breaks something? Is there a
> tinderbox-like page for Android?

I finally found someone who could retrigger the nightly build and it completed normally with your fix.
(Assignee)

Comment 7

7 years ago
(In reply to Bill Gianopoulos from comment #6)
> I finally found someone who could retrigger the nightly build and it
> completed normally with your fix.

Thanks a lot Bill, update just downloaded and it's localized ;-)
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED

Comment 8

7 years ago
Commented in bug 713442
You need to log in before you can comment on or make changes to this bug.