Skip to content
This repository was archived by the owner on Aug 7, 2025. It is now read-only.

Commit 8a95596

Browse files
bryteiseWilliam Douglas
authored andcommitted
Don't add direct include bundle packages
Direct include packages being accounted for causing the chroot bundle to be content beyond the package. This might be worth hiding behind a bundle config flag setting at some point but for now just disable.
1 parent 8f53a8f commit 8a95596

2 files changed

Lines changed: 16 additions & 5 deletions

File tree

builder/bundles.go

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -396,6 +396,22 @@ func resolvePackagesWithOptions(numWorkers int, set bundleSet, packagerCmd []str
396396
return
397397
}
398398

399+
singlePkgMap := make(repoPkgMap)
400+
for _, pkgs := range rpm {
401+
for _, pkg := range pkgs {
402+
if pkg.name == bundle.Name {
403+
singlePkgMap[pkg.repo] = append(singlePkgMap[pkg.repo], pkg)
404+
break
405+
}
406+
}
407+
if len(singlePkgMap) > 0 {
408+
break
409+
}
410+
}
411+
if len(singlePkgMap) > 0 {
412+
rpm = singlePkgMap
413+
}
414+
399415
for _, pkgs := range rpm {
400416
// Add packages to bundle's AllPackages
401417
for _, pkg := range pkgs {

builder/bundleset.go

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -58,11 +58,6 @@ func validateAndFillBundleSet(bundles bundleSet) error {
5858
for k, v := range b.DirectPackages {
5959
b.AllPackages[k] = v
6060
}
61-
for _, include := range b.DirectIncludes {
62-
for k, v := range bundles[include].AllPackages {
63-
b.AllPackages[k] = v
64-
}
65-
}
6661
}
6762

6863
return nil

0 commit comments

Comments
 (0)