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 Windows build #144

Merged
merged 9 commits into from
Dec 31, 2024
Merged

Fix Windows build #144

merged 9 commits into from
Dec 31, 2024

Conversation

jorisv
Copy link

@jorisv jorisv commented Dec 20, 2024

  • clang-cl build
  • cl build
  • Fix TU
  • CI

@jorisv jorisv force-pushed the topic/windows branch 2 times, most recently from 7e23916 to 5e83807 Compare December 20, 2024 13:45
@jorisv jorisv self-assigned this Dec 20, 2024
@jorisv
Copy link
Author

jorisv commented Dec 20, 2024

I have test on Windows with clang and clang-cl.

There is two tests that are not working right now:

  • curves_tests: Exception
  • test-s3-smooth: precision error and maybe bad stream input

@nim65s Do you think the pip workflow can work on Windows ?

@nim65s
Copy link
Contributor

nim65s commented Dec 20, 2024

cmeel does not support windows, because I have no idea how this platform is supposed to work :/

@jorisv
Copy link
Author

jorisv commented Dec 20, 2024

Ok, I can add a conda/pixi workflow if you think it's worth it.
There is also the conda-forge CI that can check Windows regression when a new release is done.

@nim65s
Copy link
Contributor

nim65s commented Dec 20, 2024

It's up to you :)

@jorisv jorisv force-pushed the topic/windows branch 3 times, most recently from 196b01f to 2e61205 Compare December 20, 2024 22:21
@jorisv jorisv force-pushed the topic/windows branch 6 times, most recently from beebc0a to 67f5814 Compare December 31, 2024 14:00
@jorisv
Copy link
Author

jorisv commented Dec 31, 2024

It's all good.
The recipe build well on Windows with cl and clang-cl.
I have added a Windows CI with conda.

.github/workflows/windows-conda.yml Outdated Show resolved Hide resolved
tests/test-so3-smooth.cpp Show resolved Hide resolved
Co-authored-by: Guilhem Saurel <guilhem.saurel@laas.fr>
@nim65s nim65s merged commit eae0677 into loco-3d:devel Dec 31, 2024
13 checks passed
@jorisv jorisv deleted the topic/windows branch December 31, 2024 15:43
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.

2 participants