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
DM-10384: builder helper for one-module-per-package pybind11 #59
Conversation
python/lsst/sconsUtils/scripts.py
Outdated
return result | ||
|
||
## | ||
# @brief Convenience function to replace standard python/*/SConscript boilerplate. | ||
# | ||
# @deprecated Use BasicSConstrcipt.python() for new code with all source |
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.
Unimportant typo here in BasicSconscript.
python/lsst/sconsUtils/scripts.py
Outdated
@@ -330,39 +330,41 @@ def rewrite_shebang(target, source, env): | |||
# | |||
# With no arguments, this will build a SWIG module with the name determined according |
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.
Not SWIG anymore :)
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.
One comment, otherwise looks fine.
return result | ||
|
||
## | ||
# @brief Convenience function to replace standard python/*/SConscript boilerplate. | ||
# | ||
# @deprecated Use BasicSConscript.python() for new code with all source |
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 there a rough "deprecated until" date you could assign to this one? "When the last lsst package is converted to the new style" would be good enough for me, but we should try to be more explicit about deprecation durations.
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.
The "when the last packages is converted" is probably the best I can promise, but I'll add that.
We no longer need Swig support, and this avoids clashes with the now-deprecated BasicSConscript.pybind11 function.
61c6bf9
to
fe194e5
Compare
No description provided.