Closed Bug 509198 Opened 15 years ago Closed 15 years ago

Sisyphus - directly mount dmg installer

Categories

(Testing Graveyard :: Sisyphus, defect)

All
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: bc, Assigned: bc)

Details

Attachments

(1 file)

Attached patch patchSplinter Review
Currently Sisyphus installs Mac OS X dmg builds using hdiutil to attach the file and the expect program to answer any licensing questions. In general this functionality isn't used in practice since Sisyphus is usually used to create its own builds. 

This broke some time ago and is too fragile in general. This patch changes this to directly mount the installer as a subdirectory of /tmp and to copy the files from their to the destination. This is supported on Mac OS X 10.4 and later.

This assumes only one fixed mount point. To generalize to include simultaneous installs it would require generating an unique mount point. I don't think that is necessary for the moment, but can be easily added later if required.
Flags: in-testsuite-
Attachment #393361 - Flags: review?(ctalbert)
Attachment #393361 - Flags: review?(ctalbert) → review+
Comment on attachment 393361 [details] [diff] [review]
patch

Cool, looks good.
http://hg.mozilla.org/tracemonkey/rev/fd2d7f7db7f5
Removing testing/sisyphus/bin/hdiutil-expect.ex;
/cvsroot/mozilla/testing/sisyphus/bin/hdiutil-expect.ex,v  <--  hdiutil-expect.ex
new revision: delete; previous revision: 1.1
done
Checking in testing/sisyphus/bin/install-build.sh;
/cvsroot/mozilla/testing/sisyphus/bin/install-build.sh,v  <--  install-build.sh
new revision: 1.11; previous revision: 1.10
done
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Product: Testing → Testing Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: