diff options
author | Brian Horton <brianh@linux.ibm.com> | 2013-10-15 15:34:34 -0500 |
---|---|---|
committer | A. Patrick Williams III <iawillia@us.ibm.com> | 2013-10-17 11:30:25 -0500 |
commit | a3a453958b4eee7d0fc0f9dd295a2176bbe7b6ba (patch) | |
tree | e55f5cbd7b020a21a9bcd64ba639513c277c8306 | |
parent | e12151bb75d0a64d92d1c9ebd9c43a001afcca8e (diff) | |
download | talos-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-x | src/build/hwpf/prcd_compile.tcl | 6 | ||||
-rwxr-xr-x | src/build/hwpf/prcd_server.tcl | 12 |
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 { |