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
Build modlunit/nocmodl with their own CMakeLists.txt #2848
base: master
Are you sure you want to change the base?
Conversation
This comment has been minimized.
This comment has been minimized.
✔️ 084effa -> Azure artifacts URL |
54a5131
to
77ee36f
Compare
77ee36f
to
5be8aeb
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
✔️ 9a01585 -> Azure artifacts URL |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
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.
Is the main purpose of this refactoring to pave the way for future replacment of nocmodl with the new NMODL?
This comment has been minimized.
This comment has been minimized.
✔️ 7968d7c -> Azure artifacts URL |
Yes, and to continue to clean the build process of nrn by separating what should be split. |
✔️ 98fca63 -> Azure artifacts URL |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2848 +/- ##
==========================================
- Coverage 67.18% 66.04% -1.14%
==========================================
Files 564 530 -34
Lines 104268 96000 -8268
==========================================
- Hits 70053 63405 -6648
+ Misses 34215 32595 -1620 ☔ View full report in Codecov by Sentry. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
✔️ 65550e9 -> Azure artifacts URL |
This comment has been minimized.
This comment has been minimized.
✔️ f97c78a -> Azure artifacts URL |
This comment has been minimized.
This comment has been minimized.
✔️ 61b317e -> Azure artifacts URL |
✔️ df9746f -> Azure artifacts URL |
This comment has been minimized.
This comment has been minimized.
Quality Gate passedIssues Measures |
✔️ 0e91489 -> Azure artifacts URL |
CMakeLists.txt
fornocmodl
CMakeLists.txt
formodlunit
CMakeLists.txt
fornrnoc
nocmodl_mod_to_cpp
cmake macrowrap_sprintf
toutils
NRN_COVERAGE_LIB
because it has been make empty since a long time.flex_target
andbison_target
instead of our owncmake
macros and functions.New targets:
modlunit
: an executable to<??>
nocmodl
: a compiler for mod filesdefault_mods
: to create a static library of default compiled mod files