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

[question] How to generate custom content for PkgConfigDeps that is compatible with pkg_config #12279

Closed
1 task done
jwillikers opened this issue Oct 10, 2022 · 4 comments
Closed
1 task done
Assignees

Comments

@jwillikers
Copy link
Contributor

When migrating from pkg_config to PkgConfigDeps, custom content frequently breaks because it uses the fairly standard variable libdir but PkgConfigDeps defines this as libdir1. Would it be possible to change the default name to libdir when there is only one library directory? This would solve the issue in all cases that I've seen on CCI so far.

CCI issue: conan-io/conan-center-index#12890

@jwillikers
Copy link
Contributor Author

Update, looks like we probably need to use libdir and includedir even when using multiple of such directories for dependencies which still expect to find those variables.

@franramirez688
Copy link
Contributor

Hi @jwillikers! Sorry, I just realized this one.

I have a question about this issue. Regardless of changing libdir1 to libdir (the same for includedir), what about the cases that we're defining several includedirN and libdirN? Sorry for the basic question, but which is the use case to use that libdir and includedir?

@franramirez688
Copy link
Contributor

franramirez688 commented Sep 4, 2023

Closing this issue as it was already solved by #13623

UPDATED: Released since Conan v2.0.8 (actually, it was since v2.0.7, but a bug appeared, and it was solved by the next one).

@jwillikers
Copy link
Contributor Author

@franramirez688 Awesome, thank you!

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

No branches or pull requests

2 participants