[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH,HURD] Fix grub-probe with userland partition support
From: |
Vladimir 'φ-coder/phcoder' Serbinenko |
Subject: |
Re: [PATCH,HURD] Fix grub-probe with userland partition support |
Date: |
Tue, 24 Apr 2012 11:12:11 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:10.0.3) Gecko/20120329 Icedove/10.0.3 |
On 24.04.2012 11:00, Samuel Thibault wrote:
> Vladimir 'φ-coder/phcoder' Serbinenko, le Tue 24 Apr 2012 10:55:21 +0200, a
> écrit :
>> On 24.04.2012 01:34, Samuel Thibault wrote:
>>> + if (errno == ENOENT && part >= 5)
>>> + {
>> This introduces an assumption that if part >=5 is missing then there are
>> no additional partitions. While this is true for msdos, it's false for
>> GPT, where such config is common after deleting a partition. Moreover,
>> APM (apple) partitions often start at 8.
> Ok, but what do you suggest? By the very nature of parted-based
> translators, there is no way to know how many partitions there are.
Can we know whether given device is partition? If we know that it is we
can iterate as long as we can, adding some restriction like your 1000
and issuing error if it's hit since it should never happen.
--
Regards
Vladimir 'φ-coder/phcoder' Serbinenko
signature.asc
Description: OpenPGP digital signature
- [PATCH,HURD] Fix grub-probe with userland partition support, Samuel Thibault, 2012/04/22
- Re: [PATCH,HURD] Fix grub-probe with userland partition support, Vladimir 'φ-coder/phcoder' Serbinenko, 2012/04/23
- Re: [PATCH,HURD] Fix grub-probe with userland partition support, Samuel Thibault, 2012/04/23
- Re: [PATCH,HURD] Fix grub-probe with userland partition support, Vladimir 'φ-coder/phcoder' Serbinenko, 2012/04/23
- Re: [PATCH,HURD] Fix grub-probe with userland partition support, Samuel Thibault, 2012/04/23
- Re: [PATCH,HURD] Fix grub-probe with userland partition support, Vladimir 'φ-coder/phcoder' Serbinenko, 2012/04/23
- Re: [PATCH,HURD] Fix grub-probe with userland partition support, Samuel Thibault, 2012/04/23
- Re: [PATCH,HURD] Fix grub-probe with userland partition support, Samuel Thibault, 2012/04/23
- Re: [PATCH,HURD] Fix grub-probe with userland partition support, Vladimir 'φ-coder/phcoder' Serbinenko, 2012/04/24
- Re: [PATCH,HURD] Fix grub-probe with userland partition support, Samuel Thibault, 2012/04/24
- Re: [PATCH,HURD] Fix grub-probe with userland partition support,
Vladimir 'φ-coder/phcoder' Serbinenko <=
- Re: [PATCH,HURD] Fix grub-probe with userland partition support, Samuel Thibault, 2012/04/24
- Re: [PATCH,HURD] Fix grub-probe with userland partition support, Vladimir 'φ-coder/phcoder' Serbinenko, 2012/04/24
- Re: [PATCH,HURD] Fix grub-probe with userland partition support, Samuel Thibault, 2012/04/24
- Re: [PATCH,HURD] Fix grub-probe with userland partition support, Samuel Thibault, 2012/04/24
- Re: [PATCH,HURD] Fix grub-probe with userland partition support, Vladimir 'φ-coder/phcoder' Serbinenko, 2012/04/24
- Re: [PATCH,HURD] Fix grub-probe with userland partition support, Samuel Thibault, 2012/04/24
- Re: [PATCH,HURD] Fix grub-probe with userland partition support, Vladimir 'φ-coder/phcoder' Serbinenko, 2012/04/24
- Re: [PATCH,HURD] Fix grub-probe with userland partition support, Samuel Thibault, 2012/04/24
- Re: [PATCH,HURD] Fix grub-probe with userland partition support, Vladimir 'φ-coder/phcoder' Serbinenko, 2012/04/24
- Re: [PATCH,HURD] Fix grub-probe with userland partition support, Samuel Thibault, 2012/04/24