[Buildroot] svn commit: trunk/buildroot/package/directfb
rep.dot.nop at gmail.com
Thu Oct 9 07:57:47 UTC 2008
On Thu, Oct 09, 2008 at 02:06:58AM -0300, Thiago A. Corrêa wrote:
>> correa> +
>> correa> +config BR2_PACKAGE_DIRECTFB_XSERVER
>> correa> + bool "build with X server backend"
>> Doesn't this need to depend on the various X implementations
>> (E.G. BR2_PACKAGE_XORG || ..XORG7 || ..TINYX ?
>I suppose it should, yes. Sorry, I didn't notice.
>Btw, we really need a better check for X. something like the linux
>kernel with BR2_HAS_X that is set on either X implementation.
>XORG for instance is an empty folder in package/xorg. When/if that
>changes, we need to revisit all packages, quite anoying.
That's the reason why we have BR2_PACKAGE_XSERVER_none; See
>> correa> +config BR2_PACKAGE_DIRECTFB_PNG
>> correa> + bool "enable PNG support"
>> correa> + default y
>> correa> + depends on BR2_PACKAGE_DIRECTFB
>> correa> + select BR2_PACKAGE_LIBPNG
>> correa> + select BR2_PACKAGE_ZLIB
>> Indentation seems wrong. Actually it's right here and wrong everywhere
>> else. Config.in uses tabs for indentation, not spaces (and help text
>> is tab+2 spaces).
>Unfortunally I can't see that using vi. Since it's not Python, I
>honestly don't see the space/tab being relevant.
It is relevant since it is the style that is perused in this project.
Try something like :match Error '\(^\s\(\s\s\)[^[:alnum:]]\|\s\+$\)'
you get the idea..
>> correa> +config BR2_PACKAGE_DIRECTFB_GIF
>> correa> + bool "enable GIF support"
Can you please strip all "enable" and "compile" off those bools. They
are highly redundant.
More information about the buildroot