search

mstorsjo / msvc-wine

Scripts for setting up and running MSVC in Wine on Linux
Other
683 stars 83 forks source link

test: Add testing of the arch specific assembler tools #79

Closed mstorsjo closed 1 year ago

mstorsjo commented 1 year ago

Just testing that running the assembler exits successfully.

Adding the testing in test-cl.sh, instead of a separate test script, although I'm open to creating a separate test script if that's preferred.

CC @huangqinjin

huangqinjin commented 1 year ago

LGTM though I would prefer a separate script for masm.

mstorsjo commented 1 year ago

LGTM though I would prefer a separate script for masm.

Do you mean specifically the x86 masm tests, or splitting all of the newly added ones here into a separate test-asm.sh?

huangqinjin commented 1 year ago

splitting all of the newly added ones here into a separate test-asm.sh

That is what I mean. Sorry for confusion.

mstorsjo commented 1 year ago

Updated the branch to move the tests to a separate file. I also reordered the tests in test.sh to test the individual tools first before testing cmake/meson.

huangqinjin commented 1 year ago

LGTM!