Skip to content

HMS-8948: Fix module stream version parsing#1299

Merged
rverdile merged 1 commit intocontent-services:mainfrom
rverdile:perl
Dec 1, 2025
Merged

HMS-8948: Fix module stream version parsing#1299
rverdile merged 1 commit intocontent-services:mainfrom
rverdile:perl

Conversation

@rverdile
Copy link
Contributor

@rverdile rverdile commented Nov 13, 2025

Summary

A bug in the yummy dependency led to incorrect module stream version parsing. This bumps the yummy version to include the fix and fixes a bug that prevented the incorrect stream from being removed

You can pull in these changes locally and edit the go.mod to point to them:
content-services/yummy#39

Testing steps

  1. Introspect RHEL 8 Appstream
  2. Search module streams for perl. Without this PR, it would return perl "5.3" (and other versions). With this PR, "5.3" should be replaced by "5.30"
  3. Search the RHEL 8 Appstream repository for the perl RPM using /rpms/names/. Include package sources.
  4. The package sources should not include perl "5.3", only perl "5.30"

@xbhouse
Copy link
Contributor

xbhouse commented Nov 13, 2025

@TenSt TenSt self-assigned this Nov 26, 2025
A bug in the yummy dependency led to incorrect module stream version
parsing. This bumps the yummy version to include the fix and fixes a
bug that prevented the incorrect stream from being removed
@rverdile rverdile marked this pull request as ready for review November 26, 2025 15:52
Copy link
Contributor

@TenSt TenSt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! I've tested it locally with latest yammy version, all looks good

@rverdile rverdile merged commit bdf715a into content-services:main Dec 1, 2025
19 checks passed
@rverdile rverdile deleted the perl branch December 1, 2025 18:39
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

Successfully merging this pull request may close these issues.

3 participants

Comments