search

skvadrik / re2c

Lexer generator for C, C++, Go and Rust.
https://re2c.org
Other
1.06k stars 169 forks source link

.github/workflows/ci-cmake.yml: avoid `pwd` and use `${{ github.works… #468

Closed trofi closed 7 months ago

trofi commented 7 months ago

…pace }}`

It loks like at least in windows actions/upload-artifact@v4 CI step fails to collect files at ${{ steps.build-info.outputs.BUILD_DIR }}/test_[0-9]*

My theory is that glob is not able to match paths like

`D:/a/re2c/re2c/install-2`

against theit unix-like form of

`/d/a/re2c/re2c/install-2`

I suspect that pwd returns us the second form while ${{ github.workspace }} might keep an original form.

skvadrik commented 7 months ago

https://github.com/skvadrik/re2c/actions/runs/7732301081/job/21081937426?pr=468#step:19:26 confirms your theory. :)

trofi commented 7 months ago

Added and removed bad commit. Should be mergeable now.