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

CI: Add Cygwin CMake run #2930

Merged
merged 15 commits into from
Jul 1, 2024
Merged

CI: Add Cygwin CMake run #2930

merged 15 commits into from
Jul 1, 2024

Conversation

DWesl
Copy link
Contributor

@DWesl DWesl commented May 24, 2024

Largely copied from MehdiChinoune's MSYS CMake CI run (#2917).

Doesn't enable DAP, and skips nc_test4_tst_filter due to a failure I don't understand and can't reproduce locally.

Largely in line with the Autotools build otherwise, except the skipped test works under Autotools (it's marked XFAIL for MINGW there).

DWesl added 13 commits May 9, 2024 11:14
Copied from MehdiChinoune's MSYS CMake CI run.
Why isn't CYGWIN_NOWINPATH working?
This should really not be needed
That's the one that needed it.
Let's see if the build works without these.  The export-everything is going to be the real test.

LDFLAGS="-Wl,--export-all-symbols"
MAKE=/usr/bin/make
Already skipped on MinGW, so I don't feel bad skipping it on Cygwin too.  Still haven't figured out why it's crashing.  I should probably look into why Autotools is fine.
I don't know that it ever worked before, and it isn't working now.
@DWesl
Copy link
Contributor Author

DWesl commented Jun 12, 2024

I suspect the DAP failure in libcurl on Linux is unrelated.

@WardF
Copy link
Member

WardF commented Jun 21, 2024

Still trying to figure out what's going on here, as the DAP issue (whether related or not) is not occurring on other branches.

@DWesl
Copy link
Contributor Author

DWesl commented Jun 22, 2024

The Ubuntu failure looks like a network issue, and the MacOS failures passed on the re-run, so I'm not sure what's going on there. On the other hand, the DAP/libcurl issue seems to have gone away.

@WardF
Copy link
Member

WardF commented Jun 28, 2024

Yeah. This is all very puzzling and not exactly a confidence builder in Github Actions.

@WardF WardF merged commit d694d38 into Unidata:main Jul 1, 2024
111 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants