[parted-devel] [patch] Use 'p' partition separator only when necessary

Jim Meyering jim at meyering.net
Mon Dec 12 13:54:54 UTC 2011


Jim Meyering wrote:

> Phillip Susi wrote:
>> On 12/07/2011 11:44 AM, Jim Meyering wrote:
>>> 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.
>>
>> I had the same thing happen to me when running the test suite on your
>> master branch.  I tried to reproduce it, but couldn't.  There seems to
>> be a bug in the kernel somewhere.  I'm running Ubuntu's 3.2-rc3
>> kernel.
>>
>>> 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.
>>
>> Damn... I could have sworn I got those using whitespace mode in emacs.
>> Oh, I see, I thought those two spaces were required but I guess not.
>>
>> How's this one:
>
> Thanks for adjusting.
> I've confirmed that with the latest F16 kernel (3.1.5-1.fc16.x86_64),
> your new test passes:
>
>   sudo make -C tests check TESTS=t6001-psep.sh VERBOSE=yes
>
> so I've pushed it.
> Thanks again.

I've just realized that your change did not include a NEWS entry.
Would you please write one more patch to do that that?

Sorry I didn't notice it before pushing.



More information about the parted-devel mailing list