[10:00] jodh: there is a typo in the failed case. [10:00] jodh: the filename will not be defined :P [10:02] xnox: need a few more bytes of context - we're talking about lp:~xnox/upstart/fix-qemu-test presumably? [10:02] jodh: yes, mere re-reproposal is on the way. [10:02] jodh: https://code.launchpad.net/~xnox/upstart/fix-qemu-test/+merge/137517 [10:03] s/mere/the/ [10:04] xnox: ah! I'd missed that. Could you also therefore wrap the unlink in assert0() like: 'assert0 (unlink (filename))' [10:05] jodh: no, since it may not exist. [10:05] cause it's in the maybe-failed code path. [10:08] xnox: gah - agreed :) [10:09] xnox: btw - I raised bug 1085836 just now. It would be great to have this facility for Raring, particularly for the DEP-8 tests. We can discuss with stgraber later. [10:13] jodh: interesting. /me ponders if we can re-exec ourselfs under sudo for a test wrapped in NEEDS_ROOT [14:09] xnox, jodh: one option would be to have a separate "root-check" target or similar which requires root permissions, those wouldn't be called at build time but could be called under sudo on the autopackagetest machine [14:28] stgraber: right. [15:12] jodh: https://launchpad.net/~xnox/+archive/scratch/+build/4031890 [15:13] jodh: stgraber: https://buildd.debian.org/status/fetch.php?pkg=upstart&arch=amd64&ver=1.6-3&stamp=1353265111 [15:14] Kernel: Linux 2.6.32-5-amd64 amd64 (x86_64)