Skip to content
This repository has been archived by the owner on Aug 4, 2022. It is now read-only.

Commit

Permalink
Bug 1593891 - Allow running periodic-updates that do not trigger a bu…
Browse files Browse the repository at this point in the history
…ild. r=sfraser

When these land for Thunderbird, I set the keyword to prevent a build manually.
periodic_file_updates.sh supports that with "-d", but the runme.sh wrapper
does not have a mechanism to set it.
This patch follows the environment variable precedent used for the other
script parameters.

Differential Revision: https://phabricator.services.mozilla.com/D52024
  • Loading branch information
jfx2006 committed Nov 7, 2019
1 parent b4b9afb commit 7fc7743
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 0 deletions.
3 changes: 3 additions & 0 deletions taskcluster/docker/periodic-updates/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,9 @@ Likewise for `DO_HPKP` and the HPKP checks, and `DO_BLOCKLIST` and the
blocklist checks. Environment variables are used rather than command line
arguments to make constructing taskcluster tasks easier.

To prevent a full build when landing with Phabricator, set the `DONTBUILD`
environment variable.

==Background==

These scripts have been moved from
Expand Down
5 changes: 5 additions & 0 deletions taskcluster/docker/periodic-updates/runme.sh
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,11 @@ then
PARAMS="${PARAMS} --suffix-list"
fi

if [ -n "${DONTBUILD}" ]
then
PARAMS="${PARAMS} -d"
fi


export ARTIFACTS_DIR="/home/worker/artifacts"
mkdir -p "$ARTIFACTS_DIR"
Expand Down

0 comments on commit 7fc7743

Please sign in to comment.