ACK. Thanks, Jim.
On Thu, 2007-05-24 at 17:53 +0200, Jim Meyering wrote: > It's been a while since I ran the privs-required test (t1100), > and running it again today, I found that it fails. BTW, you can run > it like this, if you have a spare spindle (or USB key or flash disk) > with a capacity of at least 38MB: > > sudo env DEVICE_TO_ERASE=/dev/sde DEVICE_TO_ERASE_SIZE=999MB > ./t1100-busy-label.sh > > The DEVICE_TO_ERASE_SIZE must be exactly the same string that > "parted YOUR_DEVICE print" reports. > > Here's the now-obvious fix: > ------------------ > > Fix typo in privs-required test setup. > * tests/test-lib.sh: Fix typo: s/\$parted/$parted_/. > > diff --git a/tests/test-lib.sh b/tests/test-lib.sh > index 59f85a4..050b967 100644 > --- a/tests/test-lib.sh > +++ b/tests/test-lib.sh > @@ -223,8 +223,8 @@ if test $skip_ = 0 && test "$erasable_device_required_" > != ''; then > if test "$DEVICE_TO_ERASE" != '' && test "$DEVICE_TO_ERASE_SIZE" != ''; > then > dev_=$DEVICE_TO_ERASE > sz=$DEVICE_TO_ERASE_SIZE > - parted_output=$($parted -s $dev_ print) || fail="no such device: $dev_" > - $parted -s $dev_ print|grep "^Disk $dev_: $sz$" \ > + parted_output=$($parted_ -s $dev_ print) || fail="no such device: $dev_" > + $parted_ -s $dev_ print|grep "^Disk $dev_: $sz$" \ > > /dev/null || fail="actual device size is not $sz" > # Try to see if $dev_ or any of its partitions is mounted. > # This is not reliable. FIXME: find a better way. > > _______________________________________________ > parted-devel mailing list > [email protected] > http://lists.alioth.debian.org/mailman/listinfo/parted-devel -- David Cantrell <[EMAIL PROTECTED]> Red Hat / Westford, MA
signature.asc
Description: This is a digitally signed message part
_______________________________________________ parted-devel mailing list [email protected] http://lists.alioth.debian.org/mailman/listinfo/parted-devel
