[Buildroot] [PATCH] package/dvdrw-tools: fix incorrect default choiche

Yann E. MORIN yann.morin.1998 at free.fr
Wed Mar 18 22:41:11 UTC 2015


Angelo, All,

On 2015-03-17 23:28 +0100, Angelo Compagnucci spake thusly:
> This pathc fixes default choiche BR2_PACKAGE_DVDRW_TOOLS_CDRKIT_BACKEND.
> 
> Signed-off-by: Angelo Compagnucci <angelo.compagnucci at gmail.com>
> ---
>  package/dvdrw-tools/Config.in | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/package/dvdrw-tools/Config.in b/package/dvdrw-tools/Config.in
> index 6787b66..fb4d4c2 100644
> --- a/package/dvdrw-tools/Config.in
> +++ b/package/dvdrw-tools/Config.in
> @@ -35,7 +35,7 @@ config BR2_PACKAGE_DVDRW_TOOLS
>  if BR2_PACKAGE_DVDRW_TOOLS
>  choice
>  	prompt "Media Backend"
> -	default BR2_PACKAGE_DVDRW_CDRKIT_BACKEND
> +	default BR2_PACKAGE_DVDRW_TOOLS_CDRKIT_BACKEND

In fact that's not needed since cdrkit is the first entry.

Besides, and I missed it durign the review too, we can't force cdrkit as
the default sicne it has a dependency which is not reported here; it
should have been (options elided for readability):

    choice
        bool "backend"
        default CDRKIT if HEADERS_AT_LEAST_3_0

But really, there's no reason to have a default in the first place, so I
suggest we just get rid of it:

  - if kernel headers >= 3.0, cdrkit will be used by default as it is
    the first entry in the choice;

  - otherwise, cdrkit will be unavailable, and xorriso will be used.

Regards,
Yann E. MORIN.

>  	help
>  	  Choose which media backend program to use.
>  
> -- 
> 1.9.1
> 
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'


More information about the buildroot mailing list