Skip to content

Commit

Permalink
Bug 1460402 - Update the CI to use codespell with pip instead of the …
Browse files Browse the repository at this point in the history
…apt packages (too old) r=ahal

MozReview-Commit-ID: 9QkTPyP7izS
  • Loading branch information
sylvestre committed May 9, 2018
1 parent 6d03a45 commit e487432
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 2 deletions.
2 changes: 1 addition & 1 deletion build/sparse-profiles/docker-image
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,8 @@ path:taskcluster/
path:python/mozbuild/mozbuild/action/tooltool.py
path:testing/config/tooltool-manifests/linux64/releng.manifest
path:testing/mozharness/external_tools/robustcheckout.py
path:tools/lint/spell/codespell_requirements.txt
path:tools/lint/eslint/eslint-plugin-mozilla/manifest.tt
path:tools/lint/eslint/manifest.tt
path:tools/lint/python/flake8_requirements.txt
path:tools/lint/tox/tox_requirements.txt

2 changes: 2 additions & 0 deletions taskcluster/docker/lint/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,8 @@ ADD topsrcdir/tools/lint/eslint/manifest.tt /tmp/eslint.tt
ADD topsrcdir/tools/lint/eslint/eslint-plugin-mozilla/manifest.tt /tmp/eslint-plugin-mozilla.tt
# %include tools/lint/python/flake8_requirements.txt
ADD topsrcdir/tools/lint/python/flake8_requirements.txt /tmp/flake8_requirements.txt
# %include tools/lint/spell/codespell_requirements.txt
ADD topsrcdir/tools/lint/spell/codespell_requirements.txt /tmp/codespell_requirements.txt
# %include tools/lint/tox/tox_requirements.txt
ADD topsrcdir/tools/lint/tox/tox_requirements.txt /tmp/tox_requirements.txt
RUN bash /tmp/system-setup.sh
Expand Down
9 changes: 8 additions & 1 deletion taskcluster/docker/lint/system-setup.sh
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ mkdir -p /setup
cd /setup

apt_packages=()
apt_packages+=('codespell')
apt_packages+=('curl')
apt_packages+=('locales')
apt_packages+=('git')
Expand Down Expand Up @@ -88,6 +87,14 @@ cd /setup

pip install --require-hashes -r /tmp/flake8_requirements.txt

###
# codespell Setup
###

cd /setup

pip install --require-hashes -r /tmp/codespell_requirements.txt

###
# tox Setup
###
Expand Down
2 changes: 2 additions & 0 deletions taskcluster/docker/update-verify/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,8 @@ ADD topsrcdir/tools/lint/eslint/manifest.tt /tmp/eslint.tt
ADD topsrcdir/tools/lint/eslint/eslint-plugin-mozilla/manifest.tt /tmp/eslint-plugin-mozilla.tt
# %include tools/lint/python/flake8_requirements.txt
ADD topsrcdir/tools/lint/python/flake8_requirements.txt /tmp/flake8_requirements.txt
# %include tools/lint/spell/codespell_requirements.txt
ADD topsrcdir/tools/lint/spell/codespell_requirements.txt /tmp/codespell_requirements.txt
# %include tools/lint/tox/tox_requirements.txt
ADD topsrcdir/tools/lint/tox/tox_requirements.txt /tmp/tox_requirements.txt
RUN bash /tmp/system-setup.sh
Expand Down

0 comments on commit e487432

Please sign in to comment.