You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mynewt.apache.org by cc...@apache.org on 2019/09/03 16:36:00 UTC

[mynewt-newt] 01/02: target: Make transient resolve functions public

This is an automated email from the ASF dual-hosted git repository.

ccollins pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mynewt-newt.git

commit 980505c3da2e8e21cb01d0187de920318174e473
Author: Christopher Collins <cc...@apache.org>
AuthorDate: Mon Aug 26 09:55:48 2019 -0700

    target: Make transient resolve functions public
    
    These functions need to be public so that the target builder can reoslve
    linked packages prior to dependency resolution.  If this is not done
    before dependency resolution, it is not possible to determine a
    package's true type (e.g,. a deprecated "app" package will have the
    "transient" type, but it is really an "app").
---
 newt/target/target.go | 30 +++++++++++++++---------------
 1 file changed, 15 insertions(+), 15 deletions(-)

diff --git a/newt/target/target.go b/newt/target/target.go
index 9680ecb..ad458df 100644
--- a/newt/target/target.go
+++ b/newt/target/target.go
@@ -132,7 +132,7 @@ func (target *Target) Validate(appRequired bool) error {
 		return util.NewNewtError("Target does not specify a BSP package " +
 			"(target.bsp)")
 	}
-	bsp := target.resolvePackageName(target.BspName)
+	bsp := target.ResolvePackageName(target.BspName)
 	if bsp == nil {
 		return util.FmtNewtError("Could not resolve BSP package: %s",
 			target.BspName)
@@ -149,7 +149,7 @@ func (target *Target) Validate(appRequired bool) error {
 			return util.NewNewtError("Target does not specify an app " +
 				"package (target.app)")
 		}
-		app := target.resolvePackageName(target.AppName)
+		app := target.ResolvePackageName(target.AppName)
 		if app == nil {
 			return util.FmtNewtError("Could not resolve app package: %s",
 				target.AppName)
@@ -162,7 +162,7 @@ func (target *Target) Validate(appRequired bool) error {
 		}
 
 		if target.LoaderName != "" {
-			loader := target.resolvePackageName(target.LoaderName)
+			loader := target.ResolvePackageName(target.LoaderName)
 			if loader == nil {
 				return util.FmtNewtError(
 					"Could not resolve loader package: %s", target.LoaderName)
@@ -207,7 +207,7 @@ func (target *Target) Clone(newRepo *repo.Repo, newName string) *Target {
 	return &newTarget
 }
 
-func (target *Target) resolvePackageRepoAndName(repo *repo.Repo, name string) *pkg.LocalPackage {
+func (target *Target) ResolvePackageRepoAndName(repo *repo.Repo, name string) *pkg.LocalPackage {
 	dep, err := pkg.NewDependency(repo, name)
 	if err != nil {
 		return nil
@@ -221,8 +221,8 @@ func (target *Target) resolvePackageRepoAndName(repo *repo.Repo, name string) *p
 	return pack
 }
 
-func (target *Target) resolvePackageName(name string) *pkg.LocalPackage {
-	pack := target.resolvePackageYmlName(name)
+func (target *Target) ResolvePackageName(name string) *pkg.LocalPackage {
+	pack := target.ResolvePackageYmlName(name)
 
 	if pack == nil || pack.Type() != pkg.PACKAGE_TYPE_TRANSIENT {
 		return pack
@@ -231,42 +231,42 @@ func (target *Target) resolvePackageName(name string) *pkg.LocalPackage {
 	// We follow only one level of linking here to make things easier and assuming
 	// nested linking means someone using really deprecated packages ;)
 
-	pack = target.resolvePackageRepoAndName(pack.Repo().(*repo.Repo), pack.LinkedName())
+	pack = target.ResolvePackageRepoAndName(pack.Repo().(*repo.Repo), pack.LinkedName())
 
 	return pack
 }
 
-func (target *Target) resolvePackageYmlName(name string) *pkg.LocalPackage {
-	return target.resolvePackageRepoAndName(target.basePkg.Repo().(*repo.Repo), name)
+func (target *Target) ResolvePackageYmlName(name string) *pkg.LocalPackage {
+	return target.ResolvePackageRepoAndName(target.basePkg.Repo().(*repo.Repo), name)
 }
 
 // Methods below resolve package by name and follow links to get proper package
 
 func (target *Target) App() *pkg.LocalPackage {
-	return target.resolvePackageName(target.AppName)
+	return target.ResolvePackageName(target.AppName)
 }
 
 func (target *Target) Loader() *pkg.LocalPackage {
-	return target.resolvePackageName(target.LoaderName)
+	return target.ResolvePackageName(target.LoaderName)
 }
 
 func (target *Target) Bsp() *pkg.LocalPackage {
-	return target.resolvePackageName(target.BspName)
+	return target.ResolvePackageName(target.BspName)
 }
 
 // Methods below resolve package by name as stated in YML file (so do not follow links)
 // e.g. to use as seed for dependencies calculation
 
 func (target *Target) AppYml() *pkg.LocalPackage {
-	return target.resolvePackageYmlName(target.AppName)
+	return target.ResolvePackageYmlName(target.AppName)
 }
 
 func (target *Target) LoaderYml() *pkg.LocalPackage {
-	return target.resolvePackageYmlName(target.LoaderName)
+	return target.ResolvePackageYmlName(target.LoaderName)
 }
 
 func (target *Target) BspYml() *pkg.LocalPackage {
-	return target.resolvePackageYmlName(target.BspName)
+	return target.ResolvePackageYmlName(target.BspName)
 }
 
 // Save the target's configuration elements