Skip to content

Commit

Permalink
Print the full path for errors occurring in the servo crate
Browse files Browse the repository at this point in the history
  • Loading branch information
tigercosmos committed Nov 3, 2017
1 parent c494d25 commit a0ae816
Show file tree
Hide file tree
Showing 7 changed files with 29 additions and 19 deletions.
8 changes: 4 additions & 4 deletions ports/servo/.cargo/config → .cargo/config
@@ -1,13 +1,13 @@
[target.arm-linux-androideabi]
linker = "./fake-ld-arm.sh"
linker = "./ports/servo/fake-ld-arm.sh"
ar = "arm-linux-androideabi-ar"

[target.armv7-linux-androideabi]
linker = "./fake-ld-armv7.sh"
linker = "./ports/servo/fake-ld-armv7.sh"
ar = "arm-linux-androideabi-ar"

[target.aarch64-linux-android]
linker = "./fake-ld-arm64.sh"
linker = "./ports/servo/fake-ld-arm64.sh"
ar = "aarch64-linux-android-ar"

[target.arm-unknown-linux-gnueabihf]
Expand All @@ -19,4 +19,4 @@ linker = "aarch64-linux-gnu-gcc"
ar = "aarch64-linux-gnu-ar"

[target.'cfg(target_os=windows)']
linker = "./fake-ld.cmd"
linker = "./ports/servo/fake-ld.cmd"
3 changes: 2 additions & 1 deletion .gitignore
@@ -1,5 +1,6 @@
/.servo
/.cargo
/.cargo/*
!/.cargo/config.*
/.servobuild
/target
/ports/android/bin
Expand Down
2 changes: 1 addition & 1 deletion ports/servo/fake-ld-arm.sh
Expand Up @@ -8,7 +8,7 @@ set -o errexit
set -o nounset
set -o pipefail

source fake-ld.sh
source ./ports/servo/fake-ld.sh

export _GCC_PARAMS="${@}"
call_gcc "arch-arm" "arm-linux-androideabi" "android-18" "armeabi"
2 changes: 1 addition & 1 deletion ports/servo/fake-ld-arm64.sh
Expand Up @@ -8,7 +8,7 @@ set -o errexit
set -o nounset
set -o pipefail

source fake-ld.sh
source ./ports/servo/fake-ld.sh

export _GCC_PARAMS="${@}"
call_gcc "arch-arm64" "aarch64-linux-android" "android-21" "arm64-v8a"
2 changes: 1 addition & 1 deletion ports/servo/fake-ld-armv7.sh
Expand Up @@ -8,7 +8,7 @@ set -o errexit
set -o nounset
set -o pipefail

source fake-ld.sh
source ./ports/servo/fake-ld.sh

export _GCC_PARAMS="${@}"
call_gcc "arch-arm" "arm-linux-androideabi" "android-18" "armeabi-v7a"
22 changes: 11 additions & 11 deletions python/servo/build_commands.py
Expand Up @@ -176,12 +176,14 @@ class MachCommands(CommandBase):
def build(self, target=None, release=False, dev=False, jobs=None,
features=None, android=None, verbose=False, debug_mozjs=False, params=None,
with_debug_assertions=False):

opts = params or []
opts += ["--manifest-path", self.servo_manifest()]

if android is None:
android = self.config["build"]["android"]
features = features or self.servo_features()

opts = params or []

base_path = self.get_target_dir()
release_path = path.join(base_path, "release", "servo")
dev_path = path.join(base_path, "debug", "servo")
Expand Down Expand Up @@ -318,8 +320,7 @@ def build(self, target=None, release=False, dev=False, jobs=None,
cargo_binary = "cargo" + BIN_SUFFIX

status = call(
[cargo_binary, "build"] + opts,
env=env, cwd=self.servo_crate(), verbose=verbose)
[cargo_binary, "build"] + opts, env=env, verbose=verbose)
elapsed = time() - build_start

# Do some additional things if the build succeeded
Expand Down Expand Up @@ -384,6 +385,8 @@ def build_cef(self, jobs=None, verbose=False, release=False,

ret = None
opts = []
opts += ["--manifest-path", self.cef_manifest()]

if jobs is not None:
opts += ["-j", jobs]
if verbose:
Expand All @@ -407,10 +410,7 @@ def build_cef(self, jobs=None, verbose=False, release=False,
# common dependencies with the same flags.
opts += ["--", "-C", "link-args=-Xlinker -undefined -Xlinker dynamic_lookup"]

with cd(path.join("ports", "cef")):
ret = call(["cargo", "rustc"] + opts,
env=env,
verbose=verbose)
ret = call(["cargo", "rustc"] + opts, env=env, verbose=verbose)
elapsed = time() - build_start

# Generate Desktop Notification if elapsed-time > some threshold value
Expand Down Expand Up @@ -441,20 +441,20 @@ def build_geckolib(self, jobs=None, verbose=False, release=False):

ret = None
opts = []
opts += ["--manifest-path", self.geckolib_manifest()]
features = []

if jobs is not None:
opts += ["-j", jobs]
if verbose:
opts += ["-v"]
if release:
opts += ["--release"]

if features:
opts += ["--features", ' '.join(features)]

build_start = time()
with cd(path.join("ports", "geckolib")):
ret = call(["cargo", "build"] + opts, env=env, verbose=verbose)
ret = call(["cargo", "build"] + opts, env=env, verbose=verbose)
elapsed = time() - build_start

# Generate Desktop Notification if elapsed-time > some threshold value
Expand Down
9 changes: 9 additions & 0 deletions python/servo/command_base.py
Expand Up @@ -545,6 +545,15 @@ def package_dir(package):
def servo_crate(self):
return path.join(self.context.topdir, "ports", "servo")

def servo_manifest(self):
return path.join(self.context.topdir, "ports", "servo", "Cargo.toml")

def geckolib_manifest(self):
return path.join(self.context.topdir, "ports", "geckolib", "Cargo.toml")

def cef_manifest(self):
return path.join(self.context.topdir, "ports", "cef", "Cargo.toml")

def servo_features(self):
"""Return a list of optional features to enable for the Servo crate"""
features = []
Expand Down

0 comments on commit a0ae816

Please sign in to comment.