Felix Singer has submitted this change. ( https://review.coreboot.org/c/coreboot/+/83344?usp=email )
Change subject: autoport: Print location of generated sources ......................................................................
autoport: Print location of generated sources
Autoport determines the mainboard vendor and board names based on DMI entries, which sometimes doesn't result in the most obvious name. In addition, newcomers may not be familiar with coreboot's directory structure and have no idea where to look. Print out the absolute patch of the generated sources once autoport finishes so that it is easier to locate the files.
Change-Id: I4ba00484ac57355d7539fa6e36e0e6df62719f8a Signed-off-by: Nicholas Chin nic.c3.14@gmail.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/83344 Reviewed-by: Angel Pons th3fanbus@gmail.com Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Felix Singer service+coreboot-gerrit@felixsinger.de --- M util/autoport/main.go 1 file changed, 3 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Angel Pons: Looks good to me, approved Felix Singer: Looks good to me, approved
diff --git a/util/autoport/main.go b/util/autoport/main.go index b7130b2..18fb694 100644 --- a/util/autoport/main.go +++ b/util/autoport/main.go @@ -8,6 +8,7 @@ "fmt" "log" "os" + "path/filepath" "sort" "strings" ) @@ -903,4 +904,6 @@ end GMA.Mainboard; `) } + outputPath, _ := filepath.Abs(ctx.BaseDirectory) + fmt.Printf("Done! Generated sources are in %s\n", outputPath) }