macOS - Fix mozmill to run tests
Categories
(Thunderbird :: Testing Infrastructure, defect)
Tracking
(Not tracked)
People
(Reporter: aleca, Assigned: Fallen)
Details
Attachments
(2 files, 1 obsolete file)
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.14; rv:65.0) Gecko/20100101 Firefox/65.0
Steps to reproduce:
Philipp Kewisch <mozilla@kewis.ch> took care of fixing this bug that was preventing developers on macos to run mozmill-one tests.
Updated•6 years ago
|
Updated•6 years ago
|
Comment 1•6 years ago
|
||
Updated•6 years ago
|
Comment 2•6 years ago
|
||
Philipp also offered an initial version on IRC last night:
23:52:31 - Fallen: fix: https://www.irccloud.com/pastebin/NAKFqfYR/
23:52:52 - Fallen: oh there is already ABS_DIST, might as well use that
23:54:13 - Fallen: better fix: https://www.irccloud.com/pastebin/1bLDHJF7/
I've attached it here and it does NOT break Windows, so maybe we go with this since the "better" fix isn't really better ;-)
Comment 3•6 years ago
|
||
Comment 6•6 years ago
|
||
Assignee | ||
Comment 7•6 years ago
|
||
The second one is better from a code quality POV because we are re-using variables, so maybe it makes sense to just figure out why it breaks windows. I'll need some more lines of logs from Windows to see what exactly it passes to --binary. On mac there was an array with the exact command being executed.
The issue for mac was that the APP_NAME contains spaces, so calling $(abspath) on it fails. Note $(abspath path/to/foo bar/baz) produces /abs/path/to/foo /abs/bar/baz.
Comment 8•6 years ago
|
||
Is that what you need? We can get Aceman to do you a log on Linux.
Assignee | ||
Comment 9•6 years ago
|
||
Hm interesting, so ABS_DIST is not actually absolute!? This should be fixed somewhere upstream then. Lots of hassle for such a small change so I guess go with whatever works and future self will have to deal with it.
Assignee | ||
Updated•6 years ago
|
Updated•6 years ago
|
Comment 10•6 years ago
|
||
OK, I'll get the agreed patch landed attributing it to Philipp, of course.
Comment 11•6 years ago
|
||
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/2e64da7cd108
fix 'make mozmill-one SOLO_TEST=' on Mac. f=aceman,arshad rs=jorgk
Updated•6 years ago
|
Description
•