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: 'Requires: curl' -- module does not exist #4155

Closed
7 tasks done
bazaah opened this issue Jan 23, 2023 · 7 comments
Closed
7 tasks done

pkg-config: 'Requires: curl' -- module does not exist #4155

bazaah opened this issue Jan 23, 2023 · 7 comments

Comments

@bazaah
Copy link

bazaah commented Jan 23, 2023

Description

When attempting to run pkg-config against the rdkafka.pc provided, I receive an error due to a missing module: curl.

pkg-config error
> pkg-config --print-requires --print-variables --modversion rdkafka
Package curl was not found in the pkg-config search path.
Perhaps you should add the directory containing `curl.pc'
to the PKG_CONFIG_PATH environment variable
Package 'curl', required by 'rdkafka', not found

This is caused by a change introduced in #4045 to add curl as a module.

I believe the correct module name is libcurl, as seen via

pkg-config --print-requires --print-variables --modversion libcurl

How to reproduce

Easily reproducible with access to podman/docker:

podman run --rm docker.io/archlinux/archlinux:latest \
	/bin/bash -c 'pacman -Sy --needed --noconfirm pkgconf librdkafka 2>&1 1>/dev/null && pkg-config --print-requires --print-variables --modversion rdkafka'

Checklist

IMPORTANT: We will close issues where the checklist has not been completed.

Please provide the following information:

  • librdkafka version (release number or git tag): 2.0.0
  • Apache Kafka version: -
  • librdkafka client configuration: -
  • Operating system: Archlinux, Linux - 6.0.7-arch1-1 #1 SMP PREEMPT_DYNAMIC - x86_64 GNU/Linux
  • Provide logs (with debug=.. as necessary) from librdkafka: -
  • Provide broker log excerpts: -
  • Critical issue: No
@damenc
Copy link

damenc commented Jan 24, 2023

Same here building Net::Kafka against librdkafka 2.0.0 on Manjaro Linux 5.15.85-1:

Configuring Net-Kafka-1.06
Running Makefile.PL
Package curl was not found in the pkg-config search path.
Perhaps you should add the directory containing `curl.pc'
to the PKG_CONFIG_PATH environment variable
Package 'curl', required by 'rdkafka', not found
 at Makefile.PL line 15.

@emasab
Copy link
Contributor

emasab commented Feb 6, 2023

Thanks @bazaah . I've created this PR.

@bazaah
Copy link
Author

bazaah commented Feb 14, 2023

So its been three weeks for what I assume is a relatively unassuming change.

Is there some issue I can help with? Are there unresolved questions that need to be answered? How can I help this move over the finish line?

@idzharbae
Copy link

had this problem too, we can copy libcurl.pc as curl.pc in the pkg-config path as temporary solution until @emasab 's PR is merged.

pkg-config --variable pc_path pkg-config # show pkg-config paths
cd /usr/lib/pkgconfig
cp libcurl.pc curl.pc

bazaah added a commit to bazaah/aur-ceph that referenced this issue Mar 7, 2023
This is a fix to confluentinc/librdkafka#4155, which blocks our ability
to build this package.

The plan is to apply the patch to the system file, output into a local
directory and instruct pkgconfig to search that directory.

References: #11
References: confluentinc/librdkafka#4155
@dvzrv
Copy link

dvzrv commented Apr 8, 2023

Hi! this is affecting rebuilds of other software (e.g. syslog-ng) now as the pkgconfig integration for rdkafka is broken (which in turn blocks the Python 3.11 rebuild on Arch Linux). Please merge the provided MR and release a new version! Thanks!

@bazaah
Copy link
Author

bazaah commented Apr 8, 2023

bluntly, it appears this isn't a priority for this project, and we should probably just see if the Archlinux package is willing to accept a patch like this bazaah/aur-ceph@d045174, as this has languished for ~2 months now.

@emasab
Copy link
Contributor

emasab commented Aug 2, 2023

fix released with v2.1.1

@emasab emasab closed this as completed Aug 2, 2023
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

No branches or pull requests

5 participants