-
Notifications
You must be signed in to change notification settings - Fork 129
run tests for nix #1094
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
run tests for nix #1094
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1094 +/- ##
=======================================
Coverage 91.87% 91.87%
=======================================
Files 30 30
Lines 5883 5883
=======================================
Hits 5405 5405
Misses 478 478 ☔ View full report in Codecov by Sentry. |
@@ -106,6 +106,7 @@ | |||
"-DBUILD_SHARED_LIBS=ON" | |||
"-DMANIFOLD_PAR=${if parallel then "ON" else "OFF"}" | |||
]; | |||
doCheck = true; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
build_mxe still looks like it's not testing - also build_mac_..._subdir, unless that's intentional?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
mxe cannot be tested: we are building some windows exe on linux. at least we cannot test it without another job on windows...
bulid_mac_..._subdir... is just to test the cmake consumer, see if we can build when put inside a subdirectory in a larger project.
@@ -106,6 +106,7 @@ | |||
"-DBUILD_SHARED_LIBS=ON" | |||
"-DMANIFOLD_PAR=${if parallel then "ON" else "OFF"}" | |||
]; | |||
doCheck = true; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And any idea why build_nix_tbb is so slow? It looks like it's building all of TBB before it even gets to Manifold...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added the latest tbb version for emscripten, forgot why I also use it for other native builds... fixed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
No description provided.