Re: Possible enumeration approach

[Joel Granados <jgranado redhat com>]

On Fri, May 22, 2009 at 09:35:34AM -1000, David Cantrell wrote:
> On 05/22/2009 09:27 AM, Chris Lumens wrote:
>>> This is a hack that hans proposed for the parted issue.  Don't see this
>>> causing any problems.
>>> clumens: what do you think of this for a temporary solution for the
>>> enumeration issue?
>>
>> Applying this patch to parted in package CVS, adding the following patch
>> to pyparted, rebuilding both, and putting both in an updates.img fixes
>> the problem for me in my simple test case.
>>
>> diff --git a/src/pydisk.c b/src/pydisk.c
>> index 82500e5..84dcbd4 100644
>> --- a/src/pydisk.c
>> +++ b/src/pydisk.c
>> @@ -1403,6 +1403,9 @@ PyObject *py_ped_disk_remove_partition(PyObject
>> *s, PyObject *args) {
>>           }
>>       }
>>
>> +    if (ped_disk_enumerate_on())
>> +        ped_disk_toggle_do_enumerate();
>> +
>>       ret = ped_disk_remove_partition(disk, out_part);
>>       if (ret == 0) {
>>           if (partedExnRaised) {
>>
>> My simple test case is a partition layout with multiple logical
>> partitions.  Having that as a pre-existing layout and then doing
>> autopart consistently fails for me.  These patches correct that.
>
> The fix seems reasonable to me.  Existing API functionality is/can-be  
> preserved in libparted.
>
> In pyparted, I suppose we should also expose  
> ped_disk_toggle_do_enumerate(), but that's not critical for now.
>
> The libparted patch has my vote.

ok.  I'll correct the patch (take out the parted/parted.c parts) and
build a new parted.

Regards.

>
> -- 
> David Cantrell <dcantrell redhat com>
> Red Hat / Honolulu, HI

-- 
Joel Andres Granados
Brno, Czech Republic, Red Hat.