Skip to content

Commit

Permalink
fixing a little bug with the targets that are pass-through
Browse files Browse the repository at this point in the history
Signed-off-by: Juan Bustamante <jbustamante@vmware.com>
  • Loading branch information
jjbustamante committed Mar 7, 2024
1 parent 38ce8a7 commit c7b1521
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions internal/commands/buildpack_package.go
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,7 @@ func BuildpackPackage(logger logging.Logger, cfg config.Config, packager Buildpa
Flatten: flags.Flatten,
FlattenExclude: flags.FlattenExclude,
Labels: flags.Label,
Targets: targets,
Targets: multiArchCfg.Targets(),
}); err != nil {
return err
}
Expand Down Expand Up @@ -233,7 +233,6 @@ func processTargets(flags BuildpackPackageFlags, logger logging.Logger, targets
// it's a composite buildpack, targets must be defined in package.toml
currentTargets = bpPackageCfg.Targets
}

Check warning on line 235 in internal/commands/buildpack_package.go

View check run for this annotation

Codecov / codecov/patch

internal/commands/buildpack_package.go#L233-L235

Added lines #L233 - L235 were not covered by tests

multiArchCfg, err := buildpack.NewMultiArchConfig(currentTargets, expectedTargets, logger)
if err != nil {
return buildpack.MultiArchConfig{}, err
Expand Down

0 comments on commit c7b1521

Please sign in to comment.