approvals / ApprovalTests.cpp

Native ApprovalTests for C++ on Linux, Mac and Windows
https://approvaltestscpp.readthedocs.io/en/latest/
Apache License 2.0
312 stars 51 forks source link

CppUTest is failing to build in AppVeyor's CygWin build #161

Closed claremacrae closed 3 years ago

claremacrae commented 3 years ago

On AppVeyor, the code doesn't build on Cygwin: https://ci.appveyor.com/project/isidore/approvaltests-cpp/builds/35730621/job/9arcqkvsg2b0i2in#L332

Originally posted by @claremacrae in https://github.com/approvals/ApprovalTests.cpp/issues/159#issuecomment-706770945

claremacrae commented 3 years ago

This has just started working again, with no code changes by me.

Example of passing build: https://ci.appveyor.com/project/isidore/approvaltests-cpp/builds/35750647

claremacrae commented 3 years ago

This has just started working again, with no code changes by me.

Example of passing build: https://ci.appveyor.com/project/isidore/approvaltests-cpp/builds/35750647

This was wrong - I had forgotten that I had turned off the building of CppUTest tests, so CppUTest itself was now not being built.

When I re-enabled the tests, this started failing again.

claremacrae commented 3 years ago

I tried adding -DCMAKE_BUILD_TYPE=Debug to the CMake generate step - and it resulted in mingw failing as well as Cygwin!

claremacrae commented 3 years ago

I suspect that the issue may be the mixing of Windows and Cygwin paths in the instructions generated by CMake...

For now, on Cygwin builds, I've disabled building CppUTest and our integration with it.

If any users ask for support for this, it can be investigated further.

claremacrae commented 3 years ago

I suspect that the issue may be the mixing of Windows and Cygwin paths in the instructions generated by CMake...

For now, on Cygwin builds, I've disabled building CppUTest and our integration with it.

If any users ask for support for this, it can be investigated further.

Marking this as wontfix and closing...