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

Fix find_package(hidapi) warnings #404

Merged
merged 1 commit into from
May 2, 2022
Merged

Fix find_package(hidapi) warnings #404

merged 1 commit into from
May 2, 2022

Conversation

Youw
Copy link
Member

@Youw Youw commented Apr 25, 2022

  • FindPkgConfig.cmake warns it is not called by find_package;
  • CMake package config file and exports file names should not match,
    otherwise it gets included recursively by GLOB expression;

- FindPkgConfig.cmake warns it is not called by find_package;
- CMake package `config` file and `exports` file names should not match,
otherwise it gets included recursively by GLOB expression;
@Youw Youw added the CMake Related to CMake build system label Apr 25, 2022
@Youw Youw merged commit 874b29c into master May 2, 2022
@Youw Youw deleted the cmake-export-fixes branch May 2, 2022 11:09
@stsichler
Copy link
Contributor

this commit seems to break detection of find_package() of specific versions, see #482

@Youw
Copy link
Member Author

Youw commented Nov 16, 2022

Looks like so.
We need a CI check for scenario described in #482.
Thanks for finding it.

Any chance you could provide a PR with the fix?

@Youw
Copy link
Member Author

Youw commented Nov 16, 2022

Ah, I specifically renamed the hidapi-config-version into libhidapi-version.
Not sure what was the reason for that.
It might be accidentally or I was trying to fix some of the warnings.
Would need to check.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CMake Related to CMake build system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants