grass icon indicating copy to clipboard operation
grass copied to clipboard

CI: compile OSGeo4W-Windows build with clang

Open nilason opened this issue 2 years ago • 4 comments

The last week the Windows CI runner failed with linking failure, the reason was reported here: https://github.com/msys2/MINGW-packages/issues/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".

nilason avatar Feb 08 '23 18:02 nilason

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

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

hellik avatar Feb 08 '23 19:02 hellik

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.

nilason avatar Feb 08 '23 19:02 nilason

Would that really be such a bad thing?

not a bad thing :-)

Just a case of curiosity

hellik avatar Feb 08 '23 20:02 hellik

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 avatar Feb 08 '23 20:02 nilason

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 avatar Aug 01 '24 16:08 echoix