* cfg and adt-testreport-runloop: do not cd in config files.

diff --git a/debian/changelog b/debian/changelog
index c17479ca809a74d15a9144d30806f2e79caa38f1..7fa8c88e9311b585191941c0c2522dd675f38def 100644 (file)
--- a/debian/changelog
+++ b/debian/changelog
@@ -12,6 +12,7 @@ autopkgtest (1.1.0~) unstable; urgency=low
   * hosts/* installed as examples with a big fat warning.
   * adt-testreport-cronjob: renamed from hosts/cadmium/cronjob
   * adt-testreport-runloop: copyright message.
+  * cfg and adt-testreport-runloop: do not cd in config files.
   * hosts/cadmium/*, debian/rules: onepackage_config renamed to cfg.
 
  -- Ian Jackson <ian@davenant.greenend.org.uk>  Thu, 18 Oct 2007 17:17:31 +0100

diff --git a/hosts/cadmium/cfg b/hosts/cadmium/cfg
index 08f012c11d01ed880371c3d1ccbc518b427eb561..4da106e9bc9d9f5df0b346afc64318b4e17c3fc8 100644 (file)
--- a/hosts/cadmium/cfg
+++ b/hosts/cadmium/cfg
@@ -1,4 +1,3 @@
-cd /home/iwj/adt-play
 PATH=/home/iwj/autopkgtest/runner:/usr/local/sbin:/usr/sbin:$PATH
 export PATH
 

diff --git a/hosts/cadmium/crontab b/hosts/cadmium/crontab
index 7afdce321cf8d1d5f18f779b8bad07e2894bb494..cee9cf0a3561fcf5c8effbcd1b5cc23b41364a9c 100644 (file)
--- a/hosts/cadmium/crontab
+++ b/hosts/cadmium/crontab
@@ -1,4 +1,4 @@
 # m h dom mon dow       command
-4-59/5 * * * *         autopkgtest/runner/adt-testreport-cronjob adt-play/cfg
+4-59/5 * * * *         autopkgtest/runner/adt-testreport-cronjob adt-play cfg
 22 0 * * Mon           savelog -n7 adt-play/var/log adt-play/var/overall.log
 58 7 * * *             adt-play/cleanup

diff --git a/runner/adt-testreport-cronjob b/runner/adt-testreport-cronjob
index 9d3576094c351b8ec9f7b39064960a59c9504e49..833753a550c6ac1c9b92035c86048f944a1e789f 100755 (executable)
--- a/runner/adt-testreport-cronjob
+++ b/runner/adt-testreport-cronjob
@@ -1,14 +1,15 @@
 #!/bin/bash
 #
-#  This script expects to be given one argument, the name
-#  of the config file which will set the PATH if necessary
-#  and cd to the appropriate playground.  This and all
-#  remaining arguments are passed to a-t-r.
+#  This script expects to be given at least two arguments:
+#     directory to cd to
+#     main config file for adt-testreport-runloop which sets PATH if need be
+#     remaining arguments are passed through to a-t-r unchanged
 
 set -e
+cd "$1"
+shift
 . "$1"
-
 exec >>var/overall.log 2>&1
 test -f go
 with-lock-ex -q cronjob-lock \
- adt-testreport-runloop "$@" max_test_count=100
+ adt-testreport-runloop max_test_count=100 "$@"

diff --git a/runner/adt-testreport-runloop.1 b/runner/adt-testreport-runloop.1
index 9536c7f8e4be7f9d4ce1a6569875b075c264e4b8..d3a64b0f13513b8eb382dc6efd3faa6aea7ade30 100644 (file)
--- a/runner/adt-testreport-runloop.1
+++ b/runner/adt-testreport-runloop.1
@@ -30,8 +30,8 @@ manpage for such information about its configuration as is available.
 Configuration fragment authors should bear in mind that both programs
 use \fBset -e\fR.
 
-It can be a good idea to \fBcd\fR, and set PATH, in the configuration
-file.
+It is quite permissible to set PATH and other needed environmnet
+variables in the configuration file.
 
 The following variables may be set:
 .TP