Open
Bug 1193897
Opened 9 years ago
Updated 2 years ago
upload.py shouldn't need to convert windows->msys paths
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(Not tracked)
NEW
People
(Reporter: mshal, Unassigned)
Details
Per https://bugzilla.mozilla.org/show_bug.cgi?id=1118778#c9 we should investigate and probably remove the WindowsPathToMsysPath function from build/upload.py, as it may no longer be necessary.
Comment 1•9 years ago
|
||
Note the prerequisite would be buildbot not setting the variable with a tilde in it.
Comment 2•9 years ago
|
||
I can't remember exactly what the problem was, but I know that if you have environment variables that represent unixy paths, and you launch Python by way of MSYS bash, MSYS will helpfully translate the paths to local Windows paths for you. Shouldn't be hard to stick some prints in there and test on try.
Reporter | ||
Comment 3•9 years ago
|
||
(In reply to Ted Mielczarek [:ted.mielczarek] from comment #2)
> I can't remember exactly what the problem was, but I know that if you have
> environment variables that represent unixy paths, and you launch Python by
> way of MSYS bash, MSYS will helpfully translate the paths to local Windows
> paths for you. Shouldn't be hard to stick some prints in there and test on
> try.
One odd thing I discovered in bug 1118778 is that 'make upload' does some path conversion, while 'make upload | tee file.txt' doesn't do the path conversion, and needs the WindowsPathToMsysPath function. I guess this has to do with make finding the '|' character and launching the subprocess differently (the construct_command_argv_internal function in make's job.c), though I tried comparing the two with 'make -d' and didn't see anything obvious.
Updated•7 years ago
|
Product: Core → Firefox Build System
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•