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

Specify header file name for non-standard installs #5

Merged
merged 1 commit into from
Jun 19, 2019

Conversation

woodfell
Copy link
Contributor

Some package header files can't be found automatically because they are installed to non-standard locations. By default GenericFindDependencies will search for a system header file called "lib${TARGET}/${TARGET}.h", for example the module FindSwiftnav (libswiftnav) will try to find "libswiftnav/swiftnav.h". Some packages install using a different naming scheme, libswiftnav installs "swiftnav/bits.h" (among others). This PR specifies different header file names for those packages so they can be found properly.

@woodfell woodfell requested a review from benjaminaltieri June 17, 2019 23:34
@benjaminaltieri
Copy link
Contributor

does this only affect repos that use SWIFT_PREFERRED_DEPENDENCY_SOURCE=system? (piksi_apps, buildroot, etc) Or will all cmake/common users need this fix?

@woodfell
Copy link
Contributor Author

The affected modules were unable to find dependencies from the system because of weirdly named header files. This just fixes the search for these modules, users of these packages will need to be updated with this fix but only if they are using dependencies from the system. Currently this is just piksi_apps/buildroot. Everything else uses dependencies from source and so are unaffected.

Copy link
Contributor

@benjaminaltieri benjaminaltieri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

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