refactor(packmanager): streamline package manager initialization

jake-ciolek · jake-ciolek · commit d4d0537a0059 · 2023-08-28T12:47:07.000+02:00
Create a singleton for the umbrella package manager and use it instead of global package-scoped maps. Use more dependency injection. Move the scattered init() calls to RegisterPackageManager into one place. Create a bootstrap package for handling flag/zip registration. Use the bootstrapping package in main(). Propagate errors up and report them if initialization fails. This should make things a bit more testable in the future. GitHub-Fixes: #425 Signed-off-by: Jakub Ciolek <jakub@ciolek.dev>
diff --git a/api/api.go b/api/api.go
@@ -9,19 +9,20 @@ import (
 
 	zip "api.zip"
 	"k8s.io/apimachinery/pkg/api/resource"
-	utilruntime "k8s.io/apimachinery/pkg/util/runtime"
 
 	machinev1alpha1 "kraftkit.sh/api/machine/v1alpha1"
 	networkv1alpha1 "kraftkit.sh/api/network/v1alpha1"
 	volumev1alpha1 "kraftkit.sh/api/volume/v1alpha1"
 )
 
 func init() {
-	utilruntime.Must(zip.Register(
+	gob.Register(resource.Quantity{})
+}
+
+func RegisterSchemes() error {
+	return zip.Register(
 		machinev1alpha1.AddToScheme,
 		networkv1alpha1.AddToScheme,
 		volumev1alpha1.AddToScheme,
-	))
-
-	gob.Register(resource.Quantity{})
+	)
 }
diff --git a/cmd/kraft/build/build.go b/cmd/kraft/build/build.go
@@ -82,13 +82,12 @@ func New() *cobra.Command {
 }
 
 func (opts *Build) Pre(cmd *cobra.Command, args []string) error {
-	ctx := cmd.Context()
-	pm, err := packmanager.NewUmbrellaManager(ctx)
+	ctx, err := packmanager.WithDefaultUmbrellaManagerInContext(cmd.Context())
 	if err != nil {
 		return err
 	}
 
-	cmd.SetContext(packmanager.WithPackageManager(ctx, pm))
+	cmd.SetContext(ctx)
 
 	if len(args) == 0 {
 		opts.workdir, err = os.Getwd()
diff --git a/cmd/kraft/clean/clean.go b/cmd/kraft/clean/clean.go
@@ -76,13 +76,12 @@ func New() *cobra.Command {
 }
 
 func (opts *Clean) Pre(cmd *cobra.Command, _ []string) error {
-	ctx := cmd.Context()
-	pm, err := packmanager.NewUmbrellaManager(ctx)
+	ctx, err := packmanager.WithDefaultUmbrellaManagerInContext(cmd.Context())
 	if err != nil {
 		return err
 	}
 
-	cmd.SetContext(packmanager.WithPackageManager(ctx, pm))
+	cmd.SetContext(ctx)
 
 	opts.Platform = platform.PlatformByName(opts.Platform).String()
 
diff --git a/cmd/kraft/fetch/fetch.go b/cmd/kraft/fetch/fetch.go
@@ -52,13 +52,12 @@ func New() *cobra.Command {
 }
 
 func (opts *Fetch) Pre(cmd *cobra.Command, _ []string) error {
-	ctx := cmd.Context()
-	pm, err := packmanager.NewUmbrellaManager(ctx)
+	ctx, err := packmanager.WithDefaultUmbrellaManagerInContext(cmd.Context())
 	if err != nil {
 		return err
 	}
 
-	cmd.SetContext(packmanager.WithPackageManager(ctx, pm))
+	cmd.SetContext(ctx)
 
 	opts.Platform = platform.PlatformByName(opts.Platform).String()
 
diff --git a/cmd/kraft/kraft.go b/cmd/kraft/kraft.go
@@ -14,6 +14,7 @@ import (
 
 	"kraftkit.sh/cmdfactory"
 	"kraftkit.sh/config"
+	"kraftkit.sh/internal/bootstrap"
 	"kraftkit.sh/internal/cli"
 	kitupdate "kraftkit.sh/internal/update"
 	kitversion "kraftkit.sh/internal/version"
@@ -146,5 +147,10 @@ func main() {
 		}
 	}
 
+	if err := bootstrap.InitKraftkit(ctx); err != nil {
+		log.G(ctx).Errorf("could not init kraftkit: %v", err)
+		os.Exit(1)
+	}
+
 	cmdfactory.Main(ctx, cmd)
 }
diff --git a/cmd/kraft/menu/menu.go b/cmd/kraft/menu/menu.go
@@ -55,13 +55,12 @@ func New() *cobra.Command {
 }
 
 func (opts *Menu) Pre(cmd *cobra.Command, _ []string) error {
-	ctx := cmd.Context()
-	pm, err := packmanager.NewUmbrellaManager(ctx)
+	ctx, err := packmanager.WithDefaultUmbrellaManagerInContext(cmd.Context())
 	if err != nil {
 		return err
 	}
 
-	cmd.SetContext(packmanager.WithPackageManager(ctx, pm))
+	cmd.SetContext(ctx)
 
 	opts.Platform = platform.PlatformByName(opts.Platform).String()
 
diff --git a/cmd/kraft/pkg/list/list.go b/cmd/kraft/pkg/list/list.go
@@ -58,13 +58,12 @@ func New() *cobra.Command {
 }
 
 func (*List) Pre(cmd *cobra.Command, _ []string) error {
-	ctx := cmd.Context()
-	pm, err := packmanager.NewUmbrellaManager(ctx)
+	ctx, err := packmanager.WithDefaultUmbrellaManagerInContext(cmd.Context())
 	if err != nil {
 		return err
 	}
 
-	cmd.SetContext(packmanager.WithPackageManager(ctx, pm))
+	cmd.SetContext(ctx)
 
 	return nil
 }
diff --git a/cmd/kraft/pkg/pkg.go b/cmd/kraft/pkg/pkg.go
@@ -90,13 +90,12 @@ func (opts *Pkg) Pre(cmd *cobra.Command, _ []string) error {
 		return fmt.Errorf("the `--arch` and `--plat` options are not supported in addition to `--target`")
 	}
 
-	ctx := cmd.Context()
-	pm, err := packmanager.NewUmbrellaManager(ctx)
+	ctx, err := packmanager.WithDefaultUmbrellaManagerInContext(cmd.Context())
 	if err != nil {
 		return err
 	}
 
-	cmd.SetContext(packmanager.WithPackageManager(ctx, pm))
+	cmd.SetContext(ctx)
 
 	opts.Platform = platform.PlatformByName(opts.Platform).String()
 
diff --git a/cmd/kraft/pkg/pull/pull.go b/cmd/kraft/pkg/pull/pull.go
@@ -74,13 +74,12 @@ func New() *cobra.Command {
 }
 
 func (opts *Pull) Pre(cmd *cobra.Command, _ []string) error {
-	ctx := cmd.Context()
-	pm, err := packmanager.NewUmbrellaManager(ctx)
+	ctx, err := packmanager.WithDefaultUmbrellaManagerInContext(cmd.Context())
 	if err != nil {
 		return err
 	}
 
-	cmd.SetContext(packmanager.WithPackageManager(ctx, pm))
+	cmd.SetContext(ctx)
 
 	opts.Platform = platform.PlatformByName(opts.Platform).String()
 
diff --git a/cmd/kraft/pkg/push/push.go b/cmd/kraft/pkg/push/push.go
@@ -56,13 +56,12 @@ func New() *cobra.Command {
 }
 
 func (opts *Push) Pre(cmd *cobra.Command, _ []string) error {
-	ctx := cmd.Context()
-	pm, err := packmanager.NewUmbrellaManager(ctx)
+	ctx, err := packmanager.WithDefaultUmbrellaManagerInContext(cmd.Context())
 	if err != nil {
 		return err
 	}
 
-	cmd.SetContext(packmanager.WithPackageManager(ctx, pm))
+	cmd.SetContext(ctx)
 
 	return nil
 }
@@ -111,7 +110,11 @@ func (opts *Push) Run(cmd *cobra.Command, args []string) error {
 
 	var pmananger packmanager.PackageManager
 	if opts.Format != "auto" {
-		pmananger = packmanager.PackageManagers()[pack.PackageFormat(opts.Format)]
+		umbrella, err := packmanager.PackageManagers()
+		if err != nil {
+			return err
+		}
+		pmananger = umbrella[pack.PackageFormat(opts.Format)]
 		if pmananger == nil {
 			return errors.New("invalid package format specified")
 		}
diff --git a/cmd/kraft/pkg/source/source.go b/cmd/kraft/pkg/source/source.go
@@ -44,13 +44,12 @@ func New() *cobra.Command {
 }
 
 func (*Source) Pre(cmd *cobra.Command, _ []string) error {
-	ctx := cmd.Context()
-	pm, err := packmanager.NewUmbrellaManager(ctx)
+	ctx, err := packmanager.WithDefaultUmbrellaManagerInContext(cmd.Context())
 	if err != nil {
 		return err
 	}
 
-	cmd.SetContext(packmanager.WithPackageManager(ctx, pm))
+	cmd.SetContext(ctx)
 
 	return nil
 }
diff --git a/cmd/kraft/pkg/unsource/unsource.go b/cmd/kraft/pkg/unsource/unsource.go
@@ -43,13 +43,12 @@ func New() *cobra.Command {
 }
 
 func (*Unsource) Pre(cmd *cobra.Command, _ []string) error {
-	ctx := cmd.Context()
-	pm, err := packmanager.NewUmbrellaManager(ctx)
+	ctx, err := packmanager.WithDefaultUmbrellaManagerInContext(cmd.Context())
 	if err != nil {
 		return err
 	}
 
-	cmd.SetContext(packmanager.WithPackageManager(ctx, pm))
+	cmd.SetContext(ctx)
 
 	return nil
 }
diff --git a/cmd/kraft/pkg/update/update.go b/cmd/kraft/pkg/update/update.go
@@ -44,13 +44,12 @@ func New() *cobra.Command {
 }
 
 func (*Update) Pre(cmd *cobra.Command, _ []string) error {
-	ctx := cmd.Context()
-	pm, err := packmanager.NewUmbrellaManager(ctx)
+	ctx, err := packmanager.WithDefaultUmbrellaManagerInContext(cmd.Context())
 	if err != nil {
 		return err
 	}
 
-	cmd.SetContext(packmanager.WithPackageManager(ctx, pm))
+	cmd.SetContext(ctx)
 
 	return nil
 }
diff --git a/cmd/kraft/prepare/prepare.go b/cmd/kraft/prepare/prepare.go
@@ -52,13 +52,12 @@ func New() *cobra.Command {
 }
 
 func (opts *Prepare) Pre(cmd *cobra.Command, _ []string) error {
-	ctx := cmd.Context()
-	pm, err := packmanager.NewUmbrellaManager(ctx)
+	ctx, err := packmanager.WithDefaultUmbrellaManagerInContext(cmd.Context())
 	if err != nil {
 		return err
 	}
 
-	cmd.SetContext(packmanager.WithPackageManager(ctx, pm))
+	cmd.SetContext(ctx)
 
 	opts.Platform = platform.PlatformByName(opts.Platform).String()
 
diff --git a/cmd/kraft/properclean/properclean.go b/cmd/kraft/properclean/properclean.go
@@ -72,13 +72,12 @@ func New() *cobra.Command {
 }
 
 func (*ProperClean) Pre(cmd *cobra.Command, _ []string) error {
-	ctx := cmd.Context()
-	pm, err := packmanager.NewUmbrellaManager(ctx)
+	ctx, err := packmanager.WithDefaultUmbrellaManagerInContext(cmd.Context())
 	if err != nil {
 		return err
 	}
 
-	cmd.SetContext(packmanager.WithPackageManager(ctx, pm))
+	cmd.SetContext(ctx)
 
 	return nil
 }
diff --git a/cmd/kraft/run/run.go b/cmd/kraft/run/run.go
@@ -187,16 +187,19 @@ func (opts *Run) Pre(cmd *cobra.Command, _ []string) error {
 
 	if opts.RunAs == "" || !set.NewStringSet("kernel", "project").Contains(opts.RunAs) {
 		// Set use of the global package manager.
-		pm, err := packmanager.NewUmbrellaManager(ctx)
+		ctx, err := packmanager.WithDefaultUmbrellaManagerInContext(cmd.Context())
 		if err != nil {
 			return err
 		}
 
-		cmd.SetContext(packmanager.WithPackageManager(ctx, pm))
+		cmd.SetContext(ctx)
 	}
 
 	if opts.RunAs != "" {
-		runners := runnersByName()
+		runners, err := runnersByName()
+		if err != nil {
+			return err
+		}
 		if _, ok = runners[opts.RunAs]; !ok {
 			choices := make([]string, len(runners))
 			i := 0
@@ -229,7 +232,10 @@ func (opts *Run) Run(cmd *cobra.Command, args []string) error {
 
 	var run runner
 	var errs []error
-	runners := runners()
+	runners, err := runners()
+	if err != nil {
+		return err
+	}
 
 	// Iterate through the list of built-in runners which sequentially tests and
 	// first test whether the --as flag has been set to force a specific runner or
diff --git a/cmd/kraft/run/runner.go b/cmd/kraft/run/runner.go
@@ -32,29 +32,37 @@ type runner interface {
 // runners is the list of built-in runners which are checked sequentially for
 // capability.  The first to test positive via Runnable is used with the
 // controller.
-func runners() []runner {
+func runners() ([]runner, error) {
 	r := []runner{
 		&runnerLinuxu{},
 		&runnerKernel{},
 		&runnerProject{},
 	}
 
-	for _, pm := range packmanager.PackageManagers() {
+	umbrella, err := packmanager.PackageManagers()
+	if err != nil {
+		return nil, err
+	}
+
+	for _, pm := range umbrella {
 		r = append(r, &runnerPackage{
 			pm: pm,
 		})
 	}
 
-	return r
+	return r, nil
 }
 
 // runnersByName is a utility method that returns a map of the available runners
 // such that their alias name can be quickly looked up.
-func runnersByName() map[string]runner {
-	runners := runners()
+func runnersByName() (map[string]runner, error) {
+	runners, err := runners()
+	if err != nil {
+		return nil, err
+	}
 	ret := make(map[string]runner, len(runners))
 	for _, runner := range runners {
 		ret[runner.String()] = runner
 	}
-	return ret
+	return ret, nil
 }
diff --git a/cmd/kraft/set/set.go b/cmd/kraft/set/set.go
@@ -75,13 +75,12 @@ func New() *cobra.Command {
 }
 
 func (*Set) Pre(cmd *cobra.Command, _ []string) error {
-	ctx := cmd.Context()
-	pm, err := packmanager.NewUmbrellaManager(ctx)
+	ctx, err := packmanager.WithDefaultUmbrellaManagerInContext(cmd.Context())
 	if err != nil {
 		return err
 	}
 
-	cmd.SetContext(packmanager.WithPackageManager(ctx, pm))
+	cmd.SetContext(ctx)
 
 	return nil
 }
diff --git a/cmd/kraft/unset/unset.go b/cmd/kraft/unset/unset.go
@@ -73,13 +73,12 @@ func New() *cobra.Command {
 }
 
 func (*Unset) Pre(cmd *cobra.Command, _ []string) error {
-	ctx := cmd.Context()
-	pm, err := packmanager.NewUmbrellaManager(ctx)
+	ctx, err := packmanager.WithDefaultUmbrellaManagerInContext(cmd.Context())
 	if err != nil {
 		return err
 	}
 
-	cmd.SetContext(packmanager.WithPackageManager(ctx, pm))
+	cmd.SetContext(ctx)
 
 	return nil
 }
diff --git a/internal/bootstrap/init.go b/internal/bootstrap/init.go
diff --git a/machine/qemu/init.go b/machine/qemu/init.go
diff --git a/manifest/init.go b/manifest/init.go
diff --git a/oci/init.go b/oci/init.go
diff --git a/packmanager/context.go b/packmanager/context.go
diff --git a/packmanager/umbrella.go b/packmanager/umbrella.go
diff --git a/tools/github-action/main.go b/tools/github-action/main.go
diff --git a/unikraft/elfloader/init.go b/unikraft/elfloader/init.go

Original file line number	Diff line number	Diff line change
`@@ -82,13 +82,12 @@ func New() *cobra.Command {`
`82`	`82`	`}`
`83`	`83`
`84`	`84`	`func (opts Build) Pre(cmd cobra.Command, args []string) error {`
`85`		`- ctx := cmd.Context()`
`86`		`- pm, err := packmanager.NewUmbrellaManager(ctx)`
	`85`	`+ ctx, err := packmanager.WithDefaultUmbrellaManagerInContext(cmd.Context())`
`87`	`86`	`if err != nil {`
`88`	`87`	`return err`
`89`	`88`	`}`
`90`	`89`
`91`		`- cmd.SetContext(packmanager.WithPackageManager(ctx, pm))`
	`90`	`+ cmd.SetContext(ctx)`
`92`	`91`
`93`	`92`	`if len(args) == 0 {`
`94`	`93`	`opts.workdir, err = os.Getwd()`
Original file line number	Diff line number	Diff line change
`@@ -76,13 +76,12 @@ func New() *cobra.Command {`
`76`	`76`	`}`
`77`	`77`
`78`	`78`	`func (opts Clean) Pre(cmd cobra.Command, _ []string) error {`
`79`		`- ctx := cmd.Context()`
`80`		`- pm, err := packmanager.NewUmbrellaManager(ctx)`
	`79`	`+ ctx, err := packmanager.WithDefaultUmbrellaManagerInContext(cmd.Context())`
`81`	`80`	`if err != nil {`
`82`	`81`	`return err`
`83`	`82`	`}`
`84`	`83`
`85`		`- cmd.SetContext(packmanager.WithPackageManager(ctx, pm))`
	`84`	`+ cmd.SetContext(ctx)`
`86`	`85`
`87`	`86`	`opts.Platform = platform.PlatformByName(opts.Platform).String()`
`88`	`87`
Original file line number	Diff line number	Diff line change
`@@ -52,13 +52,12 @@ func New() *cobra.Command {`
`52`	`52`	`}`
`53`	`53`
`54`	`54`	`func (opts Fetch) Pre(cmd cobra.Command, _ []string) error {`
`55`		`- ctx := cmd.Context()`
`56`		`- pm, err := packmanager.NewUmbrellaManager(ctx)`
	`55`	`+ ctx, err := packmanager.WithDefaultUmbrellaManagerInContext(cmd.Context())`
`57`	`56`	`if err != nil {`
`58`	`57`	`return err`
`59`	`58`	`}`
`60`	`59`
`61`		`- cmd.SetContext(packmanager.WithPackageManager(ctx, pm))`
	`60`	`+ cmd.SetContext(ctx)`
`62`	`61`
`63`	`62`	`opts.Platform = platform.PlatformByName(opts.Platform).String()`
`64`	`63`
Original file line number	Diff line number	Diff line change
`@@ -14,6 +14,7 @@ import (`
`14`	`14`
`15`	`15`	`"kraftkit.sh/cmdfactory"`
`16`	`16`	`"kraftkit.sh/config"`
	`17`	`+ "kraftkit.sh/internal/bootstrap"`
`17`	`18`	`"kraftkit.sh/internal/cli"`
`18`	`19`	`kitupdate "kraftkit.sh/internal/update"`
`19`	`20`	`kitversion "kraftkit.sh/internal/version"`
`@@ -146,5 +147,10 @@ func main() {`
`146`	`147`	`}`
`147`	`148`	`}`
`148`	`149`
	`150`	`+ if err := bootstrap.InitKraftkit(ctx); err != nil {`
	`151`	`+ log.G(ctx).Errorf("could not init kraftkit: %v", err)`
	`152`	`+ os.Exit(1)`
	`153`	`+ }`
	`154`	`+`
`149`	`155`	`cmdfactory.Main(ctx, cmd)`
`150`	`156`	`}`
Original file line number	Diff line number	Diff line change
`@@ -55,13 +55,12 @@ func New() *cobra.Command {`
`55`	`55`	`}`
`56`	`56`
`57`	`57`	`func (opts Menu) Pre(cmd cobra.Command, _ []string) error {`
`58`		`- ctx := cmd.Context()`
`59`		`- pm, err := packmanager.NewUmbrellaManager(ctx)`
	`58`	`+ ctx, err := packmanager.WithDefaultUmbrellaManagerInContext(cmd.Context())`
`60`	`59`	`if err != nil {`
`61`	`60`	`return err`
`62`	`61`	`}`
`63`	`62`
`64`		`- cmd.SetContext(packmanager.WithPackageManager(ctx, pm))`
	`63`	`+ cmd.SetContext(ctx)`
`65`	`64`
`66`	`65`	`opts.Platform = platform.PlatformByName(opts.Platform).String()`
`67`	`66`
Original file line number	Diff line number	Diff line change
`@@ -58,13 +58,12 @@ func New() *cobra.Command {`
`58`	`58`	`}`
`59`	`59`
`60`	`60`	`func (List) Pre(cmd cobra.Command, _ []string) error {`
`61`		`- ctx := cmd.Context()`
`62`		`- pm, err := packmanager.NewUmbrellaManager(ctx)`
	`61`	`+ ctx, err := packmanager.WithDefaultUmbrellaManagerInContext(cmd.Context())`
`63`	`62`	`if err != nil {`
`64`	`63`	`return err`
`65`	`64`	`}`
`66`	`65`
`67`		`- cmd.SetContext(packmanager.WithPackageManager(ctx, pm))`
	`66`	`+ cmd.SetContext(ctx)`
`68`	`67`
`69`	`68`	`return nil`
`70`	`69`	`}`
Original file line number	Diff line number	Diff line change
`@@ -90,13 +90,12 @@ func (opts Pkg) Pre(cmd cobra.Command, _ []string) error {`
`90`	`90`	return fmt.Errorf("the `--arch` and `--plat` options are not supported in addition to `--target`")
`91`	`91`	`}`
`92`	`92`
`93`		`- ctx := cmd.Context()`
`94`		`- pm, err := packmanager.NewUmbrellaManager(ctx)`
	`93`	`+ ctx, err := packmanager.WithDefaultUmbrellaManagerInContext(cmd.Context())`
`95`	`94`	`if err != nil {`
`96`	`95`	`return err`
`97`	`96`	`}`
`98`	`97`
`99`		`- cmd.SetContext(packmanager.WithPackageManager(ctx, pm))`
	`98`	`+ cmd.SetContext(ctx)`
`100`	`99`
`101`	`100`	`opts.Platform = platform.PlatformByName(opts.Platform).String()`
`102`	`101`
Original file line number	Diff line number	Diff line change
`@@ -74,13 +74,12 @@ func New() *cobra.Command {`
`74`	`74`	`}`
`75`	`75`
`76`	`76`	`func (opts Pull) Pre(cmd cobra.Command, _ []string) error {`
`77`		`- ctx := cmd.Context()`
`78`		`- pm, err := packmanager.NewUmbrellaManager(ctx)`
	`77`	`+ ctx, err := packmanager.WithDefaultUmbrellaManagerInContext(cmd.Context())`
`79`	`78`	`if err != nil {`
`80`	`79`	`return err`
`81`	`80`	`}`
`82`	`81`
`83`		`- cmd.SetContext(packmanager.WithPackageManager(ctx, pm))`
	`82`	`+ cmd.SetContext(ctx)`
`84`	`83`
`85`	`84`	`opts.Platform = platform.PlatformByName(opts.Platform).String()`
`86`	`85`
Original file line number	Diff line number	Diff line change
`@@ -56,13 +56,12 @@ func New() *cobra.Command {`
`56`	`56`	`}`
`57`	`57`
`58`	`58`	`func (opts Push) Pre(cmd cobra.Command, _ []string) error {`
`59`		`- ctx := cmd.Context()`
`60`		`- pm, err := packmanager.NewUmbrellaManager(ctx)`
	`59`	`+ ctx, err := packmanager.WithDefaultUmbrellaManagerInContext(cmd.Context())`
`61`	`60`	`if err != nil {`
`62`	`61`	`return err`
`63`	`62`	`}`
`64`	`63`
`65`		`- cmd.SetContext(packmanager.WithPackageManager(ctx, pm))`
	`64`	`+ cmd.SetContext(ctx)`
`66`	`65`
`67`	`66`	`return nil`
`68`	`67`	`}`
`@@ -111,7 +110,11 @@ func (opts Push) Run(cmd cobra.Command, args []string) error {`
`111`	`110`
`112`	`111`	`var pmananger packmanager.PackageManager`
`113`	`112`	`if opts.Format != "auto" {`
`114`		`- pmananger = packmanager.PackageManagers()[pack.PackageFormat(opts.Format)]`
	`113`	`+ umbrella, err := packmanager.PackageManagers()`
	`114`	`+ if err != nil {`
	`115`	`+ return err`
	`116`	`+ }`
	`117`	`+ pmananger = umbrella[pack.PackageFormat(opts.Format)]`
`115`	`118`	`if pmananger == nil {`
`116`	`119`	`return errors.New("invalid package format specified")`
`117`	`120`	`}`