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

Fix the use of MSVC in an MSYS environment #2087

Merged
merged 1 commit into from
Jan 22, 2020

Conversation

Baljak
Copy link
Contributor

@Baljak Baljak commented Jan 20, 2020

Since MSVC can also be used in an MSYS environment (for example the one provided with Git for Windows), we also need to check if the MinGW compiler is used alongside the MSYSTEM environment variable. This expand the MSYS check made in PR #2053

@MonkeyBreaker Could you check that this fixes your issue?
@kanonet Could you check as well that pybind11 + MSYS still work in your cases?

Fix #2079

@MonkeyBreaker
Copy link

@Baljak

I confirm that it now work's on my side

Great !

@wjakob
Copy link
Member

wjakob commented Jan 20, 2020

Shouldn't MSVC appear somewhere in the condition?

@Baljak
Copy link
Contributor Author

Baljak commented Jan 20, 2020

@wjakob I agree, I made the condition like this to allow more compilers than MSVC only.

@kanonet
Copy link
Contributor

kanonet commented Jan 21, 2020

I can confirm, that this does work for me with MSYS2+MinGW. I think this is good as you wrote it, this way it does only change behavior for Msys+MinGW. Did someone test MinGW without MSYS?
This is how my original PR #1851 should have looked, to bad it did not. Thx for fixing it!

@Baljak
Copy link
Contributor Author

Baljak commented Jan 21, 2020

Did someone test MinGW without MSYS?

I did and it works. Actually I made PR #2048 because I'm using MinGW alone. Now every configuration should be OK.

@MonkeyBreaker
Copy link

Hi !

Any idea when this PR will be merged on master ?
Nothing urgent, as a temporary solution we checkout on branch v2.4.

@wjakob wjakob merged commit 4f72ef8 into pybind:master Jan 22, 2020
@wjakob
Copy link
Member

wjakob commented Jan 22, 2020

Merged!

@wjakob
Copy link
Member

wjakob commented Jan 22, 2020

Thanks for tracking this down :)

@MonkeyBreaker
Copy link

Thank you !

reds-heig pushed a commit to reds-heig/giotto-learn that referenced this pull request Jan 22, 2020
After fix pybind/pybind11#2087, windows wheel should be able to compile again

Signed-off-by: julian <julian.burellaperez@heig-vd.ch>
@Baljak Baljak deleted the fix-msys-msvc branch January 31, 2020 13:11
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.

Compilation on windows Fails after PR #2053
4 participants