-
Notifications
You must be signed in to change notification settings - Fork 96
implement cmake swig #1686
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
base: main
Are you sure you want to change the base?
implement cmake swig #1686
Conversation
|
Hi @rlcee,
which require these tests: build. @Mu2e/fnalbuild-users, @Mu2e/write have access to CI actions on main. ⌛ The following tests have been triggered for bb17562: build (Build queue - API unavailable) |
|
☀️ The build tests passed at bb17562.
N.B. These results were obtained from a build of this Pull Request at bb17562 after being merged into the base branch at a8081b5. For more information, please check the job page here. |
brownd1978
left a comment
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'm curious if this is complete or still in development
DataProducts/CMakeLists.txt
Outdated
| mu2e_swig(DataProducts_swig src/pywrap.i DataProducts) | ||
| endif() | ||
|
|
||
| #if( ${BUILD_PYTHON_INTERFACE} ) |
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 this section still under development?
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.
Oops, it is left over, now deleted
Code to add the swig python wrapper for select classes.
This should be inactive until paired with the +python flag in the Offline recipe, which will be added later.