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

[glib] glib-2.0 .pc file contains non-existing include directory #15326

Closed
traversaro opened this issue Dec 27, 2020 · 2 comments · Fixed by #15360
Closed

[glib] glib-2.0 .pc file contains non-existing include directory #15326

traversaro opened this issue Dec 27, 2020 · 2 comments · Fixed by #15360
Assignees
Labels
category:port-bug The issue is with a library, which is something the port should already support

Comments

@traversaro
Copy link
Contributor

Describe the bug
The file installed/x64-linux/debug/lib/pkgconfig/glib-2.0.pc contains a reference to a non-existing include directory.

Environment

  • OS: Linux
  • Compiler: Not Relevant

To Reproduce
Steps to reproduce the behavior:

  1. ./vcpkg install gts:x64-linux
  2. Inspect the file installed/x64-linux/debug/lib/pkgconfig/glib-2.0.pc . It contains a reference to -I"${includedir}/glib-2.0" that expands to installed/x64-linux/include/glib-2.0 -I"${libdir}/glib-2.0/include", that expands to installed/x64-linux/debug/lib/glib-2.0/include . However, both directories do not exists, as the only directory that exists is ``installed/x64-linux/include/glib`.

Expected behavior
The .pc file of glib should only contain references to existing directories.

Failure logs
The PR #11273 fails with the following error due to this bug:

CMake Error in graphics/src/CMakeLists.txt:
  Imported target "GTS::GTS" includes non-existent path

    "/mnt/vcpkg-ci/installed/x64-linux/debug/lib/pkgconfig/../../../include/glib-2.0"

  in its INTERFACE_INCLUDE_DIRECTORIES.  Possible reasons include:

  * The path was deleted, renamed, or moved to another location.

  * An install or uninstall procedure did not complete successfully.

  * The installation package was faulty and references files it does not
  provide.

Additional details

Related PR: #11884 .

@PhoebeHui PhoebeHui added the category:port-bug The issue is with a library, which is something the port should already support label Dec 28, 2020
@PhoebeHui
Copy link
Contributor

@traversaro, thanks for bringing this up!

I have submitted PR #15360 to fix this issue.

@traversaro
Copy link
Contributor Author

Thanks @PhoebeHui for providing the fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-bug The issue is with a library, which is something the port should already support
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants