[parted-devel] [patch] Use 'p' partition separator only when necessary
Jim Meyering
jim at meyering.net
Wed Dec 7 16:44:46 UTC 2011
Phillip Susi wrote:
> On 12/06/2011 04:50 PM, Jim Meyering wrote:
>> Thanks for the reminder.
>> Can you write a test to exercise that patch, now?
>>
>> IMHO, any significant bug-fix or change in parted
>> should be accompanied by a test suite addition that exercises it.
>
> How does this look?
Thanks for the new test.
Unfortunately, at least on my F-16 (3.1.0-1.fc16.x86_64) desktop,
it leads to creation of a hung and unkillable parted process
that is running this command:
parted -s /pe/tests/gt-t6001-psep.sh.IQaO/root/dev/mapper/plinear-25167 \
mkpart primary fat32 1m 5m
This will probably serve as impetus for me to reboot.
Current uptime is a whopping 40 days, and a newer 3.1.4 kernel awaits.
Two nits:
When I applied that using "git am FILE", it complained about the two
trailing spaces. "make syntax-check" would have spotted those, too.
> Subject: [PATCH] Add partition separator only when required
>
...
> diff --git a/tests/t6001-psep.sh b/tests/t6001-psep.sh
> new file mode 100644
> index 0000000..45cdc3c
> --- /dev/null
> +++ b/tests/t6001-psep.sh
> @@ -0,0 +1,87 @@
> +#!/bin/sh
> +# ensure that parted names partitions on dm disks correctly
> +
> +# Copyright (C) 2008-2011 Free Software Foundation, Inc.
For a new test, the copyright year should be only the current one.
# Copyright (C) 2011 Free Software Foundation, Inc.
More information about the parted-devel
mailing list