Skip to content

Fixing package name with numeric issue in top level package resolving #456

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.

Sign up for GitHub

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

Merged
yockgen merged 3 commits into from
Mar 4, 2026
Merged

Fixing package name with numeric issue in top level package resolving #456

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
b53e6b7
Fixing package name with numeric issue in top level package resolving
yockgen Mar 2, 2026
7eaa77b
Merge branch 'main' into pg-dev-fix-rpm-version
yockgen Mar 3, 2026
62f28e5
Merge branch 'main' into pg-dev-fix-rpm-version
elvin03 Mar 4, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion internal/ospackage/rpmutils/helper.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -405,7 +405,11 @@ func ResolveTopPackageConflicts(want string, all []ospackage.PackageInfo) (ospac
candidates = append(candidates, pi)
break
}
cleanName := extractBasePackageNameFromFile(pi.Name)
// Use PkgName if available, otherwise extract from filename
cleanName := pi.PkgName
if cleanName == "" {
cleanName = extractBasePackageNameFromFile(pi.Name)
}
Comment thread
yockgen marked this conversation as resolved.
Comment thread
yockgen marked this conversation as resolved.

if isGlob {
if matchPackageRequest(want, cleanName) || matchPackageRequest(want, pi.Name) {
Expand Down
Loading