Skip to content
This repository has been archived by the owner on Feb 9, 2024. It is now read-only.

Commit

Permalink
Use Dependencies.GetPackages API instead direct package access as it (#…
Browse files Browse the repository at this point in the history
…2015)

needs to filter out the container package since it is only used for
backwards compatibility with the distribution hub and should be removed
once the hub has been updated to a more recent version.
  • Loading branch information
a-palchikov authored Aug 17, 2020
1 parent bfe9dbd commit f706a3a
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions lib/app/service/app.go
Original file line number Diff line number Diff line change
Expand Up @@ -905,8 +905,8 @@ func (r *applications) checkImportRequirements(manifest *schema.Manifest, req *a
}

// check dependencies-packages
for _, dep := range manifest.Dependencies.Packages {
locator, err := r.processMetadata(dep.Locator)
for _, dep := range manifest.Dependencies.GetPackages() {
locator, err := r.processMetadata(dep)
if err != nil {
return trace.Wrap(err)
}
Expand All @@ -920,8 +920,8 @@ func (r *applications) checkImportRequirements(manifest *schema.Manifest, req *a
}

// check dependencies-apps
for _, dep := range manifest.Dependencies.Apps {
locator, err := r.processMetadata(dep.Locator)
for _, dep := range manifest.Dependencies.GetApps() {
locator, err := r.processMetadata(dep)
if err != nil {
return trace.Wrap(err)
}
Expand Down

0 comments on commit f706a3a

Please sign in to comment.