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

Status: VERIFIED FIXED

Description

[Stefan Sitter [:ssitter]]

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.

Comment 1

[Robert Helmer [:rhelmer]]

Created attachment 236420 [details] [diff] [review]
merged from local patch on hilo

This is the effective diff of an old patch on hilo (before we put it on auto-update for bug 348679)

Comment 2

[Matthew (lilmatt) Willis]

Comment on attachment 236420 [details] [diff] [review]
merged from local patch on hilo

r1=lilmatt

Comment 3

[Matthew (lilmatt) Willis]

No lightning 1.8 == blocking0.3

Comment 4

[J. Paul Reed [:preed]]

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?

Comment 5

[Robert Helmer [:rhelmer]]

Created attachment 236840 [details] [diff] [review]
actual changes needed

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.

Comment 6

[Matthew (lilmatt) Willis]

Comment on attachment 236840 [details] [diff] [review]
actual changes needed

r1=lilmatt

We don't need no stinkin' do_installer!

Comment 7

[J. Paul Reed [:preed]]

Comment on attachment 236840 [details] [diff] [review]
actual changes needed

I'll buy that.

r=preed

Comment 8

[Robert Helmer [:rhelmer]]

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

Comment 9

[Matthew (lilmatt) Willis]

/pub/mozilla.org/calendar/lightning/nightly/latest-mozilla1.8/mac-xpi/lightning.xpi                                                  06-Sep-2006 08:33  1.7M


-> VERIFIED

Thanks!