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

pkg_config name in glib #1090

Closed
weatherhead99 opened this issue Jan 6, 2020 · 2 comments
Closed

pkg_config name in glib #1090

weatherhead99 opened this issue Jan 6, 2020 · 2 comments
Labels
blocked blocked by other issue

Comments

@weatherhead99
Copy link

when using the pkg_config generator for the glib package it generates the wrong name

Package and Environment Details (include every applicable attribute)

  • Package Name/Version: glib/2.58.3
  • Operating System+version: Linux Ubuntu 18.04
  • Compiler+version: gcc-7.4
  • Conan version: Conan 1.21.0
  • Python version: Python 3.7.1

Conan profile (output of conan profile show default or conan profile show <profile> if custom profile is in use)

[settings]
os=Linux
os_build=Linux
arch=x86_64
arch_build=x86_64
compiler=gcc
compiler.version=7
compiler.libcxx=libstdc++11
build_type=Release
[options]
[build_requires]
[env]

Steps to reproduce (Include if Applicable)

when building a package that consumes glib via pkg-config, the name it generally uses to find the pc file is "glib-2.0.pc", whereas when using conan's "pkg_config" generator, the generated name is "glib.pc". I believe this can be fixed via a simple addition of

cpp_info.names["pkg_config"] = "glib-2.0"

in the package. Happy to submit patch for this

@ericLemanissier
Copy link
Contributor

Glib actually uses several pkg-config files (gio-2.0.pc, gmodule-2.0.pc, gobject-2.0.pc, gthread-2.0.pc etc.)
As conan does not have yet a component feature (conan-io/conan#5090), the pkg-config file generated from conan can only be a merge of all these. Providing a file with the same name as the upstream project but a different content would probably be misleading, but I have no strong opinion on this.

@Croydon Croydon added the blocked blocked by other issue label Jan 7, 2020
@Croydon
Copy link
Member

Croydon commented Jul 5, 2020

This recipe is now in the Conan Center Index.

Please migrate to the recipe version there.
If this is still an issue, please create a new issue in the Conan Center Index issue tracker.

Thanks!

@Croydon Croydon closed this as completed Jul 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked blocked by other issue
Projects
None yet
Development

No branches or pull requests

3 participants