Closed Bug 375872 Opened 13 years ago Closed 13 years ago

Update sl locale for Sunbird 0.5 / Lightning 0.5 release

Categories

(Mozilla Localizations :: sl / Slovene, defect, P3)

defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: ssitter, Assigned: aleksr_si)

References

Details

Attachments

(1 file, 1 obsolete file)

Hi,

the sl locale was shipped with 0.3.1 release and is listed in 
http://lxr.mozilla.org/mozilla1.8/source/calendar/locales/shipped-locales

According to the l10n tinderbox http://tinderbox.mozilla.org/Mozilla1.8-l10n-sl/ the locale is not yet updated for the 0.5 release.

I'd like you to update your locale so that it can ship with the next release.
Blocks: 375873
Updated (as per bug 375315). Took this bug to mean that we needn't ask for review first.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Reopening per comment 2.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee: aleksr_si → miles
Status: REOPENED → NEW
Well if I need to translate something, tell me what I need to localize.
What strings?
Status: NEW → ASSIGNED
Hi Martin, see Comment #2 and your tinderbox log files at [http://tinderbox.mozilla.org/Mozilla1.8-l10n-sl/].
Attached file Updated file - timezones.dtd (obsolete) —
Hope that this updated file is ok now.
Attachment #264533 - Flags: review+
Reassigning for inspection.
Thanks, m.
Assignee: miles → ssitter
Status: ASSIGNED → NEW
Hi, I don't do reviews and I don't have checkin privs. Please check in yourself or ask your team owner for this. Thanks.
Assignee: ssitter → miles
Aleks, prosim, vnesi (please check in).
Lp, m.
Assignee: miles → aleksr_si
OS: Windows 2000 → All
Priority: -- → P3
Hardware: PC → All
New file checked in; Martin, please if you could make sure that the Tinderbox (linked above) turns green in a few hours (if it doesn't, you can click on 'L-' and scroll to the bottom, where errors will be shown); then change the resolution to 'VERIFIED'. Thanks.
Status: NEW → RESOLVED
Closed: 13 years ago13 years ago
Resolution: --- → FIXED
You fixed the strings from Bug 379462. But you also introduced new mismatches, as shown by the compare-locales script:

  In /builds/tinderbox/Sb-Mozilla1.8-l10n/Linux_2.4.21-37.EL_Clobber/mozilla/calendar/locales/en-US: (add these keys to you localization)
    pref.timezone.Africa.Asmara
    pref.timezone.Atlantic.Faroe
  In /builds/tinderbox/Sb-Mozilla1.8-l10n/Linux_2.4.21-37.EL_Clobber/mozilla/../l10n/sl/calendar: (remove these keys from your localization)
    pref.timezone.Africa.Asmera
    pref.timezone.Atlantic.Faeroe

Because of this, all sl Sunbird tinderboxen remain orange. Reopening because of this.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Ok, I didn't introduce these, I just didn't notice them so I didn't update the file. Here is the updated version, please check it in!

Thanks, m.
Attachment #264533 - Attachment is obsolete: true
Comment on attachment 264722 [details]
Another update for timezones.dtd

Martin,
it would be really helpful, if you provide the update as a patch file. That would make it much easier, to review your changes directly in Bugzilla.
Attachment #264722 - Flags: review?(aleksr_si)
Martin, 
please see comment 13 in this bug. In the future, you should really CC yourself to bugs such as these, so that you are informed instantly, when something comes up.

Thanks!
I am sorry, I am just translating, I am not a programmer.
Will cc myself in the future, sure.

Lp, m.
New file checked in. Martin, please verify.
Thanks.
Status: REOPENED → RESOLVED
Closed: 13 years ago13 years ago
Resolution: --- → FIXED
Well, Linux and Mac builds seem to have finished properly, they are green on tinderbox status page. Windows is still under way, already 3 hours and a half.  Is that normal? I guess strings files are ok, otherwise Mac and Linux wouldn't finish.

lp, m.
It's all green now!
Thanks, m.
Status: RESOLVED → VERIFIED
Comment on attachment 264722 [details]
Another update for timezones.dtd

Fix was checked in. Removing review request.
Attachment #264722 - Flags: review?(aleksr_si)
You need to log in before you can comment on or make changes to this bug.