Page MenuHomeSoftware Heritage

sphinx/Dockerfile: Actually update PATH so pre-commit can be used
ClosedPublic

Authored by ardumont on Nov 27 2019, 10:59 AM.

Details

Summary

pre-commit is not seen right now

As of the first install said to me the first time around [1] but i did not fully realize:

10:22:02 Successfully built pyyaml nodeenv
10:22:02 Installing collected packages: more-itertools, zipp, importlib-metadata, pyyaml, aspy.yaml, virtualenv, identify, nodeenv, cfgv, pre-commit
10:22:02   The script virtualenv is installed in '/home/jenkins/.local/bin' which is not on PATH.
10:22:02   Consider adding this directory to PATH or, if you prefer to suppress this warning, use --no-warn-script-location.
10:22:02   The script identify-cli is installed in '/home/jenkins/.local/bin' which is not on PATH.
10:22:02   Consider adding this directory to PATH or, if you prefer to suppress this warning, use --no-warn-script-location.
10:22:02   The script nodeenv is installed in '/home/jenkins/.local/bin' which is not on PATH.
10:22:02   Consider adding this directory to PATH or, if you prefer to suppress this warning, use --no-warn-script-location.
10:22:02   The scripts pre-commit, pre-commit-validate-config and pre-commit-validate-manifest are installed in '/home/jenkins/.local/bin' which is not on PATH.
10:22:02   Consider adding this directory to PATH or, if you prefer to suppress this warning, use --no-warn-script-location.

[1] https://jenkins.softwareheritage.org/view/all/job/jenkins-tools/job/swh-jenkins-dockerfiles/42/console

Diff Detail

Repository
rCDFJ Dockerfiles for Jenkins
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

ardumont created this revision.Nov 27 2019, 10:59 AM
ardumont edited the summary of this revision. (Show Details)Nov 27 2019, 11:00 AM
anlambert accepted this revision.Nov 27 2019, 11:02 AM
This revision is now accepted and ready to land.Nov 27 2019, 11:02 AM
ardumont updated this revision to Diff 8319.Nov 27 2019, 11:07 AM

Actually update the path in base-buster