Closed Bug 1343716 Opened 3 years ago Closed 3 years ago
Use Windows style paths for TOOLTOOL
59 bytes, text/x-review-board-request
Currently, tooltool is a python script, but when called from mozharness, it's wrapped with a shell script. What happens is that mozharness gets the msys-style paths, passes them to the shell script, then the shell script calls the python tooltool, and the transition msys->win32 makes that call use windows style paths. For bug 1313111, I want to replace tooltool with a mach command, and the shell script would go away. Calling a mach command (or python tooltool, for that matter) directly from mozharness with a msys-style path doesn't work. OTOH, calling the current shell script with a Windows style path (with forward slashes, for good measure) does work, and is future-proof.
Note that in theory, we should be able to replace all the msys paths in the mozharness configs similarly, but I didn't want to scope bloat my patch queue, or having to figure out what might break or not.
Comment on attachment 8842683 [details] Bug 1343716 - Use Windows style paths for TOOLTOOL_CACHE. https://reviewboard.mozilla.org/r/116448/#review118298
Attachment #8842683 - Flags: review?(mshal) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/482d93aa283a Use Windows style paths for TOOLTOOL_CACHE. r=mshal
You need to log in before you can comment on or make changes to this bug.