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

Require either pkg-config or pkgconf #275

Open
joshcangit opened this issue Nov 21, 2023 · 8 comments
Open

Require either pkg-config or pkgconf #275

joshcangit opened this issue Nov 21, 2023 · 8 comments

Comments

@joshcangit
Copy link

When installing this as an Apt package, it requires pkg-config as per #22.
I'm using Ubuntu 22.04 and have installed pkgconf.

If it needs this, then why not require either pkg-config or pkgconf?
I suppose there's a way to conditionally depend on one package if the other is not found or vice versa.

@joshcangit
Copy link
Author

Could modify the control file for deb packages with this.

--- control
+++ control
@@ -3,7 +3,7 @@
 Architecture: amd64
 Maintainer: Crystal Team <[email protected]>
 Installed-Size: 130120
-Depends: gcc, pkg-config, libpcre3-dev, libpcre2-dev, libevent-dev
+Depends: gcc, pkg-config|pkgconf, libpcre3-dev, libpcre2-dev, libevent-dev
 Recommends: libssl-dev, libz-dev, libxml2-dev, libgmp-dev, libyaml-dev
 Provides: crystal
 Section: devel

@straight-shoota
Copy link
Member

straight-shoota commented Nov 21, 2023

This seems a bit odd. As far as I can tell, pkgconf should provide pkg-config and thus fulfil the dependency.
Can you share the package information of your pkgconf?

Are you aware of any other packages defining pkgconf as an explicit alternative dependency?

@joshcangit
Copy link
Author

joshcangit commented Nov 21, 2023

I have apt reporting broken dependencies when installing crystal1.10.
Only if I remove pkgconf and install pkg-config beforehand then it works.
I couldn't find any packages that explicitly require pkgconf.

But some packages which depend on pkg-config could work fine with pkgconf.
I could find libavahi-client-dev as one of those.
Still, those packages have to be reinstalled before the Crystal deb package could be installed.

@straight-shoota
Copy link
Member

I can't reproduce this. I'm running the following commands on a clean ubuntu:22.04 docker container:

apt update
apt install -y pkgconf curl
curl -fsSL https://crystal-lang.org/install.sh | bash

The crystal package installs and works while pkg-config is aliased to pkgconfig. So that makes me wonder if this isn't an issue with your particular setup.

The official Debian package by the way has the same dependency on pkg-config: https://salsa.debian.org/deiv/crystal/-/blob/master/debian/control#L30

@joshcangit
Copy link
Author

Is installing from the Open Build Service the same?
The packages there seem to be a bit different from the Debian packages.

@straight-shoota
Copy link
Member

straight-shoota commented Nov 22, 2023

The install script installs packages from Open Build System. These are the only ones that we maintain.

@joshcangit
Copy link
Author

joshcangit commented Nov 22, 2023

Is it different if I set up and install the package manually without the script?

@straight-shoota
Copy link
Member

No. It's the same package.

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

2 participants