mirror of
https://github.com/servo/servo.git
synced 2025-08-04 13:10:20 +01:00
Auto merge of #19947 - o0Ignition0o:mach_run_nightly, r=jdm
Add a --nightly | -n flag to mach run commands for linux First tries to download and extract a specific nightly version to run mach commands against. <!-- Please describe your changes on the following line: --> I chose to split the Pull requests for each platform to avoid submitting a huge one, and to make sure I get the logic right. I'm able to download / extract a nightly version, and I keep nightly versions in the target folder. Windows and Mac OS support will be filed in separate PRs. This is part of step two for #19505 The mentor on the issue is jdm --- <!-- 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 it is part of a ./mach command. <!-- 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/19947) <!-- Reviewable:end -->
This commit is contained in:
commit
9eb417528b
2 changed files with 84 additions and 4 deletions
|
@ -21,6 +21,9 @@ import subprocess
|
||||||
from subprocess import PIPE
|
from subprocess import PIPE
|
||||||
import sys
|
import sys
|
||||||
import tarfile
|
import tarfile
|
||||||
|
from xml.etree.ElementTree import XML
|
||||||
|
from servo.util import download_file
|
||||||
|
import urllib2
|
||||||
|
|
||||||
from mach.registrar import Registrar
|
from mach.registrar import Registrar
|
||||||
import toml
|
import toml
|
||||||
|
@ -29,6 +32,7 @@ from servo.packages import WINDOWS_MSVC as msvc_deps
|
||||||
from servo.util import host_triple
|
from servo.util import host_triple
|
||||||
|
|
||||||
BIN_SUFFIX = ".exe" if sys.platform == "win32" else ""
|
BIN_SUFFIX = ".exe" if sys.platform == "win32" else ""
|
||||||
|
NIGHTLY_REPOSITORY_URL = "https://servo-builds.s3.amazonaws.com/"
|
||||||
|
|
||||||
|
|
||||||
@contextlib.contextmanager
|
@contextlib.contextmanager
|
||||||
|
@ -388,6 +392,77 @@ class CommandBase(object):
|
||||||
" --release" if release else ""))
|
" --release" if release else ""))
|
||||||
sys.exit()
|
sys.exit()
|
||||||
|
|
||||||
|
def get_nightly_binary_path(self, nightly_date):
|
||||||
|
if nightly_date is None:
|
||||||
|
return
|
||||||
|
if not nightly_date:
|
||||||
|
print(
|
||||||
|
"No nightly date has been provided although the --nightly or -n flag has been passed.")
|
||||||
|
sys.exit(1)
|
||||||
|
# Will alow us to fetch the relevant builds from the nightly repository
|
||||||
|
os_prefix = "linux"
|
||||||
|
if is_windows():
|
||||||
|
print("The nightly flag is not supported on windows yet.")
|
||||||
|
sys.exit(1)
|
||||||
|
if is_macosx():
|
||||||
|
print("The nightly flag is not supported on mac yet.")
|
||||||
|
sys.exit(1)
|
||||||
|
nightly_date = nightly_date.strip()
|
||||||
|
# Fetch the filename to download from the build list
|
||||||
|
repository_index = NIGHTLY_REPOSITORY_URL + "?list-type=2&prefix=nightly"
|
||||||
|
req = urllib2.Request(
|
||||||
|
"{}/{}/{}".format(repository_index, os_prefix, nightly_date))
|
||||||
|
try:
|
||||||
|
response = urllib2.urlopen(req).read()
|
||||||
|
tree = XML(response)
|
||||||
|
namespaces = {'ns': tree.tag[1:tree.tag.index('}')]}
|
||||||
|
file_to_download = tree.find('ns:Contents', namespaces).find(
|
||||||
|
'ns:Key', namespaces).text
|
||||||
|
except urllib2.URLError as e:
|
||||||
|
print("Could not fetch the available nightly versions from the repository : {}".format(
|
||||||
|
e.reason))
|
||||||
|
sys.exit(1)
|
||||||
|
except AttributeError as e:
|
||||||
|
print("Could not fetch a nightly version for date {} and platform {}".format(
|
||||||
|
nightly_date, os_prefix))
|
||||||
|
sys.exit(1)
|
||||||
|
|
||||||
|
nightly_target_directory = path.join(self.context.topdir, "target")
|
||||||
|
# Once extracted, the nightly folder name is the tar name without the extension
|
||||||
|
# (eg /foo/bar/baz.tar.gz extracts to /foo/bar/baz)
|
||||||
|
destination_file = path.join(
|
||||||
|
nightly_target_directory, file_to_download)
|
||||||
|
destination_folder = os.path.splitext(destination_file)[0]
|
||||||
|
nightlies_folder = path.join(
|
||||||
|
nightly_target_directory, 'nightly', os_prefix)
|
||||||
|
|
||||||
|
# Make sure the target directory exists
|
||||||
|
if not os.path.isdir(nightlies_folder):
|
||||||
|
print("The nightly folder for the target does not exist yet. Creating {}".format(
|
||||||
|
nightlies_folder))
|
||||||
|
os.makedirs(nightlies_folder)
|
||||||
|
|
||||||
|
# Download the nightly version
|
||||||
|
if os.path.isfile(path.join(nightlies_folder, destination_file)):
|
||||||
|
print("The nightly file {} has already been downloaded.".format(
|
||||||
|
destination_file))
|
||||||
|
else:
|
||||||
|
print("The nightly {} does not exist yet, downloading it.".format(
|
||||||
|
destination_file))
|
||||||
|
download_file(destination_file, NIGHTLY_REPOSITORY_URL +
|
||||||
|
file_to_download, destination_file)
|
||||||
|
|
||||||
|
# Extract the downloaded nightly version
|
||||||
|
if os.path.isdir(destination_folder):
|
||||||
|
print("The nightly file {} has already been extracted.".format(
|
||||||
|
destination_folder))
|
||||||
|
else:
|
||||||
|
print("Extracting to {}...".format(destination_folder))
|
||||||
|
with tarfile.open(os.path.join(nightlies_folder, destination_file), "r") as tar:
|
||||||
|
tar.extractall(destination_folder)
|
||||||
|
|
||||||
|
return path.join(destination_folder, "servo", "servo")
|
||||||
|
|
||||||
def build_env(self, hosts_file_path=None, target=None, is_build=False, geckolib=False, test_unit=False):
|
def build_env(self, hosts_file_path=None, target=None, is_build=False, geckolib=False, test_unit=False):
|
||||||
"""Return an extended environment dictionary."""
|
"""Return an extended environment dictionary."""
|
||||||
env = os.environ.copy()
|
env = os.environ.copy()
|
||||||
|
|
|
@ -58,11 +58,13 @@ class PostBuildCommands(CommandBase):
|
||||||
help='Launch with software rendering')
|
help='Launch with software rendering')
|
||||||
@CommandArgument('--bin', default=None,
|
@CommandArgument('--bin', default=None,
|
||||||
help='Launch with specific binary')
|
help='Launch with specific binary')
|
||||||
|
@CommandArgument('--nightly', '-n', default=None,
|
||||||
|
help='Specify a YYYY-MM-DD nightly build to run')
|
||||||
@CommandArgument(
|
@CommandArgument(
|
||||||
'params', nargs='...',
|
'params', nargs='...',
|
||||||
help="Command-line arguments to be passed through to Servo")
|
help="Command-line arguments to be passed through to Servo")
|
||||||
def run(self, params, release=False, dev=False, android=None, debug=False, debugger=None,
|
def run(self, params, release=False, dev=False, android=None, debug=False, debugger=None,
|
||||||
headless=False, software=False, bin=None):
|
headless=False, software=False, bin=None, nightly=None):
|
||||||
env = self.build_env()
|
env = self.build_env()
|
||||||
env["RUST_BACKTRACE"] = "1"
|
env["RUST_BACKTRACE"] = "1"
|
||||||
|
|
||||||
|
@ -95,7 +97,7 @@ class PostBuildCommands(CommandBase):
|
||||||
shell.communicate("\n".join(script) + "\n")
|
shell.communicate("\n".join(script) + "\n")
|
||||||
return shell.wait()
|
return shell.wait()
|
||||||
|
|
||||||
args = [bin or self.get_binary_path(release, dev)]
|
args = [bin or self.get_nightly_binary_path(nightly) or self.get_binary_path(release, dev)]
|
||||||
|
|
||||||
if headless:
|
if headless:
|
||||||
set_osmesa_env(args[0], env)
|
set_osmesa_env(args[0], env)
|
||||||
|
@ -160,14 +162,17 @@ class PostBuildCommands(CommandBase):
|
||||||
help='Use dev build')
|
help='Use dev build')
|
||||||
@CommandArgument('--bin', default=None,
|
@CommandArgument('--bin', default=None,
|
||||||
help='Launch with specific binary')
|
help='Launch with specific binary')
|
||||||
|
@CommandArgument('--nightly', '-n', default=None,
|
||||||
|
help='Specify a YYYY-MM-DD nightly build to run')
|
||||||
@CommandArgument(
|
@CommandArgument(
|
||||||
'params', nargs='...',
|
'params', nargs='...',
|
||||||
help="Command-line arguments to be passed through to Servo")
|
help="Command-line arguments to be passed through to Servo")
|
||||||
def rr_record(self, release=False, dev=False, bin=None, params=[]):
|
def rr_record(self, release=False, dev=False, bin=None, nightly=None, params=[]):
|
||||||
env = self.build_env()
|
env = self.build_env()
|
||||||
env["RUST_BACKTRACE"] = "1"
|
env["RUST_BACKTRACE"] = "1"
|
||||||
|
|
||||||
servo_cmd = [bin or self.get_binary_path(release, dev)] + params
|
servo_cmd = [bin or self.get_nightly_binary_path(nightly) or
|
||||||
|
self.get_binary_path(release, dev)] + params
|
||||||
rr_cmd = ['rr', '--fatal-errors', 'record']
|
rr_cmd = ['rr', '--fatal-errors', 'record']
|
||||||
try:
|
try:
|
||||||
check_call(rr_cmd + servo_cmd)
|
check_call(rr_cmd + servo_cmd)
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue