[Buildroot] [PATCH v3 1/1] yad: new package

[Vicente Olivert Riera]

Dear James Knight,

On 10/07/2015 04:46 PM, James Knight wrote:
> Vincent,
> 
> On Wed, Oct 7, 2015 at 6:08 AM, Vicente Olivert Riera
> <Vincent.Riera at imgtec.com> wrote:
>> In the next line you are going to select BR2_PACKAGE_LIBGLIB2, so you
>> need to propagate its dependencies to this package as well. That means
>> you need to add right here the following lines:
>>
>> depends on BR2_USE_WCHAR # libglib2
>> depends on BR2_TOOLCHAIN_HAS_THREADS # libglib2
>> depends on BR2_USE_MMU # libglib2
> 
> Gotcha. I'll get that fixed.
> 
> On this point, is the common practice to copy the `depends` of
> packages you `select` on (ex. for this yad package, if I select on
> `BR2_PACKAGE_LIBGLIB2`, I then check the libglib2 package and copy the
> `depends` references it has and add the command "libglib2" to each
> line)?

yes, that's mandatory. When you select a package, you have to propagate
its dependencies to the package which is selecting it. Otherwise, that
package could select the other one despite of not satisfying the
dependencies. For instance, in your case, if you select the "yad"
package, it will automatically select the "libglib2" package. And that
could be done even if your toolchain doesn't have threads support. Then,
buildroot will try to build "libglib2" and it will fail.

A package shouldn't be able to select another one if the dependencies of
the other package are not satisfied.

>> The word "from" still fits in the first line as it will not exceed the
>> 72 characters limit.
> 
> Does it? For myself I have three (3) characters left after "dialogs"
> (4+2+69+3). Is a four (4) character tab not the expected length or is
> it something else? Curious if I've been messing up all my Config.in
> files. :o

Note that the tab counts as only one character, despite of the
configuration of your text editor which displays the tabs as 4 spaces.
If you use vim, is worth to have these options in the ~/.vimrc file when
you write commit logs:

set textwidth=72
set wrap

This will wrap lines automatically when you reach the 72 characters limit.

>>> +
>>> +       http://sourceforge.net/projects/yad-dialog/
>>> +
>>> +comment "yad needs libgtk2 or libgtk3"
>>> +     depends on !BR2_PACKAGE_LIBGTK2 && !BR2_PACKAGE_LIBGTK3
>>
>> And you also need to update this comment since now you have the libglib2
>> dependencies as well. Something like this would be ok:
>>
>> comment "yad needs libgtk2 or libgtk3 and a toolchain w/ wchar, threads"
>>   depends on BR2_USE_MMU
>>   depends on !BR2_USE_WCHAR || !BR2_TOOLCHAIN_HAS_THREADS || \
>>     (!BR2_PACKAGE_LIBGTK2 && !BR2_PACKAGE_LIBGTK3)
> 
> Will do. Thanks.
> 
>> Be careful if you copy-paste the above text. You have to use tabs for
>> indentation instead of spaces :-)
> 
> Always paranoid about this; unfortunately my paranoia doesn't catch it
> all the time. :(

hehe, hopefully we will catch it during the review process :P

Regards,

Vincent.