[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[OSSTEST PATCH v2 12/17] cri-args-hostlists: Move flight_html_dir variable



This is only used in report_flight.  We are going to want to call
report_flight from outside start_email, without having to set that
variable ourselves.

The variable isn't actually used in start_email.

Signed-off-by: Ian Jackson <iwj@xxxxxxxxxxxxxx>
---
 cri-args-hostlists | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cri-args-hostlists b/cri-args-hostlists
index 52e39f33..52cac195 100644
--- a/cri-args-hostlists
+++ b/cri-args-hostlists
@@ -113,7 +113,6 @@ start_email () {
        printf '%s\n' "`getconfig EmailStdHeaders`"
        printf 'Subject: %s' "${subject_prefix:-[$branch test] }"
 
-       local flight_html_dir=$OSSTEST_HTMLPUB_DIR/
        local job_html_dir=$OSSTEST_HTML_DIR/
        local host_html_dir=$OSSTEST_HTML_DIR/host/
 
@@ -143,6 +142,7 @@ start_email () {
 
 report_flight () {
        local flight=$1
+       local flight_html_dir=$OSSTEST_HTMLPUB_DIR/
        ./sg-report-flight --html-dir=$flight_html_dir/$flight/ \
                --allow=allow.all --allow=allow.$branch \
                $sgr_args $flight >tmp/$flight.report
-- 
2.20.1




 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.