bug-parted
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#16134: [PATCH 3/3] libparted:tests: Enable _implemented_disk_label f


From: Phil Susi
Subject: bug#16134: [PATCH 3/3] libparted:tests: Enable _implemented_disk_label for atari
Date: Tue, 18 Apr 2017 11:08:55 -0400
User-agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0

If the actual test fails when ss != 512, we probably shouldn't bother
calling it in the first place.  In other words, swap the order of
get_sector_size() != 512 and the test, and change from != to ==.

Also Sebastian had a point about the main partition table commit lacking
a full message.  Could you make these corrections and repost the full
series?  Also add a NEWS entry describing the new feature.

On 10/16/2016 6:19 AM, John Paul Adrian Glaubitz wrote:
> Use get_sector_size() to ensure we're testing on 512-byte
> sectors only.
> 
> Signed-off-by: John Paul Adrian Glaubitz <address@hidden>
> ---
>  libparted/tests/common.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/libparted/tests/common.c b/libparted/tests/common.c
> index 8ef3915..2be0e3a 100644
> --- a/libparted/tests/common.c
> +++ b/libparted/tests/common.c
> @@ -82,7 +82,9 @@ _implemented_disk_label (const char *label)
>  {
>          /* FIXME: these have minor problems, so skip them, temporarily.  */
>          if (STREQ (label, "amiga")) return 0;
> -        if (STREQ (label, "atari")) return 0;
> +
> +        if (STREQ (label, "atari") && get_sector_size() != 512)
> +                return 0;
>  
>          /* Not implemented yet */
>          if (STREQ (label, "aix")) return 0;
> 





reply via email to

[Prev in Thread] Current Thread [Next in Thread]