FS#63605 - [seamonkey] doesn't build with gcc 9

Attached to Project: Community Packages
Opened by Andreas Baumann (andreas_baumann) - Friday, 30 August 2019, 12:08 GMT
Last edited by Kyle Keen (keenerd) - Thursday, 12 September 2019, 01:04 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Kyle Keen (keenerd)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:

Warnings/errors like:

/build/seamonkey/src/seamonkey-2.49.4/mozilla/js/xpconnect/src/XPCWrappedNative.cpp:
2221:41: error: directive argument is null [-Werror=format-overflow=]
2221 | name = JS_sprintf_append(name, fmt,

| ~~~~~~~~~~~~~~~~~^~~~~~~~~~~
2222 | array[i]->GetNameString());
| ~~~~~~~~~~~~~~~~~~~~~~~~~~

Following the advice in:

https://bugzilla.mozilla.org/show_bug.cgi?id=1514781

and added a section of flags after the GCC 6 flags:

# GCC 9
CFLAGS+=" -Wno-error=class-memaccess -Wno-error=format-overflow -Wno-error=stringo
p-truncation -Wno-stringop-overflow -Wno-error=type-limits -Wno-error=deprecated"
CXXFLAGS+=" -Wno-error=class-memaccess -Wno-error=format-overflow -Wno-error=strin
gop-truncation -Wno-stringop-overflow -Wno-error=attributes -Wno-error=deprecated-co
py -Wno-error=pessimizing-move -Wno-error=type-limits -Wno-error=deprecated"

makes it compile. I keep you posted whether it also runs.. :-)

Additional info:
* package version(s): 2.49.4-4

Steps to reproduce:
asp export seamonkey
cd seamonkey
staging-i686-build
This task depends upon

Closed by  Kyle Keen (keenerd)
Thursday, 12 September 2019, 01:04 GMT
Reason for closing:  Upstream
Comment by Andreas Baumann (andreas_baumann) - Friday, 30 August 2019, 12:41 GMT
/build/seamonkey/src/seamonkey-2.49.4/mozilla/js/xpconnect/src/XPCWrappedNative.cpp:
2221:41: error: â directive argument is null [-Werror=format-overflow=]
2221 | name = JS_sprintf_append(name, fmt,
| ~~~~~~~~~~~~~~~~~^~~~~~~~~~~
2222 | array[i]->GetNameString());
| ~~~~~~~~~~~~~~~~~~~~~~~~~~

Nope, needs proper fixing..
Comment by Andreas Baumann (andreas_baumann) - Friday, 30 August 2019, 13:02 GMT
The fix is actually trivial:

name = JS_sprintf_append(name, fmt,
(array[i]->GetNameString() != NULL) ? array[i]->GetNameString() : "(NULL)");

Comment by Andreas Baumann (andreas_baumann) - Friday, 30 August 2019, 13:20 GMT Comment by Andreas Baumann (andreas_baumann) - Friday, 06 September 2019, 08:36 GMT
Reported as https://bugzilla.mozilla.org/show_bug.cgi?id=1579360

But newer versions of seamonkey exist (which made have already fixed it, didn't check).
Comment by Kyle Keen (keenerd) - Thursday, 12 September 2019, 01:04 GMT
Getting the update to build has been fun. It is correct that this fix is applied, but there are new issues with linking our version of Lua, and possibly python2/3 issues in some of their build scripts. If you write a working PKGBUILD before the update is released, please email me.

Loading...