Skip to content
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

Release tag free wrapper #26

Closed
wants to merge 2 commits into from
Closed

Conversation

varunagrawal
Copy link
Collaborator

Updated the cmake files so that the non-release versions of the gtsam.so and other similar files do not have the release type tag attached to them. This also ensures the release specific information (e.g. debug info) is properly captured rather than circumvented.

@dellaert dellaert requested review from chrisbeall and removed request for dellaert May 29, 2019 14:58
@dellaert
Copy link
Member

Should we close after our off-line discussion?

@varunagrawal
Copy link
Collaborator Author

varunagrawal commented May 30, 2019 via email

@dellaert
Copy link
Member

Ok

@dellaert dellaert closed this May 30, 2019
@varunagrawal varunagrawal deleted the feature/release-free-wrapper branch September 22, 2020 19:18
varunagrawal added a commit that referenced this pull request Jan 13, 2021
186ed2c79 Merge pull request #26 from borglab/fix/required
9af1b8f09 unmake python cmake package as required

git-subtree-dir: wrap
git-subtree-split: 186ed2c792f80bbd315e747ef8622f8355c626f6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants