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

[bug] BazelDeps fails to generate with libpng due to cpp_info.location #17457

Closed
valgur opened this issue Dec 13, 2024 · 3 comments
Closed

[bug] BazelDeps fails to generate with libpng due to cpp_info.location #17457

valgur opened this issue Dec 13, 2024 · 3 comments
Assignees

Comments

@valgur
Copy link
Contributor

valgur commented Dec 13, 2024

Describe the bug

Conan 2.10.2
Ubuntu 24.04

ERROR: Error in generator 'BazelDeps': libpng/1.6.44: Cannot obtain 'location' for library 'png' in ['/home/martin/.conan2/p/b/libpn667f314ae558c/p/lib']. You can specify 'cpp_info.location' directly or report in github.com/conan-io/conan/issues if you think it should have been deduced correctly.

The only thing of note about the libpng recipe that I can see is that it's missing package_type, but adding it does not fix the issue.

How to reproduce it

conan install . this conanfile.txt

[requires]
libpng/[>=1.6 <2]

[generators]
BazelDeps
@valgur
Copy link
Contributor Author

valgur commented Dec 13, 2024

The content of package_folder for libpng is:

├── conaninfo.txt
├── conanmanifest.txt
├── include
│   ├── libpng16
│   │   ├── pngconf.h
│   │   ├── png.h
│   │   └── pnglibconf.h
│   ├── pngconf.h
│   ├── png.h
│   └── pnglibconf.h
├── lib
│   ├── cmake
│   │   └── PNG
│   ├── libpng16.a
│   └── libpng.a -> libpng16.a
└── licenses
    └── LICENSE

My guess is that it's ignoring the libpng.a/libpng.so symlink for some reason.

@franramirez688
Copy link
Contributor

Hi @valgur

Thanks a lot for reporting this 😁
Indeed, it was a bug, but this PR #17421 already solves it.
It'll be released in the next Conan 2.11.0 👏

Closing the issue as solved, but do not hesitate to reopen if there is still any other problem.

@valgur
Copy link
Contributor Author

valgur commented Dec 13, 2024

Indeed, I can confirm that it's fixed in develop2. Thanks!

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