Add support for Intel C++ about wxwidgets HOT 6 OPEN

Yes, I don't know what a CI is. So someone feel free to take this issue. I wanted to leave documentation for those googling the error messages.

from wxwidgets.

I think the value should be vc instead of icx to make cmake happy.

from wxwidgets.

We used to support icc in the past but I have no idea if it's still supported, i.e. if the code actually compiles/works correctly with it. Any serious effort to revive support for it should include adding a CI build testing it, otherwise it will just inevitably bit rot away again.

If you can do this, please open a PR adding a CI job using icc and the necessary changes to make it work. TIA!

from wxwidgets.

icc is different from icx. icx is the newer LLVM-based Intel compiler and icc is their older one that is now deprecated (to be removed in a year or two actually).

When I tried compiling wx with icx in August of last year it worked.

from wxwidgets.

Sorry, I didn't realize the name has changed. But the gist remains the same: for a compiler to be really supported, it needs to be tested in the CI. It looks like Intel even has an apt repository, so adding a job using it should be pretty simple, if anybody is interested in doing it.

from wxwidgets.

CI is "continuous integration" and refers to the builds done automatically on every push and, even more importantly, every PR done in this repository. It ensures that new changes at least don't break the build and it's even more crucial to have it for rarely used compilers like this one.

It's not difficult to add a new CI job, see .github/workflows/ci.yml, but someone needs to do it.

from wxwidgets.