Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Enable gcc -Werror=conversion-null for gcc >= 4.5 for comm-central (port bug 778980)

RESOLVED FIXED in Thunderbird 19.0


MailNews Core
Build Config
5 years ago
5 years ago


(Reporter: aceman, Assigned: aceman)


Thunderbird 19.0
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)



(1 attachment)



5 years ago
Apply this to comm-central if it is useful.

+++ This bug was initially created as a clone of Bug #778980 +++

To make gcc's NULL/nullptr warnings fatal like clang's nullptr errors, we may want to add -Werror=conversion-null to Linux's CFLAGS/CXXFLAGS for gcc >= 4.5. (The -Wconversion-null flag was new in gcc 4.5.)

If a developer overlooks these gcc warnings, then they may not realize their code change will break the OS X's clang build until they push to the tryserver. This is a waste of developer time and tryserver resources.

Comment 1

5 years ago
Created attachment 668800 [details] [diff] [review]

Here is the patch. Archaeopteryx, please try it on a try server (linux only is enough). I did not get any failures, but I compile with many components disabled.


5 years ago
Thunderbird-Try run:
Builds are green

Comment 3

5 years ago
Comment on attachment 668800 [details] [diff] [review]

The build try server was done for Thunderbird only. Can anybody try Seamonkey?
Attachment #668800 - Flags: review?(mbanner)
SeaMonkey Try run:

Comment 5

5 years ago
Thanks for the build.

There is a configure error:
configure: error: Unrecognized extension provided to --enable-extensions: venkman.

That does not seem caused by this patch.
Comment on attachment 668800 [details] [diff] [review]

The first SeaMonkey run I got wrong, but the second should work:
Attachment #668800 - Flags: review?(mbanner) → review+
(and that's r+ assuming that run passes).

Comment 8

5 years ago
Now an "Internal server error" and another run with "abort: connection ended unexpectedly" ???
If you this happens during |hg pull -u|:
That happens often. The more changesets, the more likely. Just try again.
If they are blue, they'll automatically be re-run. The builds finished off red in stage-package, but that's because SM is expecting to package the extensions that I disabled (rather than having them pulled). So the actual compilations did pass. That's about the best we can do with SM on try server.

Comment 11

5 years ago
OK, thanks. So I take it that the build works.

If any breakage appears later please point me to it.
Keywords: checkin-needed
Flags: in-testsuite-
Keywords: checkin-needed
Target Milestone: --- → Thunderbird 19.0

Comment 13

5 years ago
Ryan, any reason you did not close this bug after check-in?
Nope, just forgot. Thanks.
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.