Skip to content

Commit 4ecc73b

Browse files
authored
Merge pull request #1467 from xnox/fix-pc-for-all
sca: make pc: provides/vendored use full package version
2 parents e9b562e + b01c8b1 commit 4ecc73b

2 files changed

Lines changed: 6 additions & 19 deletions

File tree

pkg/sca/sca.go

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@ import (
2323
"io"
2424
"io/fs"
2525
"path/filepath"
26-
"regexp"
2726
"slices"
2827
"strings"
2928
"unicode"
@@ -379,8 +378,6 @@ func generateSharedObjectNameDeps(ctx context.Context, hdl SCAHandle, generated
379378
return nil
380379
}
381380

382-
var pkgConfigVersionRegexp = regexp.MustCompile("-(alpha|beta|rc|pre)")
383-
384381
// TODO(kaniini): Turn this feature on once enough of Wolfi is built with provider data.
385382
var generateRuntimePkgConfigDeps = false
386383

@@ -439,18 +436,12 @@ func generatePkgConfigDeps(ctx context.Context, hdl SCAHandle, generated *config
439436
pcName := filepath.Base(path)
440437
pcName, _ = strings.CutSuffix(pcName, ".pc")
441438

442-
// TODO: https://github.com/chainguard-dev/melange/issues/1172
443-
sigh := func(ver string) string {
444-
return strings.TrimSuffix(ver, "-release")
445-
}
446-
447-
apkVersion := pkgConfigVersionRegexp.ReplaceAllString(pkg.Version, "_$1")
448439
if isInDir(path, []string{"lib/pkgconfig/", "usr/lib/pkgconfig/", "lib64/pkgconfig/", "usr/lib64/pkgconfig/"}) {
449440
log.Infof(" found pkg-config %s for %s", pcName, path)
450-
generated.Provides = append(generated.Provides, fmt.Sprintf("pc:%s=%s", pcName, sigh(apkVersion)))
441+
generated.Provides = append(generated.Provides, fmt.Sprintf("pc:%s=%s", pcName, hdl.Version()))
451442
} else {
452443
log.Infof(" found vendored pkg-config %s for %s", pcName, path)
453-
generated.Vendored = append(generated.Vendored, fmt.Sprintf("pc:%s=%s", pcName, sigh(apkVersion)))
444+
generated.Vendored = append(generated.Vendored, fmt.Sprintf("pc:%s=%s", pcName, hdl.Version()))
454445
}
455446

456447
if generateRuntimePkgConfigDeps {

pkg/sca/sca_test.go

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -186,14 +186,10 @@ func TestVendoredPkgConfig(t *testing.T) {
186186
}),
187187
Vendored: util.Dedup([]string{
188188
"so:libecpg_compat.so.3=3",
189-
"pc:libecpg=14.10",
190-
"pc:libecpg_compat=14.10",
191-
"pc:libpgtypes=14.10",
192-
"pc:libpq=14.10",
193-
"pc:libecpg=15.5",
194-
"pc:libecpg_compat=15.5",
195-
"pc:libpgtypes=15.5",
196-
"pc:libpq=15.5",
189+
"pc:libecpg=4604-r0",
190+
"pc:libecpg_compat=4604-r0",
191+
"pc:libpgtypes=4604-r0",
192+
"pc:libpq=4604-r0",
197193
}),
198194
}
199195

0 commit comments

Comments
 (0)