Closed
Bug 920637
Opened 12 years ago
Closed 12 years ago
Nightly OS X xulrunner builds broken (OSError: [Errno 2] No such file or directory: '../../dist/universal/xulrunner/XUL.framework/Versions/Current')
Categories
(Firefox Build System :: General, defect, P1)
Firefox Build System
General
Tracking
(firefox27+ fixed)
RESOLVED
FIXED
mozilla27
People
(Reporter: gps, Assigned: gps)
References
Details
Attachments
(2 files, 1 obsolete file)
1.61 KB,
patch
|
mshal
:
review+
|
Details | Diff | Splinter Review |
1.59 KB,
patch
|
glandium
:
review+
|
Details | Diff | Splinter Review |
https://tbpl.mozilla.org/php/getParsedLog.php?id=28354687&tree=Mozilla-Central
Traceback (most recent call last):
File "/builds/slave/m-cen-osx64-xr-ntly-0000000000/build/toolkit/mozapps/installer/packager.py", line 375, in <module>
main()
File "/builds/slave/m-cen-osx64-xr-ntly-0000000000/build/toolkit/mozapps/installer/packager.py", line 369, in main
copier.copy(args.destination)
File "/builds/slave/m-cen-osx64-xr-ntly-0000000000/build/python/mozbuild/mozpack/copier.py", line 212, in copy
os.mkdir(d)
OSError: [Errno 2] No such file or directory: '../../dist/universal/xulrunner/XUL.framework/Versions/Current'
make[3]: *** [stage-package] Error 1
make[2]: *** [postflight_all] Error 2
make[1]: *** [realbuild] Error 2
make: *** [build] Error 2
This is fallout from bug 918392 changing how directories are created. The code now assumes the parent directory exists.
Assignee | ||
Updated•12 years ago
|
Priority: -- → P1
Comment 1•12 years ago
|
||
I've been assuming that the Android single-locale repacks like https://tbpl.mozilla.org/php/getParsedLog.php?id=28561264&tree=Mozilla-Central are busted from the same thing, am I actually right?
Summary: Nightly OS X xulrunner builds broken (OSError: [Errno 2] No such file or directory: '../../dist/universal/xulrunner/XUL.framework/Versions/Current') → Nightly OS X xulrunner builds and Android l10n repacks broken (OSError: [Errno 2] No such file or directory: '../../dist/universal/xulrunner/XUL.framework/Versions/Current')
Comment 2•12 years ago
|
||
(In reply to Phil Ringnalda (:philor) from comment #1)
> I've been assuming that the Android single-locale repacks like
> https://tbpl.mozilla.org/php/getParsedLog.php?id=28561264&tree=Mozilla-
> Central are busted from the same thing, am I actually right?
It's likely not the same thing.
Comment 3•12 years ago
|
||
(In reply to Mike Hommey [:glandium] from comment #2)
> It's likely not the same thing.
Actually, it probably is. Too early to think straight.
Updated•12 years ago
|
tracking-firefox27:
--- → ?
Assignee | ||
Comment 4•12 years ago
|
||
FileCopier now requires the caller to create the destination directory.
Attachment #813270 -
Flags: review?(mshal)
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → gps
Comment 5•12 years ago
|
||
Comment on attachment 813270 [details] [diff] [review]
Ensure destination directory exists in packager
Looks good, though I wonder if we will need to move the directory creation to before the '# shlibsign libraries' block since args.destination is referenced there:
copier.add(libbase + '.chk',
LibSignFile(os.path.join(args.destination,
libname)))
Attachment #813270 -
Flags: review?(mshal) → review+
Comment 6•12 years ago
|
||
Comment on attachment 813270 [details] [diff] [review]
Ensure destination directory exists in packager
Review of attachment 813270 [details] [diff] [review]:
-----------------------------------------------------------------
::: toolkit/mozapps/installer/packager.py
@@ +370,5 @@
> + try:
> + os.makedirs(args.destination)
> + except OSError as e:
> + if e.errno != errno.EEXIST:
> + raise
Why not do it in the copier?
Assignee | ||
Comment 7•12 years ago
|
||
Yeah, doing it in FileCopier is better. I thought there was some API
change that said the caller was responsible. I must be thinking of an
earlier revision or something else.
Attachment #813314 -
Flags: review?(mh+mozilla)
Comment 8•12 years ago
|
||
Comment on attachment 813314 [details] [diff] [review]
FileCopier should create destination directory if it doesn't exist
Review of attachment 813314 [details] [diff] [review]:
-----------------------------------------------------------------
::: python/mozbuild/mozpack/copier.py
@@ +169,5 @@
> have_symlinks = hasattr(os, 'symlink')
> destination = os.path.normpath(destination)
>
> + try:
> + os.makedirs(destination)
I'd rather you do that in the except for the mkdir further below.
Attachment #813314 -
Flags: review?(mh+mozilla) → review-
Assignee | ||
Comment 9•12 years ago
|
||
The mkdir() loop below is actually quite scary because it removes
symlinks and adjusts file permissions. Performing these actions parents
of destination is scary and would complicate that loop's logic. I think
this implementation is cleanest. I've added a comment to this effect.
Attachment #813451 -
Flags: review?(mh+mozilla)
Assignee | ||
Updated•12 years ago
|
Attachment #813314 -
Attachment is obsolete: true
Updated•12 years ago
|
Attachment #813451 -
Flags: review?(mh+mozilla) → review+
Assignee | ||
Comment 10•12 years ago
|
||
Status: NEW → ASSIGNED
Flags: in-testsuite-
Comment 11•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
Comment 12•12 years ago
|
||
Yeah, turns out that didn't fix the Android l10n repacks, bug 923950.
Summary: Nightly OS X xulrunner builds and Android l10n repacks broken (OSError: [Errno 2] No such file or directory: '../../dist/universal/xulrunner/XUL.framework/Versions/Current') → Nightly OS X xulrunner builds broken (OSError: [Errno 2] No such file or directory: '../../dist/universal/xulrunner/XUL.framework/Versions/Current')
Updated•12 years ago
|
status-firefox27:
--- → fixed
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
•