aboutsummaryrefslogtreecommitdiffstats
path: root/python/servo
diff options
context:
space:
mode:
authorbors-servo <lbergstrom+bors@mozilla.com>2018-12-22 13:48:50 -0500
committerGitHub <noreply@github.com>2018-12-22 13:48:50 -0500
commit72f64c4b6ea85df8e0beacb41682fd1adbaefa54 (patch)
tree4a487e6f4b3b7f3badd072e027fff1564aab9144 /python/servo
parentcc0e7fde2121bc955ef427b7a398d996496a9fae (diff)
parent10c9128eb2b8c17a66fdf6563129a9f22d81c787 (diff)
downloadservo-72f64c4b6ea85df8e0beacb41682fd1adbaefa54.tar.gz
servo-72f64c4b6ea85df8e0beacb41682fd1adbaefa54.zip
Auto merge of #22387 - o0Ignition0o:mach_run_nightly_macos, r=jdm
./mach run -n on mac os. <!-- Please describe your changes on the following line: --> Add macos support to the -n flag. Followup to #20122 , this PR will add macos support to the -n flag. This is part of step two for #19505 --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [ ] These changes fix #__ (github issue number if applicable). <!-- Either: --> - [ ] There are tests for these changes OR - [X] These changes do not require tests because this mach command does not build anything <!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.--> <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/22387) <!-- Reviewable:end -->
Diffstat (limited to 'python/servo')
-rw-r--r--python/servo/command_base.py72
1 files changed, 55 insertions, 17 deletions
diff --git a/python/servo/command_base.py b/python/servo/command_base.py
index 277f1d3aac1..6c193191fb0 100644
--- a/python/servo/command_base.py
+++ b/python/servo/command_base.py
@@ -9,6 +9,7 @@
from errno import ENOENT as NO_SUCH_FILE_OR_DIRECTORY
from glob import glob
+import shutil
import gzip
import itertools
import locale
@@ -403,6 +404,55 @@ class CommandBase(object):
" --release" if release else ""))
sys.exit()
+ def detach_volume(self, mounted_volume):
+ print("Detaching volume {}".format(mounted_volume))
+ try:
+ subprocess.check_call(['hdiutil', 'detach', mounted_volume])
+ except subprocess.CalledProcessError as e:
+ print("Could not detach volume {} : {}".format(mounted_volume, e.returncode))
+ sys.exit(1)
+
+ def detach_volume_if_attached(self, mounted_volume):
+ if os.path.exists(mounted_volume):
+ self.detach_volume(mounted_volume)
+
+ def mount_dmg(self, dmg_path):
+ print("Mounting dmg {}".format(dmg_path))
+ try:
+ subprocess.check_call(['hdiutil', 'attach', dmg_path])
+ except subprocess.CalledProcessError as e:
+ print("Could not mount Servo dmg : {}".format(e.returncode))
+ sys.exit(1)
+
+ def extract_nightly(self, nightlies_folder, destination_folder, destination_file):
+ print("Extracting to {} ...".format(destination_folder))
+ if is_macosx():
+ mounted_volume = path.join(path.sep, "Volumes", "Servo")
+ self.detach_volume_if_attached(mounted_volume)
+ self.mount_dmg(destination_file)
+ # Servo folder is always this one
+ servo_directory = path.join(path.sep, "Volumes", "Servo", "Servo.app", "Contents", "MacOS")
+ print("Copying files from {} to {}".format(servo_directory, destination_folder))
+ shutil.copytree(servo_directory, destination_folder)
+ self.detach_volume(mounted_volume)
+ else:
+ if is_windows():
+ command = 'msiexec /a {} /qn TARGETDIR={}'.format(
+ os.path.join(nightlies_folder, destination_file), destination_folder)
+ if subprocess.call(command, stdout=PIPE, stderr=PIPE) != 0:
+ print("Could not extract the nightly executable from the msi package.")
+ sys.exit(1)
+ else:
+ with tarfile.open(os.path.join(nightlies_folder, destination_file), "r") as tar:
+ tar.extractall(destination_folder)
+
+ def get_executable(self, destination_folder):
+ if is_windows():
+ return path.join(destination_folder, "PFiles", "Mozilla research", "Servo Tech Demo")
+ if is_linux:
+ return path.join(destination_folder, "servo", "servo")
+ return path.join(destination_folder, "servo")
+
def get_nightly_binary_path(self, nightly_date):
if nightly_date is None:
return
@@ -415,8 +465,7 @@ class CommandBase(object):
if is_windows():
os_prefix = "windows-msvc"
if is_macosx():
- print("The nightly flag is not supported on mac yet.")
- sys.exit(1)
+ os_prefix = "mac"
nightly_date = nightly_date.strip()
# Fetch the filename to download from the build list
repository_index = NIGHTLY_REPOSITORY_URL + "?list-type=2&prefix=nightly"
@@ -467,23 +516,12 @@ class CommandBase(object):
# Extract the downloaded nightly version
if os.path.isdir(destination_folder):
- print("The nightly file {} has already been extracted.".format(
+ print("The nightly folder {} has already been extracted.".format(
destination_folder))
else:
- print("Extracting to {} ...".format(destination_folder))
- if is_windows():
- command = 'msiexec /a {} /qn TARGETDIR={}'.format(
- os.path.join(nightlies_folder, destination_file), destination_folder)
- if subprocess.call(command, stdout=PIPE, stderr=PIPE) != 0:
- print("Could not extract the nightly executable from the msi package.")
- sys.exit(1)
- else:
- with tarfile.open(os.path.join(nightlies_folder, destination_file), "r") as tar:
- tar.extractall(destination_folder)
- bin_folder = path.join(destination_folder, "servo")
- if is_windows():
- bin_folder = path.join(destination_folder, "PFiles", "Mozilla research", "Servo Tech Demo")
- return path.join(bin_folder, "servo{}".format(BIN_SUFFIX))
+ self.extract_nightly(nightlies_folder, destination_folder, destination_file)
+
+ return self.get_executable(destination_folder)
def needs_gstreamer_env(self, target):
try: