sudara / pamplejuce

A JUCE audio plugin template. JUCE 8, Catch2, Pluginval, macOS notarization, Azure Trusted Signing, Github Actions
https://melatonin.dev/manuals/pamplejuce
MIT License
429 stars 43 forks source link

Remove Visual Studio / GitHub Actions workaround #103

Closed sudara closed 5 months ago

sudara commented 5 months ago

The following was added to the GitHub Workflow configure step:

-DCMAKE_CXX_FLAGS="/D_DISABLE_CONSTEXPR_MUTEX_CONSTRUCTOR"

This is because they broke all C++ builds, basically. See:

https://github.com/actions/runner-images/issues/10004

This issue is a reminder to check if it needs to be removed, perhaps late June or early July (unless GA is really on top of it)

zsliu98 commented 5 months ago

It seems that the fix (image 20240610.1) will complete its deployment very soon. I have just checked my repo and it has already been deployed.

https://github.com/actions/runner-images

BTW, will you go back to Clang on Windows?

sudara commented 5 months ago

Ah thanks for the heads up. I wonder what "deploying" should mean and if I should already revert.

I sort of like MSVC on Windows in the end. It's fast and there have been several problems with Clang... Maybe a good argument is that clang on windows is a niche of a niche (C++ compilation on GitHub Actions) so won't be supported as solidly as MSVC?...

zsliu98 commented 5 months ago

Ah thanks for the heads up. I wonder what "deploying" should mean and if I should already revert.

I sort of like MSVC on Windows in the end. It's fast and there have been several problems with Clang... Maybe a good argument is that clang on windows is a niche of a niche (C++ compilation on GitHub Actions) so won't be supported as solidly as MSVC?...

~I think it will finish the deployment very soon~ It has been fully deployed (seems to be much more responsive than the previous Linux one). And thanks for your opinion on MSVC. I will try it out soon.


It seems that the binary from image 20240610.1 has some problems with my local machine ... Don't know why :worried:


I reinstall MSVC locally and it works. Magic...