Test Windows x86 and x64 with Github Actions

[mvorisek]

Move Windows testing to unified place - GitHub Actions

needs #9721 and #9744

AppVeyour (orig):

Github Actions (moved):

[cmb69]

Moving Windows CI to GH actions is very welcome. Thank you! @flavioheleno and @iluuu1994 already worked on that; not sure what's the status of that.

Anyhow, I suggest to keep AppVeyor CI for a while, so we can verify that the move is complete.

[iluuu1994]

I started but didn't get very far. iluuu1994#10 I'm happy for anybody else to take over.

[flavioheleno]

I haven't gone too far either. Changes are here: flavioheleno@0ee6bc5 :-)

[mvorisek]

🚀 🚀 tests run!

guys, can you take care of the failing tests?

also, x86 cannot compile :(

[cmb69]

guys, can you take care of the failing tests?

On a very quick glance, this looks like a line ending issue. It might be necessary to checkout with LF.

also, x86 cannot compile :(

The CI runs with CFLAGS=/W1 /WX. So far, this is only supported for x64. I suggest to just run with CFLAGS=/W1 instead for now.

[mvorisek]

what is wrong with 59e357a? previous commit has no issue, but if the if is removed, build fails with command syntax error

[iluuu1994]

The condition isn't the same, maybe that's it?

Ah sorry, you actually removed the outer if. Are you sure the first if was even executed before? Anyway, sorry I can't really help here, I'm not very familiar with the Windows build process.

[mvorisek]

@cmb69 can you please help me with the one failing Windows test?

[mvorisek]

MUST BE REVERTED BEFORE MERGE

[mvorisek]

Hello, PR is passing CI, please let me know if there is anything to address

[github-actions]

There has not been any recent activity in this PR. It will automatically be closed in 7 days if no further action is taken.

[iluuu1994]

@mvorisek Thank you for this PR! I opened #10664 with a minimal set of changes.