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

Re: r56960 - in trunk/packages/partman/partman-base: choose_partition/partition_tree debian



On Fri, Dec 26, 2008 at 02:19:16PM +0000, Colin Watson wrote:
> Log:
> Disable backup while displaying device/partition locked errors; it
> makes no sense and it can cause us to exit without closing the FIFO to
> parted_server (closes: #505477, LP: #274219).
> […] 
> Modified: trunk/packages/partman/partman-base/choose_partition/partition_tree/do_option
> ==============================================================================
> --- trunk/packages/partman/partman-base/choose_partition/partition_tree/do_option	(original)
> +++ trunk/packages/partman/partman-base/choose_partition/partition_tree/do_option	Fri Dec 26 14:19:16 2008
> @@ -18,7 +18,9 @@
>  	db_subst partman-base/devicelocked MESSAGE "$locked"
>  	db_set partman-base/devicelocked false
>  	db_input critical partman-base/devicelocked
> -	db_go
> +	db_capb
> +	db_go || true
> +	db_capb backup

This should probably be "backup align" instead.  Otherwise it will break
the columned output.

>  	exit 0
>  fi
>  
> @@ -38,7 +40,9 @@
>  		db_subst partman-base/partlocked MESSAGE "$locked"
>  		db_set partman-base/partlocked false
>  		db_input critical partman-base/partlocked
> -		db_go
> +		db_capb
> +		db_go || true
> +		db_capb backup

Same here.

>  		close_dialog
>  		exit 0
>  	fi

Cheers,
-- 
Jérémy Bobbio                        .''`. 
lunar@debian.org                    : :Ⓐ  :  # apt-get install anarchism
                                    `. `'` 
                                      `-   

Attachment: signature.asc
Description: Digital signature


Reply to: