Mike Hommey [:glandium] Assignee
	Description • 10 years ago

While -Wl,--as-needed is not our default, I've been using it in Debian packages for a while, and it works well. I plan to make that the default on our builds too, but I need to ensure this works on android and b2g before that, and I want this fix in the tree independently.

	Mike Hommey [:glandium] Assignee
	Comment 2 • 10 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/de5c523c5ed4

	Ed Morley [:emorley]
	Comment 3 • 10 years ago

https://hg.mozilla.org/mozilla-central/rev/de5c523c5ed4

	Ryan VanderMeulen [:RyanVM]
	Comment 4 • 10 years ago

https://hg.mozilla.org/releases/mozilla-aurora/rev/730801cf5af4

	Nathan Froyd [:froydnj]
	Comment 5 • 10 years ago

Comment on attachment 8436579 [details] [diff] [review]
Force mozgtk stubs dependency on gtk libraries when -Wl,--as-needed is used

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

::: widget/gtk/mozgtk/gtk2/Makefile.in
@@ +9,5 @@
> +# If LDFLAGS contains -Wl,--as-needed, we need to add -Wl,--no-as-needed
> +# before the gtk libraries, otherwise the linker will drop those dependencies
> +# because no symbols are used from them. But those dependencies need to be
> +# kept for things to work properly.
> +ifeq (,$(filter -Wl,--as-needed),$(LDFLAGS))

Um, how does this (and its gtk3 counterpart) work?  Notice that the parenthesization of the |filter| call is wrong, and therefore I think we always take the true branch here...

	Mike Hommey [:glandium] Assignee
	Comment 6 • 10 years ago

(In reply to Nathan Froyd (:froydnj) from comment #5)
> Um, how does this (and its gtk3 counterpart) work?  Notice that the
> parenthesization of the |filter| call is wrong, and therefore I think we
> always take the true branch here...

In fact, we always take the false branch[1]. Which is why I didn't notice it didn't work as expected. Please feel free to fixup.

(1) it ultimately expands to ifeq (,,$(LDFLAGS)), which, even if LDFLAGS is empty, is ifeq (,,), which a) make is happy not to consider a syntax error, and b) apparently means "if '' equals ','.

	Nathan Froyd [:froydnj]
	Comment 7 • 10 years ago

Comment on attachment 8436579 [details] [diff] [review]
Force mozgtk stubs dependency on gtk libraries when -Wl,--as-needed is used

(In reply to Mike Hommey [:glandium] from comment #6)
> (In reply to Nathan Froyd (:froydnj) from comment #5)
> > Um, how does this (and its gtk3 counterpart) work?  Notice that the
> > parenthesization of the |filter| call is wrong, and therefore I think we
> > always take the true branch here...
> 
> In fact, we always take the false branch[1]. Which is why I didn't notice it
> didn't work as expected. Please feel free to fixup.
> 
> (1) it ultimately expands to ifeq (,,$(LDFLAGS)), which, even if LDFLAGS is
> empty, is ifeq (,,), which a) make is happy not to consider a syntax error,
> and b) apparently means "if '' equals ','.

I can just imagine the fun of trying to ensure the comma in -Wl,--as-needed is parsed as part of the argument and not as part of the make syntax.  One more thing that would be easier in moz.build.

	Mike Hommey [:glandium] Assignee
	Comment 8 • 10 years ago

(In reply to Nathan Froyd (:froydnj) from comment #7)
> Comment on attachment 8436579 [details] [diff] [review]
> Force mozgtk stubs dependency on gtk libraries when -Wl,--as-needed is used
> 
> (In reply to Mike Hommey [:glandium] from comment #6)
> > (In reply to Nathan Froyd (:froydnj) from comment #5)
> > > Um, how does this (and its gtk3 counterpart) work?  Notice that the
> > > parenthesization of the |filter| call is wrong, and therefore I think we
> > > always take the true branch here...
> > 
> > In fact, we always take the false branch[1]. Which is why I didn't notice it
> > didn't work as expected. Please feel free to fixup.
> > 
> > (1) it ultimately expands to ifeq (,,$(LDFLAGS)), which, even if LDFLAGS is
> > empty, is ifeq (,,), which a) make is happy not to consider a syntax error,
> > and b) apparently means "if '' equals ','.
> 
> I can just imagine the fun of trying to ensure the comma in -Wl,--as-needed
> is parsed as part of the argument and not as part of the make syntax.

$(COMMA)