Rework types a little

Author: markusthoemmes

pkg/apk/apk/implementation.go

@@ -786,9 +786,10 @@ func (a *APK) InstallPackages(ctx context.Context, sourceDateEpoch *time.Time, a
 				if err != nil {
 					return fmt.Errorf("failed to read .PKGINFO for %s: %w", pkg, err)
 				}
-				infos[i] = pkgInfo
+				asPackage := pkgInfo.AsPackage(exp.ControlHash, uint64(exp.Size))
+				infos[i] = &asPackage
 
-				installedFiles, err := a.installPackage(ctx, pkgInfo, exp, sourceDateEpoch)
+				installedFiles, err := a.installPackage(ctx, &asPackage, exp, sourceDateEpoch)
 				if err != nil {
 					return fmt.Errorf("installing %s: %w", pkg, err)
 				}

pkg/apk/expandapk/expandapk.go

@@ -21,7 +21,6 @@ import (
 	"path/filepath"
 	"strings"
 	"sync"
-	"time"
 
 	"chainguard.dev/apko/internal/tarfs"
 	"chainguard.dev/apko/pkg/apk/types"
@@ -104,12 +103,12 @@ type APKExpanded struct {
 	SignatureSize int64
 
 	sync.Mutex
-	parsedPkgInfo *types.Package
+	parsedPkgInfo *types.PackageInfo
 	controlData   []byte
 }
 
 // PkgInfo parses and returns the .PKGINFO file.
-func (a *APKExpanded) PkgInfo() (*types.Package, error) {
+func (a *APKExpanded) PkgInfo() (*types.PackageInfo, error) {
 	a.Lock()
 	defer a.Unlock()
 	if a.parsedPkgInfo != nil {
@@ -127,31 +126,7 @@ func (a *APKExpanded) PkgInfo() (*types.Package, error) {
 		return nil, fmt.Errorf("parsing .PKGINFO: %w", err)
 	}
 
-	pkg := &types.Package{
-		Name:             pkginfo.Name,
-		Version:          pkginfo.Version,
-		Arch:             pkginfo.Arch,
-		Description:      pkginfo.Description,
-		License:          pkginfo.License,
-		Origin:           pkginfo.Origin,
-		Maintainer:       pkginfo.Maintainer,
-		URL:              pkginfo.URL,
-		Checksum:         a.ControlHash,
-		Dependencies:     pkginfo.Dependencies,
-		Provides:         pkginfo.Provides,
-		InstallIf:        pkginfo.InstallIf,
-		Size:             uint64(a.Size),
-		InstalledSize:    pkginfo.Size,
-		ProviderPriority: pkginfo.ProviderPriority,
-		BuildTime:        time.Unix(pkginfo.BuildDate, 0).UTC(),
-		BuildDate:        pkginfo.BuildDate,
-		RepoCommit:       pkginfo.RepoCommit,
-		Replaces:         pkginfo.Replaces,
-		DataHash:         pkginfo.DataHash,
-		Triggers:         pkginfo.Triggers,
-	}
-
-	a.parsedPkgInfo = pkg
+	a.parsedPkgInfo = pkginfo
 	return a.parsedPkgInfo, nil
 }
 

pkg/apk/types/package.go

@@ -31,6 +31,34 @@ type PackageInfo struct {
 	Triggers         []string `ini:"triggers,,allowshadow"`
 }
 
+// AsPackage converts PackageInfo to a Package struct with the given controlHash and size.
+func (pkginfo *PackageInfo) AsPackage(controlHash []byte, size uint64) Package {
+	return Package{
+		Name:             pkginfo.Name,
+		Version:          pkginfo.Version,
+		Arch:             pkginfo.Arch,
+		Description:      pkginfo.Description,
+		License:          pkginfo.License,
+		Origin:           pkginfo.Origin,
+		Maintainer:       pkginfo.Maintainer,
+		URL:              pkginfo.URL,
+		Dependencies:     pkginfo.Dependencies,
+		Provides:         pkginfo.Provides,
+		InstallIf:        pkginfo.InstallIf,
+		InstalledSize:    pkginfo.Size,
+		ProviderPriority: pkginfo.ProviderPriority,
+		BuildDate:        pkginfo.BuildDate,
+		RepoCommit:       pkginfo.RepoCommit,
+		Replaces:         pkginfo.Replaces,
+		DataHash:         pkginfo.DataHash,
+		Triggers:         pkginfo.Triggers,
+
+		BuildTime: time.Unix(pkginfo.BuildDate, 0).UTC(),
+		Checksum:  controlHash,
+		Size:      uint64(size),
+	}
+}
+
 // Package represents a single package with the information present in an
 // APKINDEX.
 type Package struct {