Closed
Bug 1395047
Opened 7 years ago
Closed 7 years ago
--enable-tests for MinGW build is broken because it thinks it's linux
Categories
(Firefox Build System :: General, enhancement)
Firefox Build System
General
Tracking
(firefox58 fixed)
RESOLVED
FIXED
mozilla58
Tracking | Status | |
---|---|---|
firefox58 | --- | fixed |
People
(Reporter: tjr, Assigned: tjr)
References
(Depends on 1 open bug, Blocks 1 open bug)
Details
(Whiteboard: [tor])
Attachments
(1 file)
> WARNING - TEST-UNEXPECTED-FAIL | /home/worker/workspace/build/src/testing/xpcshell/selftest.py | XPCShellTestsTests.testAddTaskRunNextTest, line 1049: No module named fix_linux_stack
I traced this to http://searchfox.org/mozilla-central/source/testing/mozbase/mozrunner/mozrunner/utils.py#284 where it thinks we are Linux, because well we sure look like Linux I bet!
The problem is, I'm not quite sure where in mozinfo we set isLinux, and even if I corrected it, I'm not sure we could run tests successfully in the current way we run tests!
Comment 1•7 years ago
|
||
(In reply to Tom Ritter [:tjr] from comment #0)
> The problem is, I'm not quite sure where in mozinfo we set isLinux, and even
> if I corrected it, I'm not sure we could run tests successfully in the
> current way we run tests!
We set it here:
http://searchfox.org/mozilla-central/source/testing/mozbase/mozinfo/mozinfo/mozinfo.py#255
or thereabouts. At least that selftest check probably needs to be updated to consider the *target* machine we're running on, rather than the host...and there are probably many other instances besides, since 'isLinux' probably worked OK for the other major cross-compilation case of Android on Linux.
Comment 2•7 years ago
|
||
(In reply to Nathan Froyd [:froydnj] from comment #1)
> (In reply to Tom Ritter [:tjr] from comment #0)
> > The problem is, I'm not quite sure where in mozinfo we set isLinux, and even
> > if I corrected it, I'm not sure we could run tests successfully in the
> > current way we run tests!
>
> We set it here:
>
> http://searchfox.org/mozilla-central/source/testing/mozbase/mozinfo/mozinfo/
> mozinfo.py#255
>
> or thereabouts. At least that selftest check probably needs to be updated
> to consider the *target* machine we're running on, rather than the
> host...and there are probably many other instances besides, since 'isLinux'
> probably worked OK for the other major cross-compilation case of Android on
> Linux.
n.b.: we don't (and have never) run `make check` on any of our cross-compiled builds, for this and other reasons.
Comment hidden (mozreview-request) |
Comment 4•7 years ago
|
||
We went through this same rigamarole with the cross-mac builds in bug 1340698.
Comment 5•7 years ago
|
||
mozreview-review |
Comment on attachment 8916374 [details]
Bug 1395047 Don't run 'make check' on a cross compiled build
https://reviewboard.mozilla.org/r/187538/#review192812
::: browser/config/mozconfigs/win32/mingw32:50
(Diff revision 1)
>
> # GCC compiling for Windows exposes a lot of warnings. We are tracking them in Bug 1394433
> ac_add_options --disable-warnings-as-errors
>
> # Temporary config settings until we get these working on mingw
> -ac_add_options --disable-tests
> +ac_add_options --enable-tests
You should be able to drop this, --enable-tests is the default.
Attachment #8916374 -
Flags: review?(ted) → review+
Comment hidden (mozreview-request) |
Assignee | ||
Comment 7•7 years ago
|
||
(In reply to Ted Mielczarek [:ted.mielczarek] from comment #5)
> You should be able to drop this, --enable-tests is the default.
Thanks, fixed. Will land this once I get Bug 1406736 sorted (which is breaking the -tests build).
Comment hidden (mozreview-request) |
Assignee | ||
Updated•7 years ago
|
Keywords: checkin-needed
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/7ce72fc1ce81
Don't run 'make check' on a cross compiled build r=ted
Keywords: checkin-needed
Comment 10•7 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox58:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
Updated•7 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•