Closed Bug 349192 Opened 15 years ago Closed 15 years ago

Ship Thunderbird 1.5.0.x lt (Lithuanian)

Categories

(Thunderbird :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: rimas, Assigned: rhelmer)

References

Details

(Keywords: fixed1.8.0.7, fixed1.8.0.9)

Attachments

(2 files, 1 obsolete file)

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; lt-LT; rv:1.8.0.5) Gecko/20060731 Ubuntu/dapper-security Firefox/1.5.0.5
Build Identifier: Thunderbird 1.5.0.x

While the Lithuanian translation of Thundebrird 2.0 is already in the CVS, it's not there in case of 1.5.0.x. However, it already exists, passes compare-locales.pl (at least seems to), and is quite well tested out, and available for download as a CE build for windows (http://ims.mii.lt/thunderbird/) and also as a cross-platform xpi file (http://files.akl.lt/Internetas/Mozilla-Thunderbird-LT/1.5/).

This is a request to accept it upstream. I heard from Axel, that Tb 2.0 is only planned to be released in 2007, so there's still a lot of time left, and I believe that it's not yet that useless to mess around with a (not that) new locale in 1.5 a little.

Reproducible: Always
Component: Registration & Management → General
Product: Mozilla Localizations → Thunderbird
Version: unspecified → 1.5
Attached patch The proposed translation (obsolete) — Splinter Review
Flags: blocking1.8.0.8?
Flags: blocking1.8.0.7?
The previously submitted patch had this already #expand'ed:
#expand <!ENTITY  lang.version          "__MOZILLA_LOCALE_VERSION__">

This one corrects the problem.
Attachment #234440 - Attachment is obsolete: true
Assignee: registration → mscott
Status: UNCONFIRMED → NEW
Ever confirmed: true
QA Contact: registration → general
Summary: Please accept Lithuanian localization of Thunderbird into MOZILLA_1_8_0_BRANCH → Ship Thunderbird 1.5.0.x lt (Lithuanian)
Axel: I think this is your call about whether this can make 1.5.0.7 -- I imagine there's not much trademark review in Thunderbird compared to Firefox.
Do we even have a tb 1.5.0.7?
Status: NEW → ASSIGNED
Good point, it's not clear when. There *will* be a "next release" though and this language will have to be added to the appropriate tb lists.
Who will check this in to the 1.8.0 branch?

rhelmer: we'll need a patch for the ship-locales file as well.
Flags: blocking1.8.0.7? → blocking1.8.0.7+
I could check this in if somebody opens up the tree.

btw: is there an easy way to cvs-add such amount of files quickly?
The tree is open. It's "locked down" meaning only approved patches, but this one is approved.
Comment on attachment 234442 [details] [diff] [review]
The proposed translation

Approved to land on the 1.8.0 branch, a=dveditz for drivers
Attachment #234442 - Flags: approval1.8.0.8+
may I just commit the files now?
yes, please commit the files. Thx.
Checked in.

Would you please add Tb in 1.8.0 to the list of what Tinderboxes build? (Axel?)
Assignee: mscott → rhelmer
Status: ASSIGNED → NEW
Flags: blocking1.8.0.8?
Attachment #235817 - Flags: review?
Attachment #235817 - Flags: review? → review?(l10n)
There was a surprise checkin this morning by tatjanaj@julius.ktl.mii.lt to checkin forgotten files. Luckily for the LT team we slipped and the official builds hadn't started by then.

Rimas: make sure your team knows we're in the end-game here, and *everything* happens by exception. If you mess up a checkin, we need a new bug and approval request to cover fixing it. Yes, it slows things down, but it's critical that everyone know when we think things are done, when we're waiting on things, and when problems come up.

What are we waiting for to mark this bug "fixed1.8.0.7"? Trademark review? maybe we can split that off into its own bug to make things clearer.
I'm waiting for builds for the localization team to QA.
rhelmer, could you post a URL to good builds once they're done?
Tatjana, Rimas, once we have builds, please do some extensive testing via litmus and report the corresponding query URLs here?
(In reply to comment #14)
> There was a surprise checkin this morning by tatjanaj@julius.ktl.mii.lt to
> checkin forgotten files. Luckily for the LT team we slipped and the official
> builds hadn't started by then.
> 
> Rimas: make sure your team knows we're in the end-game here, and *everything*
> happens by exception. If you mess up a checkin, we need a new bug and approval
> request to cover fixing it. Yes, it slows things down, but it's critical that
> everyone know when we think things are done, when we're waiting on things, and
> when problems come up.

Sorry for that. Those files don't seem to actually be used anywhere, so I thought we wouldn't need them for the build, so I didn't include them in the patch. Apparently, that was a bad assumption... I'll keep that in mind next time.

(In reply to comment #15)
> I'm waiting for builds for the localization team to QA.
> rhelmer, could you post a URL to good builds once they're done?
> Tatjana, Rimas, once we have builds, please do some extensive testing via
> litmus and report the corresponding query URLs here?

Sure. Whenever I get a link to a binary, I'll test it out. So far I can only see  thunderbird-1.5.0.5.lt.* files in ftp://ftp.mozilla.org/pub/mozilla.org/thunderbird/nightly/latest-mozilla1.8.0-l10n/. Is that the right place to check?
Depends on: 350691
The builds are on, and we're seeing the following problems still, as of
http://tinderbox.mozilla.org/showlog.cgi?log=Mozilla1.8.0-l10n-lt/1157738220.20087.gz&fulltext=1:

iconv -f UTF-8 -t CP1257 /cygdrive/c/builds/tinderbox/Tb-Mozilla1.8.0-l10n-Release/WINNT_5.2_Clobber/mozilla/../l10n/lt/mail/installer/installer.inc > \
  installer.inc
iconv: /cygdrive/c/builds/tinderbox/Tb-Mozilla1.8.0-l10n-Release/WINNT_5.2_Clobber/mozilla/../l10n/lt/mail/installer/installer.inc: cannot convert
make[2]: *** [repackage-win32-installer] Error 1

... plus ...

Properties in ./chrome/messenger/messengercompose/composeMsgs.properties don't match:
  In c:/builds/tinderbox/Tb-Mozilla1.8.0-l10n-Release/WINNT_5.2_Clobber/mozilla/mail/locales/en-US: (add these to your localization)
    sendMsgTitle
  In c:/builds/tinderbox/Tb-Mozilla1.8.0-l10n-Release/WINNT_5.2_Clobber/mozilla/../l10n/lt/mail: (remove these from your localization)
    subjectDlogTitle

Dan, this missed the train, could you '-' the blocking1.8.0.7 flag and request the blocking1.8.0.8 flag, then we can reevaluate this for the next round.

Tatjana, Rimas, please file bugs on these issues, or one, and make them block this bug. Request blocking flags on the bugs and approval on the patches for 1.8.0.8, so that we can land those fixes in the next engineering period. Once they get approved, of course.
Comment on attachment 235817 [details] [diff] [review]
add lt to shipped_locales

Unsetting the review flag, the patch is good, the time sadly is not.
Attachment #235817 - Flags: review?(l10n)
(In reply to comment #17)

Axel, I don't see the errors you mentioned in the latest build. I think you're mistaken. These errors have been fixed days ago, and it's weird they show up now. Could it be something wrong with the tinderboxes maybe?

http://tinderbox.mozilla.org/showlog.cgi?log=Mozilla1.8.0-l10n-lt/1157778240.10770.gz&fulltext=1
Rob, on 
http://tinderbox.mozilla.org/showlog.cgi?log=Mozilla1.8.0-l10n/1157737039.1264.gz&fulltext=1
which is the windows build log you mailed to me, the pull date for l10n is

mk_add_options MOZ_CO_LOCALES_DATE="19 Jul 2006 00:00:00 +0000"

and not "2006-08-31 00:00:00 PDT" as your mail said.

I didn't check the other builds, but please reconfirm all l10n builds, including the tagged firefox tree. That pulls l10n from the release tag, but we should verify that that's the right status of the tree.
(In reply to comment #20)
> Rob, on 
> http://tinderbox.mozilla.org/showlog.cgi?log=Mozilla1.8.0-l10n/1157737039.1264.gz&fulltext=1
> which is the windows build log you mailed to me, the pull date for l10n is
> 
> mk_add_options MOZ_CO_LOCALES_DATE="19 Jul 2006 00:00:00 +0000"
> 
> and not "2006-08-31 00:00:00 PDT" as your mail said.
> 
> I didn't check the other builds, but please reconfirm all l10n builds,
> including the tagged firefox tree. That pulls l10n from the release tag, but we
> should verify that that's the right status of the tree.
> 


Axel, looks like you are right and these are not configured correctly. It looks like we are going to respin en-US again anyway, so I am going to wait until then to respin the l10n builds. 
Mind re-requesting review on the shipped-locales patch when the builds are done and green? They really should be, as the nightlies were.
Sorry for not catching that the first round.
Comment on attachment 235817 [details] [diff] [review]
add lt to shipped_locales

We finally have green Clobber builds here:
http://tinderbox.mozilla.org/showbuilds.cgi?tree=Mozilla1.8.0-l10n-lt&maxdate=1157947981&legend=0
Attachment #235817 - Flags: review?
Comment on attachment 235817 [details] [diff] [review]
add lt to shipped_locales

We have a sign-off and should be good to go.
Attachment #235817 - Flags: review? → review+
Comment on attachment 235817 [details] [diff] [review]
add lt to shipped_locales

We have a sign-off and should be good to go.
checked into MOZILLA_1_8_0_BRANCH:

Checking in shipped-locales;
/cvsroot/mozilla/mail/locales/Attic/shipped-locales,v  <--  shipped-locales
new revision: 1.1.2.1.2.2; previous revision: 1.1.2.1.2.1
done
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
Not really sure why attachment 234442 [details] [diff] [review] says "approval1.8.0.9+", as that patch was committed a long time ago. However, setting fixed1.8.0.9 keyword to get it off the not-committed list.
Keywords: fixed1.8.0.9
You need to log in before you can comment on or make changes to this bug.