Skip to content

Commit

Permalink
manifest: clarify that tomli is available in stdlib for rhel10
Browse files Browse the repository at this point in the history
The code is a bit too terse for my liking right now and does
just have an empty `case DISTO_EL10` - so this commit adds a
comment.
  • Loading branch information
mvo5 authored and achilleas-k committed Oct 8, 2024
1 parent 6c0b213 commit f5dadef
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions pkg/manifest/os.go
Original file line number Diff line number Diff line change
Expand Up @@ -319,6 +319,7 @@ func (p *OS) getBuildPackages(distro Distro) []string {
case DISTRO_EL8:
packages = append(packages, "python3-pytoml")
case DISTRO_EL10:
// no extra package needed, stdlib has "tomli"
default:
packages = append(packages, "python3-toml")
}
Expand All @@ -335,6 +336,7 @@ func (p *OS) getBuildPackages(distro Distro) []string {
case DISTRO_EL8:
packages = append(packages, "python3-pytoml")
case DISTRO_EL10:
// no extra package needed, stdlib has "tomli"
default:
packages = append(packages, "python3-toml")
}
Expand Down

0 comments on commit f5dadef

Please sign in to comment.