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

Windows fixes #548

Merged
merged 31 commits into from
Mar 3, 2019
Merged

Windows fixes #548

merged 31 commits into from
Mar 3, 2019

Conversation

isuruf
Copy link
Member

@isuruf isuruf commented Mar 2, 2019

I've also skipped the tests mentioned in #517 (comment)

@isuruf
Copy link
Member Author

isuruf commented Mar 2, 2019

Here's an appveyor build, https://ci.appveyor.com/project/isuruf/flint2/builds/22761341

@isuruf
Copy link
Member Author

isuruf commented Mar 2, 2019

This PR also adds limited support for MPIR 3.0.0 on windows. (from limited I mean using mpz_get_d_2exp properly. Correct solution is to use mpz_get_2exp_d in MPIR master)

@tthsqe12
Copy link
Contributor

tthsqe12 commented Mar 3, 2019

I'm impressed you got this working on windows, but, why is fmpz_mpoly_divides_array a known failure?

@wbhart wbhart merged commit 2b8f8ac into flintlib:trunk Mar 3, 2019
@isuruf
Copy link
Member Author

isuruf commented Mar 4, 2019

I don't know why that test fails. I've reported the failures at #517 (comment). It's better to have CI working somehow instead of nothing at all.

@isuruf isuruf deleted the windows branch March 4, 2019 03:37
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.

3 participants