Closed Bug 350977 Opened 18 years ago Closed 18 years ago

No lightning.xpi for Mac available since 21-Aug-2006 (trunk and mozilla1.8 branch)

Categories

(Calendar :: General, defect)

PowerPC
macOS
defect
Not set
blocker

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: ssitter, Assigned: rhelmer)

Details

(Whiteboard: [no l10n impact])

Attachments

(1 file, 1 obsolete file)

Hilo tinderbox is currently building Lightning on trunk and mozilla1.8 branch.

New builds of thunderbird-*.en-US.mac.dmg are uploaded to ftp server but no lightning.xpi builds.

The last mac-xpi/lightning.xpi version available on
ftp://ftp.mozilla.org/pub/mozilla.org/calendar/lightning/nightly/latest-trunk/
ftp://ftp.mozilla.org/pub/mozilla.org/calendar/lightning/nightly/latest-mozilla1.8/
ftp://ftp.mozilla.org/pub/mozilla.org/mozilla/nightly/experimental/hilo-trunk/
ftp://ftp.mozilla.org/pub/mozilla.org/mozilla/nightly/experimental/hilo-mozilla1.8/

is dated back to 21-Aug-2006. There is no sign that the lightning.xpi is uploaded to the wrong folder (e.g. as in Bug 341089).

This definitely blocks testing work and release of Lightning 0.3.
Attached patch merged from local patch on hilo (obsolete) — — Splinter Review
This is the effective diff of an old patch on hilo (before we put it on auto-update for bug 348679)
Assignee: nobody → rhelmer
Status: NEW → ASSIGNED
Attachment #236420 - Flags: second-review?(preed)
Attachment #236420 - Flags: first-review?(mattwillis)
Comment on attachment 236420 [details] [diff] [review]
merged from local patch on hilo

r1=lilmatt
Attachment #236420 - Flags: first-review?(mattwillis) → first-review+
No lightning 1.8 == blocking0.3
Flags: blocking0.3+
Whiteboard: [no l10n impact]
Comment on attachment 236420 [details] [diff] [review]
merged from local patch on hilo

I think this change would cause tinderbox to die on l10n builds.

See http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/tools/tinderbox/post-mozilla-rel.pl&rev=1.106#763

I have that right?
Attachment #236420 - Flags: second-review?(preed) → second-review-
Attached patch actual changes needed — — Splinter Review
We don't actually need the patch from hilo that changes do_installer(). Tinderbox just needs to get the mac-xpi dir into $lightningXpiStageDir if is_mac() is true.
Attachment #236420 - Attachment is obsolete: true
Attachment #236840 - Flags: second-review?(preed)
Attachment #236840 - Flags: first-review?(mattwillis)
Comment on attachment 236840 [details] [diff] [review]
actual changes needed

r1=lilmatt

We don't need no stinkin' do_installer!
Attachment #236840 - Flags: first-review?(mattwillis) → first-review+
Comment on attachment 236840 [details] [diff] [review]
actual changes needed

I'll buy that.

r=preed
Attachment #236840 - Flags: second-review?(preed) → second-review+
landed:

Checking in post-mozilla-rel.pl;
/cvsroot/mozilla/tools/tinderbox/post-mozilla-rel.pl,v  <--  post-mozilla-rel.pl
new revision: 1.107; previous revision: 1.106
done
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
/pub/mozilla.org/calendar/lightning/nightly/latest-mozilla1.8/mac-xpi/lightning.xpi                                                  06-Sep-2006 08:33  1.7M


-> VERIFIED

Thanks!
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: