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

cmake: add support for compiling bitpit with MSVC #447

Merged
merged 35 commits into from
Feb 26, 2024
Merged

Conversation

andrea-iob
Copy link
Member

Add support form compiling bitpit with MSVC.

@andrea-iob andrea-iob force-pushed the bitpit.msvc branch 3 times, most recently from 5695926 to 79beaa4 Compare February 18, 2024 15:11
@andrea-iob andrea-iob force-pushed the bitpit.msvc branch 13 times, most recently from cf970b0 to 08566a3 Compare February 18, 2024 21:55
@andrea-iob andrea-iob force-pushed the bitpit.msvc branch 2 times, most recently from c5c45c4 to 9659b07 Compare February 18, 2024 22:12
@edoardolombardi
Copy link
Member

This commit looks unrelated and it introduces changes in the API usage 057fa2d.
If not needed for functional purposes, please, exclude it from the pull request.

@andrea-iob
Copy link
Member Author

andrea-iob commented Feb 20, 2024

This commit looks unrelated and it introduces changes in the API usage 057fa2d. If not needed for functional purposes, please, exclude it from the pull request.

It's needed because there is a windows include that defines a macro named ERROR.

@andrea-iob
Copy link
Member Author

I fixed a problem introduced by commit 5eb85b2.

@andrea-iob andrea-iob merged commit 64e4839 into master Feb 26, 2024
10 checks passed
@andrea-iob andrea-iob deleted the bitpit.msvc branch February 26, 2024 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants