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 versions may not align with apk versions #1172

Open
jonjohnsonjr opened this issue Apr 24, 2024 · 2 comments
Open

pkg-config versions may not align with apk versions #1172

jonjohnsonjr opened this issue Apr 24, 2024 · 2 comments

Comments

@jonjohnsonjr
Copy link
Contributor

We ran into this with bash-dev which has:

curl -sL https://packages.wolfi.dev/os/aarch64/bash-dev-5.2.21-r1.apk | tar -Oxz usr/lib/pkgconfig/bash.pc | grep Version
Version: 5.2-release

Which makes apk add fail (but for some reason only with a local .apk???) which means we can't bump bash in wolfi.

We kind of try to do this with

var pkgConfigVersionRegexp = regexp.MustCompile("-(alpha|beta|rc|pre)")
but that only works for suffixes that also happen to be valid apk suffixes.

I'm not sure how we want to handle this, maybe we just ignore versions that aren't valid? Do we try to trim suffixes until it's a valid version? Who knows.

@xnox
Copy link
Contributor

xnox commented Apr 25, 2024

IMHO the version for pc that we put in APK metadata should be the package metadata, not contents of pc Version.

Whilst it feels like the soname, it really isn't.

@kaniini
Copy link
Contributor

kaniini commented May 15, 2024

Yeah I agree, it was a mistake to do this in abuild, too. I think we should push to just change it to $pkgver everywhere.

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

3 participants