Closed Bug 1541179 Opened 5 months ago Closed 5 months ago

Update and tidy suite/app/ build files

Categories

(SeaMonkey :: Build Config, defect)

defect
Not set

Tracking

(seamonkey2.53 affected, seamonkey2.57esr fixed)

RESOLVED FIXED
seamonkey2.65
Tracking Status
seamonkey2.53 --- affected
seamonkey2.57esr --- fixed

People

(Reporter: iann_bugzilla, Assigned: iann_bugzilla)

References

(Blocks 1 open bug)

Details

Attachments

(14 files, 21 obsolete files)

4.22 KB, patch
frg
: review+
stefanh
: feedback+
Details | Diff | Splinter Review
3.41 KB, patch
frg
: review+
stefanh
: feedback+
Details | Diff | Splinter Review
1.65 KB, patch
frg
: review+
Details | Diff | Splinter Review
1.35 KB, patch
frg
: review+
Details | Diff | Splinter Review
2.15 KB, patch
frg
: review+
stefanh
: feedback+
Details | Diff | Splinter Review
1.89 KB, patch
frg
: review+
Details | Diff | Splinter Review
1.61 KB, patch
frg
: review+
stefanh
: feedback+
Details | Diff | Splinter Review
2.89 KB, patch
frg
: review+
stefanh
: feedback+
Details | Diff | Splinter Review
1.08 KB, patch
frg
: review+
Details | Diff | Splinter Review
981 bytes, patch
frg
: review+
Details | Diff | Splinter Review
1.76 KB, patch
frg
: review+
Details | Diff | Splinter Review
753 bytes, patch
frg
: review+
Details | Diff | Splinter Review
1.64 KB, patch
frg
: review+
Details | Diff | Splinter Review
2.88 KB, patch
iann_bugzilla
: review+
iann_bugzilla
: feedback+
iann_bugzilla
: approval-comm-esr60+
Details | Diff | Splinter Review

Port various patches from browser/app build files and tidy up:

  • Bug 743280 - cosmetic edits removed from bug 739710
  • Bug 748389 - consolidate Mac bundle identifier logic in configure, and use it to specify the Mac bundle ID of the installing app in installed web apps' bundles
  • Bug 787180 - Switch between FINAL_TARGET and DIST/bin in various places under browser/
  • Bug 910781 - add support for FINAL_TARGET_FILES
  • Bug 1207890 - Post: Hacks to make --disable-compile-environment work on Mac OS X
  • Bug 1244999 - Move icon-related DEFINES to moz.build in browser/app
  • Bug 1352982 - clobber OSX .app directory before rsyncing files in
  • Bug 1380690 - Part 1 - Save repo and object dir paths in Mac bundle Info.plist files
  • Bug 862770 - Fix --disable-compile-environment a little
  • Bug 1235982 - Investigate enabling VS2015's "Control Flow Guard" (CFG) security checks using /guard:cf
  • Bug 793735 - Make XRE_StartupTimelineRecord() generate TimeStamps and modify its callers to use the appropriate timers
  • Bug 858928 - Switch XRE_StartupTimelineRecord() from PRTime to TimeStamp
  • Bug 1347360 - Remove unnecessary USE_LIBS directive
  • Bug 1387650 - Associate json files with the browser on OS X
  • Bug 1405577 - Restore the ability to open new windows in new spaces when a window is already in fullscreen mode on macOS
  • Bug 1424709 - Force disable the OSX system "nano allocator"
  • Bug 1305339 and bug 1310518 - improve downloads quarantine support and fix crashes
Attached patch Port bug 743280 (obsolete) — Splinter Review
Attached patch a) Port bug 743280 (obsolete) — Splinter Review

Also removes unneeded substitutions for InfoPlist.strings.in as only @MAC_APP_NAME@ exists

Attachment #9055278 - Attachment is obsolete: true
Attached patch b) Port bug 748389 (obsolete) — Splinter Review
Attached patch Port bug 787180 (obsolete) — Splinter Review
Attached patch Port bug 787180 (obsolete) — Splinter Review

Include reversals in Bug 789529 - Fix for updater tests can't find channel-prefs.js - move channel-prefs back to its gre subdir

Attachment #9055317 - Attachment is obsolete: true
Attached patch Port Bug 910781 (obsolete) — Splinter Review
Attached patch c) Port bug 787180 (obsolete) — Splinter Review

Less replacements with FINAL_TARGET

Attachment #9055319 - Attachment is obsolete: true
Attached patch d) Port Bug 910781 (obsolete) — Splinter Review

Unbitrotted

Attachment #9055345 - Attachment is obsolete: true
Attached patch e) Port bug 1207890 (obsolete) — Splinter Review
Attached patch f) Port bug 1244999 (obsolete) — Splinter Review

As the DEFINES names don't match the icon names, solution isn't as neat as the Firefox version.

Attached patch Port bug 394984 and bug 1278042 (obsolete) — Splinter Review
Attached patch h) Port bug 1352982 (obsolete) — Splinter Review
Attached patch i) Port bug 1380690 (obsolete) — Splinter Review

Factor in fix from bug 1407285

Attachment #9055506 - Attachment is obsolete: true
Attachment #9055288 - Attachment description: Port bug 743280 → a) Port bug 743280
Attachment #9055308 - Attachment description: Port bug 748389 → b) Port bug 748389
Attachment #9055479 - Attachment description: Port bug 787180 → c) Port bug 787180
Attachment #9055480 - Attachment description: Port Bug 910781 → d) Port Bug 910781
Attachment #9055484 - Attachment description: Port bug 1207890 → e) Port bug 1207890
Attachment #9055497 - Attachment description: Port bug 1244999 → f) Port bug 1244999
Attachment #9055519 - Attachment description: Port bug 394984 and bug 1278042 → g) Port bug 394984 and bug 1278042
Attachment #9055508 - Attachment description: Port bug 1352982 → h) Port bug 1352982
Attachment #9055512 - Attachment description: Port bug 1380690 → i) Port bug 1380690
Attached patch j) Port bug 862770 (obsolete) — Splinter Review
Attached patch k) Port bug 1235982 (obsolete) — Splinter Review
Attached patch l) Port bug 793735 (obsolete) — Splinter Review

Includes changes from Bug 793735, Bug 858928 and Bug 1045783

Attached patch m) Port bug 1347360 (obsolete) — Splinter Review
Attached patch n) Port changes to Info.plist.in (obsolete) — Splinter Review
Attached patch Tidy up license file insertion (obsolete) — Splinter Review
Attachment #9056177 - Flags: review?(frgrahl)
Attachment #9055479 - Flags: review?(frgrahl)
Attachment #9055480 - Flags: review?(frgrahl)
Attachment #9055497 - Flags: review?(frgrahl)
Attachment #9055527 - Flags: review?(frgrahl)
Attachment #9055804 - Flags: review?(frgrahl)
Attachment #9055837 - Flags: review?(frgrahl)
Attachment #9055841 - Flags: review?(frgrahl)
Attachment #9055288 - Attachment is obsolete: true
Attachment #9056381 - Flags: review?(frgrahl)
Attachment #9055308 - Attachment is obsolete: true
Attachment #9056382 - Flags: review?(frgrahl)
Attachment #9055479 - Attachment is obsolete: true
Attachment #9055479 - Flags: review?(frgrahl)
Attachment #9056383 - Flags: review?(frgrahl)
Attachment #9055480 - Attachment is obsolete: true
Attachment #9055480 - Flags: review?(frgrahl)
Attachment #9056384 - Flags: review?(frgrahl)
Attachment #9055484 - Attachment is obsolete: true
Attachment #9056385 - Flags: review?(frgrahl)
Attachment #9055497 - Attachment is obsolete: true
Attachment #9055497 - Flags: review?(frgrahl)
Attachment #9056386 - Flags: review?(frgrahl)
Attachment #9055508 - Attachment is obsolete: true
Attachment #9056387 - Flags: review?(frgrahl)
Attachment #9055512 - Attachment is obsolete: true
Attachment #9056388 - Flags: review?(frgrahl)
Attachment #9055527 - Attachment is obsolete: true
Attachment #9055527 - Flags: review?(frgrahl)
Attachment #9056389 - Flags: review?(frgrahl)
Attachment #9055804 - Attachment is obsolete: true
Attachment #9055804 - Flags: review?(frgrahl)
Attachment #9056390 - Flags: review?(frgrahl)
Attachment #9055837 - Attachment is obsolete: true
Attachment #9055837 - Flags: review?(frgrahl)
Attachment #9056391 - Flags: review?(frgrahl)
Attachment #9055841 - Attachment is obsolete: true
Attachment #9055841 - Flags: review?(frgrahl)
Attachment #9056392 - Flags: review?(frgrahl)
Attached patch m) Port changes to Info.plist.in (obsolete) — Splinter Review
Attachment #9055893 - Attachment is obsolete: true
Attachment #9056393 - Flags: review?(frgrahl)
Attachment #9056393 - Flags: feedback?(stefanh)
Attachment #9056177 - Attachment is obsolete: true
Attachment #9056177 - Flags: review?(frgrahl)
Attachment #9056394 - Flags: review?(frgrahl)
Attachment #9056381 - Flags: feedback?(stefanh)
Attachment #9056382 - Flags: feedback?(stefanh)
Attachment #9056385 - Flags: feedback?(stefanh)
Attachment #9056387 - Flags: feedback?(stefanh)
Attachment #9056388 - Flags: feedback?(stefanh)
Blocks: 1542608
Attachment #9055519 - Attachment is obsolete: true
Comment on attachment 9056381 [details] [diff] [review]
a) Port bug 743280

As discussed on irc. Tested on esr60 so good to go there too.
Attachment #9056381 - Flags: review?(frgrahl)
Attachment #9056381 - Flags: review+
Attachment #9056381 - Flags: approval-comm-esr60+
Comment on attachment 9056382 [details] [diff] [review]
b) Port bug 748389

As discussed on irc. Tested on esr60 so good to go there too.
Attachment #9056382 - Flags: review?(frgrahl)
Attachment #9056382 - Flags: review+
Attachment #9056382 - Flags: approval-comm-esr60+
Comment on attachment 9056383 [details] [diff] [review]
c) Port bug 787180

As discussed on irc. Tested on esr60 so good to go there too.
Attachment #9056383 - Flags: review?(frgrahl)
Attachment #9056383 - Flags: review+
Attachment #9056383 - Flags: approval-comm-esr60+
Comment on attachment 9056384 [details] [diff] [review]
d) Port Bug 910781

As discussed on irc. Tested on esr60 so good to go there too.
Attachment #9056384 - Flags: review?(frgrahl)
Attachment #9056384 - Flags: review+
Attachment #9056384 - Flags: approval-comm-esr60+
Comment on attachment 9056385 [details] [diff] [review]
e) Port bug 1207890

As discussed on irc. Tested on esr60 so good to go there too.
Attachment #9056385 - Flags: review?(frgrahl)
Attachment #9056385 - Flags: review+
Attachment #9056385 - Flags: approval-comm-esr60+
Comment on attachment 9056386 [details] [diff] [review]
f) Port bug 1244999

As discussed on irc. Tested on esr60 so good to go there too.
Attachment #9056386 - Flags: review?(frgrahl)
Attachment #9056386 - Flags: review+
Attachment #9056386 - Flags: approval-comm-esr60+
Comment on attachment 9056387 [details] [diff] [review]
g) Port bug 1352982

As discussed on irc. Tested on esr60 so good to go there too.
Attachment #9056387 - Flags: review?(frgrahl)
Attachment #9056387 - Flags: review+
Attachment #9056387 - Flags: approval-comm-esr60+
Comment on attachment 9056388 [details] [diff] [review]
h) Port bug 1380690

As discussed on irc. Tested on esr60 so good to go there too.
Attachment #9056388 - Flags: review?(frgrahl)
Attachment #9056388 - Flags: review+
Attachment #9056388 - Flags: approval-comm-esr60+
Comment on attachment 9056389 [details] [diff] [review]
i) Port bug 862770

As discussed on irc. Tested on esr60 so good to go there too.
Attachment #9056389 - Flags: review?(frgrahl)
Attachment #9056389 - Flags: review+
Attachment #9056389 - Flags: approval-comm-esr60+
Comment on attachment 9056390 [details] [diff] [review]
j) Port bug 1235982

As discussed on irc. Tested on esr60 so good to go there too.
Attachment #9056390 - Flags: review?(frgrahl)
Attachment #9056390 - Flags: review+
Attachment #9056390 - Flags: approval-comm-esr60+
Comment on attachment 9056391 [details] [diff] [review]
k) Port bug 793735

As discussed on irc. Tested on esr60 so good to go there too.
Attachment #9056391 - Flags: review?(frgrahl)
Attachment #9056391 - Flags: review+
Attachment #9056391 - Flags: approval-comm-esr60+
Comment on attachment 9056392 [details] [diff] [review]
l) Port bug 1347360

As discussed on irc. Tested on esr60 so good to go there too.
Attachment #9056392 - Flags: review?(frgrahl)
Attachment #9056392 - Flags: review+
Attachment #9056392 - Flags: approval-comm-esr60+
Comment on attachment 9056393 [details] [diff] [review]
m) Port changes to Info.plist.in

As discussed on irc. Tested on esr60 so good to go there too.
Attachment #9056393 - Flags: review?(frgrahl)
Attachment #9056393 - Flags: review+
Attachment #9056393 - Flags: approval-comm-esr60+
Comment on attachment 9056394 [details] [diff] [review]
n) Tidy up license file insertion

As discussed on irc. Tested on esr60 so good to go there too.
Attachment #9056394 - Flags: review?(frgrahl)
Attachment #9056394 - Flags: review+
Attachment #9056394 - Flags: approval-comm-esr60+

Compiled 2.57 Windows, Linux and macOS x64. Tested the Windows and macOS builds and didn't see any problems.

Comment on attachment 9056393 [details] [diff] [review]
m) Port changes to Info.plist.in

+	<key>LSFileQuarantineEnabled</key>
+	<true/>

This will cause all files created by SM to be quarantined. It makes sense for the browser, but I'm not sure about the other parts. For example, if I got it right, opening a file created by Composer will cause the OS to display a warning.
Comment on attachment 9056388 [details] [diff] [review]
h) Port bug 1380690

Hmm, Thunderbird doesn't have this. What does it do?
Comment on attachment 9056381 [details] [diff] [review]
a) Port bug 743280

Not tested, but makes sense (and frg tested it)
Attachment #9056381 - Flags: feedback?(stefanh) → feedback+
Comment on attachment 9056382 [details] [diff] [review]
b) Port bug 748389

Not tested, but makes sense (and frg tested it).
Attachment #9056382 - Flags: feedback?(stefanh) → feedback+
Comment on attachment 9056387 [details] [diff] [review]
g) Port bug 1352982

Not tested, but makes sense (and frg tested it)
Attachment #9056387 - Flags: feedback?(stefanh) → feedback+
Comment on attachment 9056385 [details] [diff] [review]
e) Port bug 1207890

Not tested, but makes sense.
Attachment #9056385 - Flags: feedback?(stefanh) → feedback+

(In reply to Stefan [:stefanh] from comment #51)

Comment on attachment 9056388 [details] [diff] [review]
h) Port bug 1380690

Hmm, Thunderbird doesn't have this. What does it do?

Just puts some extra info in the package for OSX

(In reply to Stefan [:stefanh] from comment #50)

Comment on attachment 9056393 [details] [diff] [review]
m) Port changes to Info.plist.in

  • <key>LSFileQuarantineEnabled</key>
  • <true/>

This will cause all files created by SM to be quarantined. It makes sense
for the browser, but I'm not sure about the other parts. For example, if I
got it right, opening a file created by Composer will cause the OS to
display a warning.

As far as I can see there is only an exclusion path that sits alongside this setting, not an inclusion path. If it cannot distinguish between stuff that is created locally (e.g. in composer) and stuff that is coming in from the internet (web content and emails) it is probably best we leave this out.

Attachment #9056388 - Flags: feedback?(stefanh) → feedback+
Comment on attachment 9056393 [details] [diff] [review]
m) Port changes to Info.plist.in

> (In reply to Stefan [:stefanh] from comment #50)
> > Comment on attachment 9056393 [details] [diff] [review]
> > m) Port changes to Info.plist.in
> > 
> > +	<key>LSFileQuarantineEnabled</key>
> > +	<true/>
> > 
> > This will cause all files created by SM to be quarantined. It makes sense
> > for the browser, but I'm not sure about the other parts. For example, if I
> > got it right, opening a file created by Composer will cause the OS to
> > display a warning.
> 
> As far as I can see there is only an exclusion path that sits alongside this setting, not an inclusion path. If it cannot distinguish between stuff that is created locally (e.g. in composer) and stuff that is coming in from the internet (web content and emails) it is probably best we leave this out.

Yes, f+ if leaving that out. IIRC l decided to not use this (many years ago) for the above reasons.
Attachment #9056393 - Flags: feedback?(stefanh) → feedback+

Reviewed LSFileQuarantineEnabled section, carrying forward r/f/a

Attachment #9056393 - Attachment is obsolete: true
Attachment #9056647 - Flags: review+
Attachment #9056647 - Flags: feedback+
Attachment #9056647 - Flags: approval-comm-esr60+
Keywords: checkin-needed

Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/38f4f5cb9abc
Part 1: Update and tidy suite/app/ build files. r=frg
https://hg.mozilla.org/comm-central/rev/2bb46bd6ed1f
Part 2: Update and tidy suite/app/ build files. r=frg
https://hg.mozilla.org/comm-central/rev/1232d72e39d1
Part 3: Update and tidy suite/app/ build files. r=frg
https://hg.mozilla.org/comm-central/rev/215d8ed0ad37
Part 4: Update and tidy suite/app/ build files. r=frg
https://hg.mozilla.org/comm-central/rev/bea0db75d407
Part 5: Update and tidy suite/app/ build files. r=frg
https://hg.mozilla.org/comm-central/rev/1d08ace47f9f
Part 6: Update and tidy suite/app/ build files. r=frg
https://hg.mozilla.org/comm-central/rev/488f5744c3c4
Part 7: Update and tidy suite/app/ build files. r=frg
https://hg.mozilla.org/comm-central/rev/4b544be36ee0
Part 8: Update and tidy suite/app/ build files. r=frg
https://hg.mozilla.org/comm-central/rev/fce7da3fbd8e
Part 9: Update and tidy suite/app/ build files. r=frg
https://hg.mozilla.org/comm-central/rev/9e78d4c03991
Part 10: Update and tidy suite/app/ build files. r=frg
https://hg.mozilla.org/comm-central/rev/f50798a0397c
Part 11: Update and tidy suite/app/ build files. r=frg
https://hg.mozilla.org/comm-central/rev/09d459e4a13b
Part 12: Update and tidy suite/app/ build files. r=frg
https://hg.mozilla.org/comm-central/rev/a16041732d6b
Part 13: Update and tidy suite/app/ build files. r=frg
https://hg.mozilla.org/comm-central/rev/8dca02aa228a
Part 14: Update and tidy suite/app/ build files - Tidy up license file insertion. r=frg

Status: ASSIGNED → RESOLVED
Closed: 5 months ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.65
Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/bc622f0bd3ec
Fix part 14: Add dependency on LICENSE_TXT_FILE to libs step. rs=bustage-fix
You need to log in before you can comment on or make changes to this bug.