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

git status --porcelain is not working in CI #9539

Closed
berland opened this issue Dec 13, 2024 · 1 comment · Fixed by #9540
Closed

git status --porcelain is not working in CI #9539

berland opened this issue Dec 13, 2024 · 1 comment · Fixed by #9540
Labels

Comments

@berland
Copy link
Contributor

berland commented Dec 13, 2024

There is a github workflow to check that the tests do not pollute the files on disk. This is not working for the moment, as it is failing due to wrong syntax in a sed command:

Run # Run this before the 'Test CLI' entry below, which produces a few files that are accepted for now. Exclude the wheel.
  # Run this before the 'Test CLI' entry below, which produces a few files that are accepted for now. Exclude the wheel.
  git status --porcelain | sed '/ert.*.whl$\|\/block_storage$/d'
  test -z "$(git status --porcelain | sed '/ert.*.whl$\\|\\/block_storage$/d')"
  shell: /usr/bin/bash -e {0}
  env:
    ERT_SHOW_BACKTRACE: 1
    ECL_SKIP_SIGNAL: 1
    UV_SYSTEM_PYTHON: 1
    pythonLocation: /opt/hostedtoolcache/Python/3.11.11/x64
    PKG_CONFIG_PATH: /opt/hostedtoolcache/Python/3.11.11/x64/lib/pkgconfig
    Python_ROOT_DIR: /opt/hostedtoolcache/Python/3.11.11/x64
    Python[2](https://github.com/equinor/ert/actions/runs/12314606504/job/34371076740?pr=9108#step:11:2)_ROOT_DIR: /opt/hostedtoolcache/Python/3.11.11/x64
    Python3_ROOT_DIR: /opt/hostedtoolcache/Python/3.11.11/x64
    LD_LIBRARY_PATH: /opt/hostedtoolcache/Python/[3](https://github.com/equinor/ert/actions/runs/12314606504/job/34371076740?pr=9108#step:11:3).11.11/x64/lib
    UV_CACHE_DIR: /home/runner/work/_temp/setup-uv-cache
?? .coverage
?? case.data
?? cov1.xml
?? cov2.xml
?? junit.xml
sed: can't find label for jump to `lock_storage$/d'
@berland
Copy link
Contributor Author

berland commented Dec 13, 2024

This problem seems to occur between sed version 4.5 (RHEL8: no error) and version 4.7 (on debian 10.13). Ubuntu 22.04 has sed 4.8.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant