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-31721: Rename package to cpputils #1
Conversation
84854bc
to
439e4d8
Compare
This package is the utils package but with all the python-only code removed from it. What remains is the C++ code and the pybind11 interfaces to that code.
439e4d8
to
0358688
Compare
This makes it possible for C++ applications that previously used the utils C++ code to still use it without modification. Changing to lsst::cpputils requires all the code to change because of the complexity of forwarding namespaces from the utils package to the cpputils package through a deprecation period.
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.
Overall OK.
ups/cpputils.cfg
Outdated
@@ -9,6 +9,6 @@ dependencies = { | |||
|
|||
config = lsst.sconsUtils.Configuration( | |||
__file__, | |||
headers=["lsst/utils/Utils.h"], | |||
headers=["lsst/cpputils/Utils.h"], |
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.
This doesn't look right. I'm surprised Jenkins worked. Although it would work if you install the symlinks mentioned above.
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.
definitely all passed. I don't really understand what that line does because Utils.h
is formally deprecated.
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 removed Utils.h completely and put packaging.h in there instead. I also adjusted afw to stop using Utils.h.
This reverts commit aae2ac6.
a9b0f98
to
a44d6b6
Compare
This allows legacy code to still look for lsst::utils A symlink is used per file. This makes it explicit which include files are supporting deprecation. Also, directory symlinks are not currently supported by sconsUtils install.
Has not been used for 3 years since DM-10384.
a44d6b6
to
82ae60c
Compare
This package is the utils package but with all the python-only code
removed from it (see lsst/utils#97). What remains is the C++ code and the pybind11
interfaces to that code.