summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBrian Horton <brianh@linux.ibm.com>2013-10-15 15:34:34 -0500
committerA. Patrick Williams III <iawillia@us.ibm.com>2013-10-17 11:30:25 -0500
commita3a453958b4eee7d0fc0f9dd295a2176bbe7b6ba (patch)
treee55f5cbd7b020a21a9bcd64ba639513c277c8306
parente12151bb75d0a64d92d1c9ebd9c43a001afcca8e (diff)
downloadtalos-hostboot-a3a453958b4eee7d0fc0f9dd295a2176bbe7b6ba.tar.gz
talos-hostboot-a3a453958b4eee7d0fc0f9dd295a2176bbe7b6ba.zip
fixes and enhancements to prcd_compile.tcl
. don't create pnor directory if -O (noretrieve) option used; . add hostboot.bin to server retrieve list . add to errors checked from build . exit (instead of abort) if sandbox directory not found Change-Id: I40d5db126a1f5549d5a3b5981e9e13c24300848e RTC: 88622 Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/6711 Tested-by: Jenkins Server Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
-rwxr-xr-xsrc/build/hwpf/prcd_compile.tcl6
-rwxr-xr-xsrc/build/hwpf/prcd_server.tcl12
2 files changed, 14 insertions, 4 deletions
diff --git a/src/build/hwpf/prcd_compile.tcl b/src/build/hwpf/prcd_compile.tcl
index 61520c4dc..577b7f06b 100755
--- a/src/build/hwpf/prcd_compile.tcl
+++ b/src/build/hwpf/prcd_compile.tcl
@@ -412,8 +412,10 @@ if { ![info exists output_dir] } {
set output_dir "./"
}
-# Make the output directory
-eval {exec} "mkdir -p $output_dir/pnor"
+if { ![info exists noretrieve] } {
+ # Make the output directory
+ eval {exec} "mkdir -p $output_dir/pnor"
+}
lappend cmds ":INFO userid $userid version $version"
diff --git a/src/build/hwpf/prcd_server.tcl b/src/build/hwpf/prcd_server.tcl
index a86414608..5491886c5 100755
--- a/src/build/hwpf/prcd_server.tcl
+++ b/src/build/hwpf/prcd_server.tcl
@@ -175,6 +175,14 @@ proc AquireData { sock } {
} else {
set filename $b
}
+
+ if { ![file exists $src_path] } {
+ puts $sock "ERROR: sandbox directory not found"
+ puts $log "$sock: sandbox directory not found"
+ CloseOut $sock
+ return
+ }
+
# we can't just find -name $filename because that won't find path/filename. -wholename does
# that, but we need the $filename to not have any ./ prefix if the user included that.
# additionally, find usually outputs the file(s) separated by a newline. if there
@@ -652,7 +660,7 @@ proc SendSandboxNew { sock git_sh} {
# client. Note: Everything from stderr gets returned
##################################################################
-set explist [list {^make.*Error.*} {ERROR:.*} {error:.*} {^IfScrub..E>.*} {^Parse Error.*} {^Error.*} {^exception caught.*} ]
+set explist [list {^make.*Error.*} {.*NOT FOUND.*} {ERROR:.*} {error:.*} {^IfScrub..E>.*} {^Parse Error.*} {^Error.*} {^exception caught.*} ]
##################################################################
## This event catches the output of the sandbox when the pipe become readable
@@ -746,7 +754,7 @@ proc SendPnorFiles { sock obj_dir } {
set pnor_files {}
# Send the image files
- if {[catch {set pnor_files [glob -dir $obj_dir hostboot.header.bin hostboot_extended.bin hostboot.stage.bin]} res]} {
+ if {[catch {set pnor_files [glob -dir $obj_dir hostboot.header.bin hostboot_extended.bin hostboot.bin hostboot.stage.bin]} res]} {
puts $sock "ERROR: Needed image files not found in $obj_dir"
puts $log "$sock: Needed image files not found in $obj_dir"
} else {
OpenPOWER on IntegriCloud