Closed Bug 1534028 Opened 5 years ago Closed 5 years ago

Remove omni format checking from SeaMonkey build files.

Categories

(SeaMonkey :: Build Config, enhancement)

enhancement
Not set
normal

Tracking

(seamonkey2.49esr wontfix, seamonkey2.63 wontfix, seamonkey2.53+ fixed, seamonkey2.57esr+ fixed)

RESOLVED FIXED
seamonkey2.64
Tracking Status
seamonkey2.49esr --- wontfix
seamonkey2.63 --- wontfix
seamonkey2.53 + fixed
seamonkey2.57esr + fixed

People

(Reporter: frg, Assigned: frg)

Details

Attachments

(2 files)

We are checking MOZ_OMNIJAR and MOZ_PACKAGER_FORMAT omni in our build files.
I am not aware that there is another format still used and supported in SeaMonkey. We are the only product doing checks outside of toolkit and I think it can safely be removed.

Tested with 2.53 and 2.57

Attachment #9049750 - Flags: review?(iann_bugzilla)
Attachment #9049750 - Flags: approval-comm-esr60?
Comment on attachment 9049750 [details] [diff] [review]
1534028-omni.patch

When there is a good chunk of whitespace only changes it would be useful having two versions of the patch uploaded, the one for checkin with the white space changes and the one for review without the white space changes.
r/a=me
Attachment #9049750 - Flags: review?(iann_bugzilla)
Attachment #9049750 - Flags: review+
Attachment #9049750 - Flags: approval-comm-esr60?
Attachment #9049750 - Flags: approval-comm-esr60+

@DIR_MACOS@update.locale is always removed if found so no need to check for MOZ_UPDATER.

Setting a+ works :)

Attachment #9051497 - Flags: review+
Attachment #9051497 - Flags: approval-comm-esr60+

Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/2cf104a9541f
Remove MOZ_OMNIJAR defines and checks from suite. r=IanN
https://hg.mozilla.org/comm-central/rev/eaecfb05b4a4
Remove update.locale unconditionally during install. r=me

Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED

https://hg.mozilla.org/releases/comm-esr60/rev/71052d0667bda17390ff6dbc0449d5506a9f6353
Remove MOZ_OMNIJAR defines and checks from suite. r=IanN a=IanN

https://hg.mozilla.org/releases/comm-esr60/rev/dbe20c1319c72baa66291e611056ee75f7865e29
Remove update.locale unconditionally during install. r=me a=me

Target Milestone: --- → seamonkey2.64
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: