-
Notifications
You must be signed in to change notification settings - Fork 242
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 HEAD
] 3 of 37 tests are failing (in Ubuntu 24.04.1 LTS of GitHub Actions)
#343
Comments
HEAD
] 3 of 37 tests are failing (on Ubuntu 24.04 LTS)HEAD
] 3 of 37 tests are failing (in Ubuntu 24.04.1 LTS of GitHub Actions)
@henry2cox it turns out that vanilla Ubuntu 24.04 does not show the same symptoms, so it must be something in the GitHub Actions images that GitHub provides 🤔 Here's the FROM ubuntu:24.04
RUN apt-get update \
&& \
apt-get install --no-install-recommends --yes -V \
libcapture-tiny-perl \
libdatetime-perl \
libdevel-cover-perl \
libdigest-md5-file-perl \
libfile-spec-native-perl \
libjson-xs-perl \
libscalar-list-utils-perl \
llvm \
python3-coverage \
python3-xlsxwriter \
libgd-perl \
git \
make
RUN perl -MCPAN -e 'install(Memory::Process)' # no package in Debian/Ubuntu
RUN ln -s python3-coverage /usr/bin/coverage # until issue #347 is fixed
RUN git clone --depth 1 https://github.com/linux-test-project/lcov
WORKDIR /lcov/
RUN make check |
@henry2cox PS: Could it be the version of GCC used? GitHub Actions seems to have GCC 13.2.0 — do you see anything at https://github.com/actions/runner-images/blob/main/images/ubuntu/Ubuntu2404-Readme.md that already looks like trouble? |
Signed-off-by: Henry Cox <[email protected]>
See linux-test-project#343. Signed-off-by: Henry Cox <[email protected]>
Signed-off-by: Henry Cox <[email protected]>
See linux-test-project#343. Signed-off-by: Henry Cox <[email protected]>
See linux-test-project#343. Signed-off-by: Henry Cox <[email protected]>
See linux-test-project#343. Signed-off-by: Henry Cox <[email protected]>
See linux-test-project#343. Signed-off-by: Henry Cox <[email protected]>
I tried with image |
Thinking aloud: could be related to #341 |
Signed-off-by: Henry Cox <[email protected]>
See #343. Signed-off-by: Henry Cox <[email protected]>
Possibly, I lost the plot...which issue do you think might be version related? I think that the 'simple' case just needs another portability fix (related to one of the shortcuts I took, when I wrote the test). |
@henry2cox none in particular and all of them at the same time. |
Update for the record for anyone else reading and interested: With #353 merged now, the failures are down to 2 and the output is:
|
Signed-off-by: Henry Cox <[email protected]>
Signed-off-by: Henry Cox <[email protected]>
Update: With the latest fixes from #354 this single one remains left:
|
GitHub regression fixes. #343, #348 Signed-off-by: Henry Cox <[email protected]>
@henry2cox the remaining issue seems to have been fixed by 193cc76 . Can this issue be closed as fixed in your eyes? |
Yes...I think that this one and #348 can both be closed - |
@henry2cox excellent! Closing as fixed… |
This is with Ubuntu 24.04 LTS:
The text was updated successfully, but these errors were encountered: