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@xenproject.org>

diff --git a/cri-args-hostlists b/cri-args-hostlists
index 52e39f3333666e5bb98f9a53f28fde2b00d265c7..52cac1952e556358e384305446142fcad4a6d402 100644 (file)
--- 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