Last Comment Bug 549390 - Package new layers library on shared builds
: Package new layers library on shared builds
Status: VERIFIED FIXED
: fixed-seamonkey2.0.4
Product: SeaMonkey
Classification: Client Software
Component: Build Config (show other bugs)
: Trunk
: All All
: -- major (vote)
: seamonkey2.1a1
Assigned To: Serge Gautherie (:sgautherie)
:
Mentors:
Depends on: layers
Blocks:
  Show dependency treegraph
 
Reported: 2010-03-01 11:55 PST by Robert Kaiser (not working on stability any more)
Modified: 2010-03-01 22:01 PST (History)
2 users (show)
bugzillamozillaorg_serge_20140323: in‑testsuite-
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
fix? (1.22 KB, patch)
2010-03-01 13:10 PST, Robert O'Callahan (:roc) (Exited; email my personal email if necessary)
no flags Details | Diff | Review
(Bv1-191) Support downgrading [Checkin: Comment 8] (903 bytes, patch)
2010-03-01 13:25 PST, Serge Gautherie (:sgautherie)
kairo: review+
kairo: approval‑seamonkey2.0.4+
Details | Diff | Review

Description Robert Kaiser (not working on stability any more) 2010-03-01 11:55:30 PST
After seeing that our packaged tests are broken and some bug 534425 comment claiming it should be backed out because of that, I decided to first take a look at more details. It felt somewhat strange that alive tests, malloc/laek and check tests were successful but builds failed to even start up for packaged tests, so I suspected a packaging issue.

In Linux package-compare, I found this file wasn't packaged:

bin/liblayers.so


I guess we need to package that file with its different names for every platform in shared builds, and that probably will fix the tests.

I need to leave quite soon today, but I wanted this filed so someone else can potentially make progress here.
Comment 1 Robert O'Callahan (:roc) (Exited; email my personal email if necessary) 2010-03-01 13:10:52 PST
Created attachment 429576 [details] [diff] [review]
fix?

Seems like the right thing to do, but I'm not sure if it will fix things.
Comment 2 Serge Gautherie (:sgautherie) 2010-03-01 13:17:05 PST
http://hg.mozilla.org/comm-central/rev/7e6d3908a938
(Av1) Just package it
Comment 3 Serge Gautherie (:sgautherie) 2010-03-01 13:20:15 PST
Comment on attachment 429576 [details] [diff] [review]
fix?


Sorry, "Mid-air collision detected!"...
Comment 4 Serge Gautherie (:sgautherie) 2010-03-01 13:25:42 PST
Created attachment 429580 [details] [diff] [review]
(Bv1-191) Support downgrading
[Checkin: Comment 8]
Comment 5 Robert Kaiser (not working on stability any more) 2010-03-01 13:35:57 PST
(In reply to comment #2)
> http://hg.mozilla.org/comm-central/rev/7e6d3908a938
> (Av1) Just package it

Sorry, but in the future, please get a r= over IRC at least to actually do this. Should be easy to catch someone who can bless it and fixing somebody else's bustage without any blessing from anyone is not nice style.
Comment 6 Robert Kaiser (not working on stability any more) 2010-03-01 13:37:23 PST
That said, 1) thanks to both Rob and Serge for caring about it - and 2) Serge, the mail/ part of Rob's patch might still be relevant to Thunderbird folks!
Comment 7 Serge Gautherie (:sgautherie) 2010-03-01 14:09:27 PST
(In reply to comment #5)

> Sorry, but in the future, please get a r= over IRC at least to actually do
> this. Should be easy to catch someone who can bless it and fixing somebody

Who? You wrote in comment 0 that you weren't available any longer...

> else's bustage without any blessing from anyone is not nice style.

Next time, just don't cc me and ask for someone else to take over!

NB: I'm so used to get all the blame for warning and fixing things when bustage author gets none :-(
Comment 8 Serge Gautherie (:sgautherie) 2010-03-01 14:18:10 PST
Comment on attachment 429580 [details] [diff] [review]
(Bv1-191) Support downgrading
[Checkin: Comment 8]


http://hg.mozilla.org/releases/comm-1.9.1/rev/2660052256b3
Comment 9 Serge Gautherie (:sgautherie) 2010-03-01 15:01:29 PST
V.Fixed, per SM21 tinderboxes.
Comment 10 Takanori MATSUURA 2010-03-01 18:49:18 PST
When I build xulrunner or firefox, liblayers.so does not exist as a shared library.

I suppose liblayers.a is generated and it is combined to libxul.so for xulrunner/firefox.

Does bug 543159 fix this?
Comment 11 Robert Kaiser (not working on stability any more) 2010-03-01 22:01:04 PST
(In reply to comment #7)
> > Sorry, but in the future, please get a r= over IRC at least to actually do
> > this. Should be easy to catch someone who can bless it and fixing somebody
> 
> Who? You wrote in comment 0 that you weren't available any longer...

Someone like me (I probably would have even still been around on IRC at that time), Neil, Standard8 - or even roc - should be around on IRC.

> NB: I'm so used to get all the blame for warning and fixing things when bustage
> author gets none :-(

(In reply to comment #10)
> When I build xulrunner or firefox, liblayers.so does not exist as a shared
> library.

They do not build shared builds by default, if you build --enable-shared, you get it. We need to do shared builds as we can't support libxul builds right now and static builds don't succeed/work for tests.

> I suppose liblayers.a is generated and it is combined to libxul.so for
> xulrunner/firefox.

Right. It's just due to the different linking.

> Does bug 543159 fix this?

No, and apart from that, we're not even ready to unsupport static build packages, let alone shared ones.

Note You need to log in before you can comment on or make changes to this bug.