925326 - move addition of -Wno-uninitialized in bindings test Makefile to be effective

Status: RESOLVED WONTFIX

(Core :: DOM: Core & HTML, defect)

Comment 1

[Nathan Froyd [:froydnj]]

Attachment: move addition of -Wno-uninitialized in bindings test Makefile to be effective

At some point, rules.mk started modifying CXXFLAGS.  The modification of CXXFLAGS,
then, needs to be moved after the inclusion of rules.mk.  The main bindings
Makefile already had this done.

Comment 2

[Gregory Szorc [:gps]]

Comment on attachment 815344 [details] [diff] [review]
move addition of -Wno-uninitialized in bindings test Makefile to be effective

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

Umm, no. CXXFLAGS should be modified *before* rules.mk. This patch probably works out of sheer luck. Perhaps you need to re-add config.mk to this Makefile.in?

Comment 3

[Nathan Froyd [:froydnj]]

(In reply to Gregory Szorc [:gps] from comment #2)
> Umm, no. CXXFLAGS should be modified *before* rules.mk. This patch probably
> works out of sheer luck. Perhaps you need to re-add config.mk to this
> Makefile.in?

Ah, I see, rules.mk includes config.mk if it's not already included.  Maybe we should just make rules.mk complain in such cases, and make whatever magic generates topsrcdir and the like include config.mk too?

I'll fixup the patch.

Comment 4

[Chris Peterson [:cpeterson]]

This change is no longer necessary because the tree is now -Wuninitialized clean (bug 1001975), including dom/bindings. I am removing these -Wno-uninitialized CXXFLAGS in bug 1240592.