Branch: refs/heads/main
Revision: 1f5bc3b
Author: aconsroe-hpe
Link: Update CI container: add tzdata and locales by aconsroe-hpe · Pull Request #19475 · chapel-lang/chapel · GitHub
Log Message:
Merge pull request #19475 from aconsroe-hpe/fix/docker-ci-update
Update CI container: add tzdata and locales
reviewed by @ronawho
In preparing a container to use with CI.yml, I identified two
additional packages we need for testing: tzdata and locales.
tzdata is used to get the correct merge date for perf annotations
locales is necessary to avoid an error when testing the docs:
[Working on file chpldoc.doc.chpl]
...
[Executing diff chpldoc.doc.good chpldoc.doc.1.comp.out.tmp]
0a1,9
> perl: warning: Setting locale failed.
> perl: warning: Please check that your locale settings:
> LANGUAGE = (unset),
> LC_ALL = (unset),
> LC_CTYPE = "C.UTF-8",
> LC_COLLATE = "C",
> LANG = "en_US.UTF-8"
> are supported and installed on your system.
> perl: warning: Falling back to the standard locale ("C").
[Error matching compiler output for $CHPL_HOME/chapel/chapel-test-MH818I/chpldoc.doc]
LANG gets set by sub_test.py and perl prints that warning without locales,
the perl invocation comes from timedexec
This needs to get merged so the updated container image is available
before changing CI.yml to actually use the container
Tested in a fork to confirm CI.yml tests pass using this container
Modified Files:
M .github/workflows/build-CI-container.yml
M util/dockerfiles/github-ci/Dockerfile
Compare: https://github.com/chapel-lang/chapel/compare/1ceffcfb7f2e...1f5bc3b5bdc7