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

Re: patch to parted-1.7.1



Hello,

> @@ -1181,7 +1181,7 @@ init_ui ()
>  	readline_state.in_readline = 0;
>  #endif
>  
> -#ifdef HAVE_SIGACTION
> +#ifdef HAVE_SIGINFO
>      sigset_t curr;
>      sigfillset (&curr);
>  

That makes sense indeed: what's useful here is the siginfo part.

> @@ -656,7 +656,7 @@ doggy_first_block_write:
>  
>  	/* We are now left with (strictly) less then a store block to write
>  	   to disk.  Thus, we read the block, overlay the buffer and flush.  */
> -	PED_ASSERT (count * PED_SECTOR_SIZE
> +	PED_ASSERT (count * PED_SECTOR_SIZE_DEFAULT
>  			< arch_specific->store->block_size, return 0);
>  
>  doggy_last_block_read:

They forgot to do it indeed.

> --- partedorig/parted-1.7.1/debian/patches/00list	2007-01-22 16:13:06.000000000 +0530
> +++ parted/parted-1.7.1/debian/patches/00list	2008-05-09 23:25:22.000000000 +0530
> @@ -11,7 +11,6 @@ parted-print-name
>  
>  # Will also stay debian specific, but depends on some of the above :/
>  lvm2
> -s390
>  sparc-fix-raid
>  
>  # Patch which may be obsolet.

It can not be done this way : that patch is needed for s390.  Maybe what
we can do instead is to fix the patch itself into not adding s390.c to
liblabels_la_SOURCES when OS == gnu.

Samuel


Reply to: