Closed Bug 1043351 Opened 6 years ago Closed 6 years ago
SSEConfig is not built
I obviously missed that in bug 894941, which didn't add it back in CPP_UNIT_TESTS.
Forgot to move TestZip.cpp.
And with a path correction
Sorry for the noise. I just figured the condition could also be lifted, because of the new location.
The funny thing is that this doesn't build without bug 1043344. Bug interdependency ftw.
And here's a version that doesn't depend on bug 1043344.
This one is good. Like previous, but with another TEST_TOOL_DIRS. https://tbpl.mozilla.org/?tree=Try&rev=4524a2930a7f
Attachment #8461614 - Flags: review?(mshal)
Comment on attachment 8461614 [details] [diff] [review] Build ShowSSEConfig as a C++ unit test after it was forgotten in bug 894941 and move linker tests to avoid conflicts Looks fine to me, and this one seems to build locally for me :)
Attachment #8461614 - Flags: review?(mshal) → review+
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
QA Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.