The default bug view has changed. See this FAQ.

Package new layers library on shared builds

VERIFIED FIXED in seamonkey2.1a1

Status

SeaMonkey
Build Config
--
major
VERIFIED FIXED
7 years ago
7 years ago

People

(Reporter: Robert Kaiser, Assigned: sgautherie)

Tracking

({fixed-seamonkey2.0.4})

Trunk
seamonkey2.1a1
fixed-seamonkey2.0.4
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

7 years ago
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.
Created attachment 429576 [details] [diff] [review]
fix?

Seems like the right thing to do, but I'm not sure if it will fix things.
Assignee: nobody → roc
Attachment #429576 - Flags: review?
(Assignee)

Comment 2

7 years ago
http://hg.mozilla.org/comm-central/rev/7e6d3908a938
(Av1) Just package it
Assignee: roc → sgautherie.bz
Severity: normal → major
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1a1
(Assignee)

Comment 3

7 years ago
Comment on attachment 429576 [details] [diff] [review]
fix?


Sorry, "Mid-air collision detected!"...
Attachment #429576 - Attachment is obsolete: true
Attachment #429576 - Flags: review?
(Assignee)

Comment 4

7 years ago
Created attachment 429580 [details] [diff] [review]
(Bv1-191) Support downgrading
[Checkin: Comment 8]
Attachment #429580 - Flags: review?(kairo)
Attachment #429580 - Flags: approval-seamonkey2.0.4?
(Reporter)

Updated

7 years ago
Attachment #429580 - Flags: review?(kairo)
Attachment #429580 - Flags: review+
Attachment #429580 - Flags: approval-seamonkey2.0.4?
Attachment #429580 - Flags: approval-seamonkey2.0.4+
(Reporter)

Comment 5

7 years ago
(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.
(Reporter)

Comment 6

7 years ago
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!
(Assignee)

Comment 7

7 years ago
(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 :-(
(Assignee)

Comment 8

7 years ago
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
Attachment #429580 - Attachment description: (Bv1-191) Support downgrading → (Bv1-191) Support downgrading [Checkin: Comment 8]
(Assignee)

Updated

7 years ago
Keywords: fixed-seamonkey2.0.4
(Assignee)

Comment 9

7 years ago
V.Fixed, per SM21 tinderboxes.
Status: RESOLVED → VERIFIED

Comment 10

7 years ago
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?
(Reporter)

Comment 11

7 years ago
(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.
You need to log in before you can comment on or make changes to this bug.