Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use yaml for buildbot steps #325

Merged
merged 12 commits into from Apr 20, 2016

Tweak testing steps

Use linux-dev to run all lints because it is the fastest builder; these
are platform-independent.

Ensure that the lockfile_changed.sh and manifest_changed.sh scripts are
run for every builder, since these are platform-dependent. Note: these
should also be added for the Windows builder.

This is also meant as an example of how much easier a YAML file makes
it to edit our build scripts; note the lack of needing to know about
`logfiles` or other parameters which have been forgotten in the past.
  • Loading branch information
aneeshusa committed Apr 20, 2016
commit 9d71e2d6ce6120032d603c31eaa0910af1fa9de6
@@ -122,4 +122,5 @@ def make_step(self, command):
env=envs.build_windows),
steps.Compile(command=["bash", "-l", "-c", "./mach test-unit"],
env=envs.build_windows),
# TODO: run lockfile_changed.sh and manifest_changed.sh scripts
])
@@ -1,5 +1,4 @@
mac-rel-wpt:
- ./mach test-tidy --no-progress
- ./mach build --release
- ./mach test-wpt-failure
- ./mach test-wpt --release --processes 4 --log-raw test-wpt.log --log-errorsummary wpt-errorsummary.log
@@ -32,6 +31,7 @@ linux-dev:
- ./mach build-geckolib
- bash ./etc/ci/lockfile_changed.sh
- bash ./etc/ci/manifest_changed.sh
- bash ./etc/ci/check_no_unwrap.sh

linux-rel:
- ./mach build --release
@@ -42,7 +42,6 @@ linux-rel:
- ./mach build-geckolib --release
- bash ./etc/ci/lockfile_changed.sh
- bash ./etc/ci/manifest_changed.sh
- bash ./etc/ci/check_no_unwrap.sh

android:
- ./mach build --android --dev
@@ -57,6 +56,10 @@ android-nightly:

arm32:
- ./mach build --rel --target=arm-unknown-linux-gnueabihf
- bash ./etc/ci/lockfile_changed.sh
- bash ./etc/ci/manifest_changed.sh

arm64:
- ./mach build --rel --target=aarch64-unknown-linux-gnu
- bash ./etc/ci/lockfile_changed.sh
- bash ./etc/ci/manifest_changed.sh
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.