-
Notifications
You must be signed in to change notification settings - Fork 1
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
Make PackageCompiler.jl compilation selectable in cmake #128
Conversation
- required JULIA_PROJECT_PATH (so does testing) - install missing - examples missing
So far compilation in an out-of-source build directory seems to work. Installation and examples are still missing. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #128 +/- ##
=======================================
Coverage 98.05% 98.05%
=======================================
Files 13 13
Lines 565 565
=======================================
Hits 554 554
Misses 11 11
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
s.t. no init.o will end up in the source tree
- also add Fortran module
apparently necessary s.t. a rule for building libtrixi.so is generated
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.
LGTM overall, thanks a lot for doing this! It would be great if you could also document how to enable the PC build in the README and/or docs. I've left a few minor notes as well.
I also added the cmake call to the docs. Thanks for the reminder! |
this allows to define dependencies
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.
LGTM! 🥳
Resolves #124
USE_PACKAGE_COMPILER
JULIA_PROJECT_PATH
being set (as does testing)