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

[cmake] Re-fix the cmake config files #787

Closed

Conversation

JackBoosY
Copy link

Continue PR #775.

  1. Use cmake function EXPORT instead of hardcode.
  2. DRACO_LIB_DIR -> DRACO_LIBRARY_DIR (See draco: Improve installation and packaging support. #775 comment).
  3. Add cmake vairable DRACO_FOUND.

@JackBoosY
Copy link
Author

CLA signed.

@JackBoosY
Copy link
Author

Any news here?

@tomfinegan
Copy link
Contributor

Not yet, perhaps later in the week-- vcpkg support is not a project priority.

@JackBoosY
Copy link
Author

Not yet, perhaps later in the week-- vcpkg support is not a project priority.

Yeah vcpkg is not high pirority, but this is a bug.

@tomfinegan tomfinegan self-assigned this Jan 18, 2022
@tomfinegan tomfinegan self-requested a review January 18, 2022 22:42
@tomfinegan
Copy link
Contributor

Closing this in favor of the changes in #796. At present I don't see a really good reason for using the export functionality to achieve what the addition of a find_library() call to the config file can handle. Please open a new issue if problems persist after the latest update to the CMake config file.

@tomfinegan tomfinegan closed this Feb 3, 2022
@JackBoosY JackBoosY deleted the dev/jack/refactory-cmake-config branch February 5, 2022 08:04
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