Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove Strawberry GCC compiler in Windows CI #41

Merged
merged 4 commits into from
Feb 3, 2023

Conversation

stefanv
Copy link
Member

@stefanv stefanv commented Jan 27, 2023

No description provided.

@stefanv stefanv force-pushed the fix-windows-ci-path branch from e2fc146 to 23c4ab9 Compare January 27, 2023 00:32
@stefanv stefanv force-pushed the fix-windows-ci-path branch from 23c4ab9 to e3c7ff0 Compare January 27, 2023 00:34
@stefanv stefanv force-pushed the fix-windows-ci-path branch from e26bbee to 6b421a8 Compare January 27, 2023 18:20
@stefanv stefanv force-pushed the fix-windows-ci-path branch from 99aaeac to b29cfa7 Compare January 27, 2023 20:47
@stefanv stefanv force-pushed the fix-windows-ci-path branch from 982ff3d to 789a8b5 Compare January 27, 2023 22:38
Remove-Item -Path C:\Strawberry -Recurse
Remove-Item -Path C:\ProgramData\Chocolatey\bin\gcc.exe
Remove-Item -Path C:\msys64 -Recurse -Force
Remove-Item -Path "C:\Program Files\LLVM" -Recurse
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How would this work when using devpy in NumPy? Would we need to add this CI step?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess it depends which compiler you want. If you're happy with clang, presumably you can just remove gcc. These steps are only necessary to simulate having the Visual Studio compilers.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yup -- this isn't really a devpy thing anyway. It happens for random C/C++ projects that don't have anything to do with python, also.

Using something like https://github.com/ilammy/msvc-dev-cmd is a good idea IMO, at least for GitHub actions. It guarantees that cl.exe is set up correctly and is somewhere in PATH, and if it's in PATH then it will always win as far as Meson is concerned.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR now correctly simulates an identical setup in CI to compare to the case where an end user on a personal laptop has MSVC installed, but never went and installed some other gcc/clang compiler.

i.e. it simulates the common case.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@stefanv
Copy link
Member Author

stefanv commented Jan 31, 2023

rf #38

@stefanv stefanv merged commit 1b1ed96 into scientific-python:main Feb 3, 2023
@jarrodmillman jarrodmillman added this to the 0.1 milestone Mar 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants