Closed Bug 1512882 (SM2.49.5) Opened 2 years ago Closed 11 months ago

Tracking bug for build and release of SeaMonkey 2.49.5

Categories

(SeaMonkey :: Release Engineering, enhancement, P1)

SeaMonkey 2.49 Branch
enhancement

Tracking

(seamonkey2.49esr affected)

RESOLVED FIXED
Tracking Status
seamonkey2.49esr --- affected

People

(Reporter: frg, Assigned: ewong)

References

()

Details

Attachments

(13 files, 3 obsolete files)

608.53 KB, patch
iann_bugzilla
: feedback+
iann_bugzilla
: approval-comm-esr52+
Details | Diff | Splinter Review
4.36 KB, patch
iann_bugzilla
: review+
iann_bugzilla
: approval-comm-esr52+
Details | Diff | Splinter Review
2.84 KB, patch
iann_bugzilla
: review+
iann_bugzilla
: approval-comm-esr52+
Details | Diff | Splinter Review
2.10 KB, patch
iann_bugzilla
: review+
iann_bugzilla
: approval-comm-esr52+
Details | Diff | Splinter Review
8.88 KB, image/png
Details
2.14 KB, patch
frg
: review+
iann_bugzilla
: approval-comm-esr52+
Details | Diff | Splinter Review
3.19 KB, patch
frg
: review+
Details | Diff | Splinter Review
8.75 KB, patch
iann_bugzilla
: review+
iann_bugzilla
: approval-comm-esr52+
Details | Diff | Splinter Review
68.62 KB, patch
frg
: review+
Details | Diff | Splinter Review
2.68 KB, patch
frg
: review+
Details | Diff | Splinter Review
3.07 KB, patch
frg
: review+
Details | Diff | Splinter Review
1.90 KB, patch
frg
: review+
Details | Diff | Splinter Review
587 bytes, patch
Details | Diff | Splinter Review
+++ This bug was initially created as a clone of Bug #1459133 +++

This is a tracking bug for Build and Release of SeaMonkey 2.49.5

We expect an actual release on tba.
Alias: SM2.49.5
Depends on: SM2495-RELNOTE
patches-2_49_5.zip contains the patch queue which can be used to build 2.49.5. It needs to be checked into mozilla-esr52 as a branch.

Basically all esr52 patches are from the old TB 52 verbranch.
esr60 patches are security backports up to Gecko 60.2. I plan to port additonal fixes while the 2.49.5 builds are being prepared.
Rest is mostly VS2017 support and a few pickings from upstreak.

The 3 1512882* patches are my own and have not been reviewed by anyone. See individual attachments.

Bills 2.49.x build contains all of them. I removed private local and WIP patches from the official queue.

The SeaMonkey comm-esr52 branch need just to be merged with the default branch. Currently everything needed is already checked in.
Attachment #9030495 - Flags: feedback?(iann_bugzilla)
Attachment #9030495 - Flags: feedback?(ewong)
Individual patch needed for newer rust versions and clang 6 for macOS support in case we want to use cross compile.
Use the older Windows search icon on Windows 8.x. The default skinny 10 one looks terrible here.
Assignee: nobody → frgrahl
The wow_helper was removed after 52 because it is only needed on Vista and XP x64 when using an x86 build. It will not compile or link with VS2017 without this fix because the compiler and linker are in a different location.
Locally all including l10n is peachy but config changes might be required. 

I would recommend:

VS2017 15.6.7 or 15.8.3+ (not sure if a tooltool package exists). 
Windows toolkit 10.0.17763.x
Latest rust for whcih a tooltool package is available.
gcc as is or 5.x 
clang and macOS sdk as is unless we drop the universal build.
Assignee: frgrahl → nobody
Kinda lost here.  Do you mean all those patches need to be applied to our relbranch?

We don't have a vs2017 tooltool package.  Will need to get it off tb's tc or something.

This is shaping up to be a harsh release.
Comment on attachment 9030497 [details] [diff] [review]
1512882-clang-osx.patch pushed see comment 31

esr52 only.
Various compiler fixes
- Support clang 6 for cross compiling macOS builds
- Potential MSVC fix
- Remove [root] from Cargo.lock to enable compiling with later rust versions.
Attachment #9030497 - Flags: review?(iann_bugzilla)
Attachment #9030497 - Flags: approval-comm-esr52?
Comment on attachment 9030501 [details] [diff] [review]
1512882-search.patch
pushed see comment 30

mozilla esr52 SeaMonkey relbranch only

Use the close icon from previous Windows version in 8.x too.
Attachment #9030501 - Flags: review?(iann_bugzilla)
Attachment #9030501 - Flags: approval-comm-esr52?
In lower toolbar over the tabs. Fits right in. The later older Windows 10 style one from Firefox looks awful here.
Comment on attachment 9030504 [details] [diff] [review]
1512882-vs2017-x86.patch pushed see comment 32

mozilla esr52 SeaMonkey relbranch only

Support VS2017 compiling x64 wow helper in x86 builds.
Attachment #9030504 - Flags: review?(iann_bugzilla)
Attachment #9030504 - Flags: approval-comm-esr52?
Comment on attachment 9030497 [details] [diff] [review]
1512882-clang-osx.patch pushed see comment 31

Actually mozilla esr52 SeaMonkey relbranch only too
Comment on attachment 9030495 [details] [diff] [review]
mozilla esr52 new SeaMonkey 2.49 ESR branch patches pushed 01/13/2019

f=me
Attachment #9030495 - Flags: feedback?(iann_bugzilla)
Attachment #9030495 - Flags: feedback+
Attachment #9030495 - Flags: approval-comm-esr52+
Comment on attachment 9030497 [details] [diff] [review]
1512882-clang-osx.patch pushed see comment 31

r/a=me
Attachment #9030497 - Flags: review?(iann_bugzilla)
Attachment #9030497 - Flags: review+
Attachment #9030497 - Flags: approval-comm-esr52?
Attachment #9030497 - Flags: approval-comm-esr52+
Comment on attachment 9030501 [details] [diff] [review]
1512882-search.patch
pushed see comment 30

r/a=me
Attachment #9030501 - Flags: review?(iann_bugzilla)
Attachment #9030501 - Flags: review+
Attachment #9030501 - Flags: approval-comm-esr52?
Attachment #9030501 - Flags: approval-comm-esr52+
Comment on attachment 9030504 [details] [diff] [review]
1512882-vs2017-x86.patch pushed see comment 32

r/a=me
Attachment #9030504 - Flags: review?(iann_bugzilla)
Attachment #9030504 - Flags: review+
Attachment #9030504 - Flags: approval-comm-esr52?
Attachment #9030504 - Flags: approval-comm-esr52+
Attachment #9030495 - Flags: feedback?(ewong)
Attached patch bug_1512882.diff (obsolete) — Splinter Review

Notes regarding this patch:

  1. It's based on the new_infra_clean branch and not the seamonkey-production
    one that we used to use. The new BB infra uses BB 1.20 which requires
    some upheaval (and personal touches).
  2. This patch is to give our new bb something to chew on as it's failing
    with c-c.
  3. I expect the build # to rack up. (Place your bets here.... :P )
Assignee: nobody → ewong
Attachment #9035223 - Flags: review?(frgrahl)
Attached patch bug_1512882.diff (obsolete) — Splinter Review

Updated the branches and now you have the review honor :)

Changes:

  • milestone is 52.9.1.
  • new csets for comm-esr52 and mozilla-esr52
  • branch SEAMONKEY_2_49_ESR_RELBRANCH from comm-esr52. I suspect relbranchPrefix": "SEA_COMM" needs to be adjusted or a new tag is needed.

l10n tags are unchanged from 2.49.4. I didn't verify the file.

Attachment #9035223 - Attachment is obsolete: true
Attachment #9035223 - Flags: review?(frgrahl)
Attachment #9036196 - Flags: review?(ewong)
Comment on attachment 9036196 [details] [diff] [review]
bug_1512882.diff

Review of attachment 9036196 [details] [diff] [review]:
-----------------------------------------------------------------

Will need to change the cset for comm-esr52 as I'll be pushing a patch (if needed, pending your clarification) to have ESR build with VS2017.
Attachment #9036196 - Flags: review?(ewong) → review+
Comment on attachment 9036249 [details] [diff] [review]
Use vs2017 for building. Pushed see comment 33

Review of attachment 9036249 [details] [diff] [review]:
-----------------------------------------------------------------

This might need a ac_add_options --with-visual-studio-version=2017 in the mozconfigs. Not sure if it picks VS2017 if 2015 is not available.
Attachment #9036249 - Flags: review?(frgrahl) → review+

Comment on attachment 9036249 [details] [diff] [review]
Use vs2017 for building. Pushed see comment 33

need to push this to comm-esr52

Attachment #9036249 - Flags: approval-comm-esr52?

Comment on attachment 9036249 [details] [diff] [review]
Use vs2017 for building. Pushed see comment 33

a=me

Attachment #9036249 - Flags: approval-comm-esr52? → approval-comm-esr52+
Attached patch build config for 2.49.5 (obsolete) — Splinter Review

cset from comm-esr52 corrected after latest config push. r+ retained.

Attachment #9036196 - Attachment is obsolete: true
Attachment #9036277 - Flags: review+

opps wrong file. r+ retained.

Attachment #9036277 - Attachment is obsolete: true
Attachment #9036278 - Flags: review+

Comment on attachment 9036278 [details] [diff] [review]
2.49.5 build config pushed in comment 28

ewong for the push I can't find the new branch you mentioned in buildbot config. Do I/we need to create one?

Flags: needinfo?(ewong)

(In reply to Frank-Rainer Grahl (:frg) from comment #25)

Comment on attachment 9036278 [details] [diff] [review]
2.49.5 build config

ewong for the push I can't find the new branch you mentioned in buildbot
config. Do I/we need to create one?

It should be there.

https://hg.mozilla.org/SeaMonkey/buildbot-configs/

Flags: needinfo?(ewong)

Thanks. Didn't know about this one. Only knew the seamonkey-production branch in the normal repo.
https://hg.mozilla.org/build/buildbot-configs/

Attachment #9036278 - Attachment description: 2.49.5 build config → 2.49.5 build config pushed in comment 28
Duplicate of this bug: 1520728
Depends on: 1521254
Comment on attachment 9030501 [details] [diff] [review]
1512882-search.patch
pushed see comment 30

https://hg.mozilla.org/releases/mozilla-esr52/rev/0f943991c228
Bug 1512882 - Use Windows 7 search icon on Windows 8.x. r=IanN a=IanN CLOSED TREE DONTBUILD
Attachment #9030501 - Attachment description: 1512882-search.patch → 1512882-search.patch pushed see comment 30
Comment on attachment 9030497 [details] [diff] [review]
1512882-clang-osx.patch pushed see comment 31

https://hg.mozilla.org/releases/mozilla-esr52/rev/905d6e43e2ec
Bug 1512882 - Allow macOS cross compile with clang 6 plus VS2017 and later rust fix. r=IanN a=IanN CLOSED TREE DONTBUILD
Attachment #9030497 - Attachment description: 1512882-clang-osx.patch → 1512882-clang-osx.patch pushed see comment 31
Comment on attachment 9030504 [details] [diff] [review]
1512882-vs2017-x86.patch pushed see comment 32

https://hg.mozilla.org/releases/mozilla-esr52/rev/d47e78bd1354
Bug 1512882 - Support building comm-esr52 x86 with VS2017. r=IanN a=IanN CLOSED TREE DONTBUILD
Attachment #9030504 - Attachment description: 1512882-vs2017-x86.patch → 1512882-vs2017-x86.patch pushed see comment 32
Comment on attachment 9036249 [details] [diff] [review]
Use vs2017 for building. Pushed see comment 33

https://hg.mozilla.org/releases/comm-esr52/rev/2cb6da32d02e
Bug 1512882 - Update VS tooltool packages for VS2017 15.9.5. r=frg a=IanN
Attachment #9036249 - Attachment description: use vs2017 for building. → Use vs2017 for building. Pushed see comment 33
Attachment #9030495 - Attachment description: mozilla esr52 new SeaMonkey 2.49 ESR branch patches. → mozilla esr52 new SeaMonkey 2.49 ESR branch patches pushed 01/13/2019

Allows to use a branch for checkout. I didn't try a clone but should work too.

Uses the default changesets for cZ and DOMi which are compatible with 2.49.5

If rev is specified this will be used for the update. Incompatible with branch but if the rev is in the branch the source will be updated to it.

Attachment #9073438 - Flags: review?(iann_bugzilla)
Attachment #9073438 - Flags: approval-comm-esr52?

Patch for our SEAMONKEY_2_49_RELBRANCH to undo the changes done to disable extension building because of l10n bug 1231349

r+ a+ some time ago from IanN over irc. If not let me know.

There is no easy way to set a branch to the default branch and we did sone changes in between to the branch so backing out the original changes would be cumbersome.

Attachment #9073439 - Flags: review+
Attachment #9073439 - Flags: approval-comm-esr52+
Comment on attachment 9073438 [details] [diff] [review]
1512882-client_repo.patch  pushed in comment 37

LGTM r/a=me
Attachment #9073438 - Flags: review?(iann_bugzilla)
Attachment #9073438 - Flags: review+
Attachment #9073438 - Flags: approval-comm-esr52?
Attachment #9073438 - Flags: approval-comm-esr52+
Attachment #9073438 - Attachment filename: 1512882-client_repo.patch → 1512882-client_repo.patch pushed in comment 37
Attachment #9073438 - Attachment description: 1512882-client_repo.patch → 1512882-client_repo.patch pushed in comment 37
Attachment #9073438 - Attachment filename: 1512882-client_repo.patch pushed in comment 37 → 1512882-client_repo.patch
Attachment #9073439 - Attachment description: 1512882-restore-extensions.patch → 1512882-restore-extensions.patch pushed in comment 37
Attachment #9078192 - Flags: review+
Attachment #9078192 - Flags: approval-comm-esr52+
Attachment #9078193 - Flags: review+
Attachment #9078193 - Flags: approval-comm-esr52+
Attachment #9078194 - Flags: review+
Attachment #9078194 - Flags: approval-comm-esr52+

Considering Attachment #9073439 [details] [diff] (restore extensions patch):

I try to disable irc and inspector at build time (yes, questionable idea, but it is buid od a package for a distribution with its own rules etc.)
For this, I specify "--enable-extensions=default,-inspector,-irc" (or just "--enable-extensions=gio" to avoid defaults).

It leads to a situation when dist/bin/distribution/extensions directory just does not exist (since nothing need it). And then suite/app/Makefile complains:
make[5]: *** No rule to make target ../../dist/bin/distribution/extensions', needed bypack-ext'. Stop.

Manually adding of this dir removes the message only, the things still are failed anyway.

This is fixed in bug 1559419. So far I was under the assumption that it does not apply to comm-esr52. If we need another 2.49 I will add it there too but just putting the line in app/Makefile.in should fix it for local builds.

Yep, still missing in build3

Well, the patch from bug #1559419 fixes the build itself, but there are still issues when I make langpacks (a bit separately from the usual build process).

After the normal build is complete, I perform:

pushd obj-*/suite/locales

for lang in $languages
do
make merge-$lang LOCALE_MERGEDIR=../../l10n-merges
make langpack-$lang LOCALE_MERGEDIR=../../l10n-merges
done

popd

and for the first lang(=cs) in the list "make langpack-cs" faled with:

make: *** ../../extensions/irc/locales: No such file or directory. Stop.
make: *** [libs-cs] Error 2

Since ready-made xpi langpacks (as in 2.49.4) no more provided, it is an issue.

(BTW, whether it is possible to have version, or date, or git etc. in the name of l10n tarball? And it would be fine to have one common initial subdir in tarball as well...)

whether it is possible to have version
Sure it has. No more work at the night... :)

Having common initial subdir for l10n tarball would be fine for rpm-based distros.

One more patch needed to build SM-2.49.5 with system nss/nspr.
Something changed since 2.49.4, and now we should remove "crmf" stuff there.

Probably related somehow to bug #1371991 , but it removes in another place...

For bug #1512881 comment #44:

make: *** ../../extensions/irc/locales: No such file or directory. Stop.
make: *** [libs-cs] Error 2

can be fixed by a kludge:

diff -Nrbu seamonkey-2.49.5/seamonkey-2.49.5/suite/locales/Makefile.in seamonkey-2.49.5-OK/seamonkey-2.49.5/suite/locales/Makefile.in
--- seamonkey-2.49.5/seamonkey-2.49.5/suite/locales/Makefile.in 2019-07-14 00:01:29.000000000 +0300
+++ seamonkey-2.49.5-OK/seamonkey-2.49.5/suite/locales/Makefile.in 2019-08-19 22:45:21.358643583 +0300
@@ -123,7 +123,7 @@
@$(MAKE) -C $(DEPTH)/toolkit/locales libs-$*
@$(MAKE) -C $(DEPTH)/services/sync/locales AB_CD=$* XPI_NAME=locale-$*
@$(MAKE) -C ../../editor/ui/locales AB_CD=$* XPI_NAME=locale-$*
- @$(MAKE) -C $(DEPTH)/extensions/irc/locales libs-$*
+ @$(MAKE) -C $(DEPTH)/extensions/irc/locales libs-$* || :
@$(MAKE) -C $(DEPTH)/extensions/spellcheck/locales AB_CD=$* XPI_NAME=locale-$*
@$(MAKE) -C $(DEPTH)/intl/locales AB_CD=$* XPI_NAME=locale-$*
@$(MAKE) -C $(DEPTH)/devtools/client/locales AB_CD=$* XPI_NAME=locale-$*

But then, at "make install" stage,

Error: /home/buc/rpmbuild/BUILD/seamonkey-2.49.5/seamonkey-2.49.5/obj-x86_64-pc-linux-gnu/suite/installer/package-manifest:639: Missing file(s): bin/distribution/extensions/{59c81df5-4b7a-477b-912d-4e0fdf64e5f2}.xpi
Error: /home/buc/rpmbuild/BUILD/seamonkey-2.49.5/seamonkey-2.49.5/obj-x86_64-pc-linux-gnu/suite/installer/package-manifest:642: Missing file(s): bin/distribution/extensions/inspector@mozilla.org.xpi

It seems that [chatzilla] and [inspector] sections should be ifdef'ed (like for MOZ_CALENDAR) in suite/installer/package-manifest.in ...

Is it applicable just to remove unneeded distribution/extensions/* files after the package is built, instead of non-working "--enable-extensions"? Or maybe its presence is mentioned something else in the generated code?

Well we were actually happy to build them again so tests disabling them were not really done. This is all fixed in 2.53 and up where you can turn them on and off in the mozconfig with enable directives.

You can try backing out https://hg.mozilla.org/releases/comm-esr52/rev/3fe8be0e3a645eb17b5e4ac6902677014102706d which will restore the 2.49.4 state for extensions.

This is all fixed in 2.53 and up where you can turn them on and off in the mozconfig with enable directives.

Fine.

Then bug #1512882 comment #46 still remains for me (ie. build with system nss).

Then bug #1512882 comment #46 still remains for me (ie. build with system nss).

Some time ago backported and so fixed in 2.53 too. When 2.49.5 is released we hope to do a 2.53 beta soon. We do not plan to fix anything else in 2.49.x.

Blocks: 482143

The journey of esr52 has come to an end unless a major bug is discovered soon. Thanks to all involved and see you in 2.53.

Status: NEW → RESOLVED
Closed: 11 months ago
Resolution: --- → FIXED
Blocks: 1329379
You need to log in before you can comment on or make changes to this bug.