The default bug view has changed. See this FAQ.

Update SeaMonkey package-manifest.in files with new *.manifest files

RESOLVED FIXED in seamonkey2.1a3

Status

SeaMonkey
Build Config
RESOLVED FIXED
7 years ago
5 years ago

People

(Reporter: Ian Neal, Assigned: Robert Kaiser)

Tracking

Trunk
seamonkey2.1a3
Dependency tree / graph

Firefox Tracking Flags

(blocking-seamonkey2.1 a3+)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

7 years ago
With changes from bug 575740 and its blockers the package-manifest.in files need updating to reflect all the new manifest files that have been introduced.
You just need this for SeaMonkey - I did Thunderbird once we got the static builds green and I was doing the initial setup for them (assuming at the time that they would run, but as my last comment on bug 575740 shows I still need to come up with a fix for that).
Component: Build Config → Build Config
Product: MailNews Core → SeaMonkey
QA Contact: build-config → build-config
Summary: Update comm-central package-manifest.in files with new *.manifest files → Update SeaMonkey package-manifest.in files with new *.manifest files

Updated

7 years ago
blocking-seamonkey2.1: --- → a3+
(Assignee)

Updated

7 years ago
Depends on: 576900
(Assignee)

Comment 2

7 years ago
Created attachment 456764 [details] [diff] [review]
first step: fix Linux (at least)

This patch fixes everything I could find in Linux package-compare - there might be some things in Mac and/or Windows that are still missing, but it's easier to find out about those when the majority of cases is out of the way.
Assignee: nobody → kairo
Status: NEW → ASSIGNED
Attachment #456764 - Flags: review?(bugspam.Callek)
(Assignee)

Comment 3

7 years ago
Created attachment 456766 [details] [diff] [review]
first step: fix Linux (at least) [checked in]

Erm, forgot to s/bin/@BINPATH@/ in the package manifest.
Attachment #456764 - Attachment is obsolete: true
Attachment #456766 - Flags: review?(bugspam.Callek)
Attachment #456764 - Flags: review?(bugspam.Callek)
Comment on attachment 456766 [details] [diff] [review]
first step: fix Linux (at least) [checked in]

Mostly an rs+. But I did skim that the lines look good; did not actually compare vs the package-compare output though. If you'd prefer that I do the compare itself I'll do so just will be a longer turnaround.
Attachment #456766 - Flags: review?(bugspam.Callek) → review+
(In reply to comment #4)
> Comment on attachment 456766 [details] [diff] [review]
> first step: fix Linux (at least)
a=me too
(Assignee)

Comment 6

7 years ago
Comment on attachment 456766 [details] [diff] [review]
first step: fix Linux (at least) [checked in]

Pushed as http://hg.mozilla.org/comm-central/rev/39e2a272c3de - let's see how far that takes us :)
Attachment #456766 - Attachment description: first step: fix Linux (at least) → first step: fix Linux (at least) [checked in]
(Assignee)

Comment 7

7 years ago
Created attachment 456841 [details] [diff] [review]
second step: fix Windows

Mac looks good in terms of manifests, the Windows compare-locales brought up one more, though, which is fixed by this patch as well.
Attachment #456841 - Flags: review?(bugspam.Callek)
Comment on attachment 456841 [details] [diff] [review]
second step: fix Windows

a=Callek too.
Attachment #456841 - Flags: review?(bugspam.Callek) → review+
(Assignee)

Comment 9

7 years ago
Pushed as http://hg.mozilla.org/comm-central/rev/2075bf73c39d - so this is fixed for the moment.
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
(Assignee)

Updated

7 years ago
Target Milestone: --- → seamonkey2.1a3
Depends on: 568691
You need to log in before you can comment on or make changes to this bug.