Jan Beich Assignee
	Description • 10 years ago

media/webrtc/trunk/webrtc/modules/modules_desktop_capture/Unified_cpp_ules_desktop_capture0.o: In function `webrtc::BlockDifference(unsigned char const*, unsigned char const*, int)':
media/webrtc/trunk/webrtc/modules/desktop_capture/differ_block.cc:47: undefined reference to `webrtc::BlockDifference_SSE2_W32(unsigned char const*, unsigned char const*, int)'
media/webrtc/trunk/webrtc/modules/desktop_capture/differ_block.cc:47: undefined reference to `webrtc::BlockDifference_SSE2_W32(unsigned char const*, unsigned char const*, int)'
/usr/bin/ld: libxul.so: hidden symbol `_ZN6webrtc24BlockDifference_SSE2_W32EPKhS1_i' isn't defined
/usr/bin/ld: final link failed: Bad value

http://mozillaproject.osuosl.org:8010/builders/runtests/builds/1778/steps/shell/logs/stdio

	Jan Beich Assignee
	Comment 1 • 10 years ago

Landry, can you check the patch is enough for powerpc? WebRTC is disabled on sparc (not in whitelist) and I can only test on x86 (i386 and amd64).

	Gian-Carlo Pascutto [:gcp]
	Comment 2 • 10 years ago

Comment on attachment 8458096 [details] [diff] [review]
fix

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

Can you file an upstream issue at webrtc.org with your patch?

	Jan Beich Assignee
	Comment 3 • 10 years ago

No unless upstream migrates to a less privacy-invasive hosting.

new issue -> sign in -> create account -> use current mail -> "Sorry, we cannot serve your request for a new account at this moment." or "Verify your account" without "No, thanks" button

Couple that with CLA signing even for one-off patches. And the codereview tool lacking search for description/commit to find stalled duplicates.

	Jan Beich Assignee
	Comment 4 • 10 years ago

https://tbpl.mozilla.org/?tree=Try&rev=2855a2d75909

	Steve Singer (:stevensn)
	Comment 5 • 10 years ago

This patch allows me to compile on ppc (linux).
(Firefox crashes shortly after startup in  FinalizeArenas but I think that is unrelated to this bug)

	Jan Beich Assignee
	Comment 6 • 10 years ago

Even if related there's still benefit in letting buildbot catch other errors (non-x86, non-SPS, non-ION, etc).

	Randell Jesup [:jesup] (needinfo me)
	Comment 7 • 10 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/9520f63174d0

	Steve Singer (:stevensn)
	Comment 8 • 10 years ago

I've opened Bug 1041307 for the crash I am seeing it is unrelated to this.

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

https://hg.mozilla.org/mozilla-central/rev/9520f63174d0

	Landry Breuil (:gaston)
	Comment 10 • 10 years ago

Will try to build m-i tip on ppc with webrtc enabled now that this issue and 1041307 are fixed.

	Landry Breuil (:gaston)
	Comment 11 • 10 years ago

Comment on attachment 8458096 [details] [diff] [review]
fix

For some reason, m-i tip fails to build for me now on ppc, so i wont be able to comment on the actual issue in this bug for now.....