Skip to content

CI: compile OSGeo4W-Windows build with clang#2811

Closed
nilason wants to merge 1 commit intoOSGeo:mainfrom
nilason:test_win
Closed

CI: compile OSGeo4W-Windows build with clang#2811
nilason wants to merge 1 commit intoOSGeo:mainfrom
nilason:test_win

Conversation

@nilason
Copy link
Contributor

@nilason nilason commented Feb 8, 2023

The last week the Windows CI runner failed with linking failure, the reason was reported here: msys2/MINGW-packages#15469. Searching for a fix/alternative/solution I came up with this PR.

The original solution has now apparently been fixed, but I put this up as a "proof of concept".

@hellik
Copy link
Member

hellik commented Feb 8, 2023

I put this up as a "proof of concept".

for some kind of a "general" switch to clang?

@nilason
Copy link
Contributor Author

nilason commented Feb 8, 2023

I put this up as a "proof of concept".

for some kind of a "general" switch to clang?

Would that really be such a bad thing?

Anyways, I had no intention to actually pushing for this PR, but put it up in case a similar situation comes up with no working CI runner for Windows.
(On the other hand MS and Visual Studio seems to embrace clang in general, soo....)
All the same, I'll leave this PR on the shelf (just like #2281), to reach for if needed.

@hellik
Copy link
Member

hellik commented Feb 8, 2023

Would that really be such a bad thing?

not a bad thing :-)

Just a case of curiosity

@nilason
Copy link
Contributor Author

nilason commented Feb 8, 2023

Would that really be such a bad thing?

not a bad thing :-)

Just a case of curiosity

I think the CI runner should reflect the way the release build is built. Now, the release dist is built with gcc, then gcc should preferably be used for CI too. But if the CI not working for a prolonged period of time, it's good to have an alternative.

@nilason nilason added the CI Continuous integration label Feb 15, 2023
@neteler neteler added this to the 8.4.0 milestone Aug 16, 2023
@echoix echoix marked this pull request as draft January 27, 2024 18:23
@wenzeslaus wenzeslaus modified the milestones: 8.4.0, Future Apr 27, 2024
@echoix
Copy link
Member

echoix commented Aug 1, 2024

Is it worth keeping the draft open, or simply closing it unmerged and reference it when needed (or reopening) if ever it is needed?

@echoix echoix closed this Aug 21, 2024
@nilason nilason deleted the test_win branch May 14, 2025 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Continuous integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants