[v5,1/1] yad: new package

Signed-off-by: James Knight <james.knight@rockwellcollins.com>
---
Changes v4 -> v5:
  - Increment to version 0.31.2.
  - Cleanup yad menu option to be visible after Xorg/MMU is selected. 

Changes v3 -> v4:
  - Add missing libglib2 dependencies (pointed out by Vicente Olivert
     Riera).
  - Cleanup YAD help description to maximize the 72-character limit
     space (suggested by Vicente Olivert Riera).

Changes v2 -> v3:
  - Increment to version 0.31.1 (removed need for HTML patch introduced
     in v2 change; suggested by Vicente Olivert Riera).

Changes v1 -> v2:
  - Increment to version 0.29.0 (with updated hash and patch fix).
  - Removed complex gtk library selection. Defaults to libgtk3, if
     available; elsewise, uses libgtk2 (suggested from comments by
     Baruch Siach, Thomas Petazzoni and Peter Korsgaard).
---
 package/Config.in     |  1 +
 package/yad/Config.in | 17 +++++++++++++++++
 package/yad/yad.hash  |  2 ++
 package/yad/yad.mk    | 24 ++++++++++++++++++++++++
 4 files changed, 44 insertions(+)
 create mode 100644 package/yad/Config.in
 create mode 100644 package/yad/yad.hash
 create mode 100644 package/yad/yad.mk

Dear James Knight,

On 10/24/2015 12:10 AM, James Knight wrote:
> Signed-off-by: James Knight <james.knight@rockwellcollins.com>
[snip]
> diff --git a/package/yad/Config.in b/package/yad/Config.in
> new file mode 100644
> index 0000000..9b16729
> --- /dev/null
> +++ b/package/yad/Config.in
> @@ -0,0 +1,17 @@
> +config BR2_PACKAGE_YAD
> +	bool "yad"
> +	depends on BR2_PACKAGE_XORG7
> +	depends on BR2_PACKAGE_LIBGTK2 || BR2_PACKAGE_LIBGTK3
> +	depends on BR2_USE_WCHAR # libglib2
> +	depends on BR2_TOOLCHAIN_HAS_THREADS # libglib2
> +	depends on BR2_USE_MMU # libglib2
> +	select BR2_PACKAGE_LIBGLIB2
> +	help
> +	  YAD (yet another dialog) is a tool for create graphical dialogs from

That line exceeds the 72 characters limit considering 8-characters-wide
tabs.

Apart from that minor detail, which I guess after 5 versions of your
patch the maintainer will not mind to fix it for you, your patch looks
good to me.

Reviewed-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com>
Tested-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com>

Build test for MIPS architecture:

$ file output/target/usr/bin/yad
output/target/usr/bin/yad: ELF 32-bit LSB executable, MIPS, MIPS32 rel2
version 1 (SYSV), dynamically linked (uses shared libs), for GNU/Linux
2.6.32, with unknown capability 0xf41 = 0x756e6700, with unknown
capability 0x70100 = 0x1040000, stripped


Regards,

Vincent.

Vincent,

First off, thanks for reviewing/testing (as always).

On Tue, Oct 27, 2015 at 10:41 AM, Vicente Olivert Riera
<Vincent.Riera@imgtec.com> wrote:
>> +     help
>> +       YAD (yet another dialog) is a tool for create graphical dialogs from
>
> That line exceeds the 72 characters limit considering 8-characters-wide
> tabs.

Sorry, I'm confused. I only moved the word "from" into the lined based
off your previous suggestion:

On Wed, Oct 7, 2015 at 6:08 AM, Vicente Olivert Riera
<Vincent.Riera@imgtec.com> wrote:
>> +     select BR2_PACKAGE_LIBGLIB2
>> +     help
>> +       YAD (yet another dialog) is a tool for create graphical dialogs
>> +       from shell scripts.
>
> The word "from" still fits in the first line as it will not exceed the
> 72 characters limit.

Am I missing something?

Dear James Knight,

On 10/28/2015 09:34 PM, James Knight wrote:
> Vincent,
> 
> First off, thanks for reviewing/testing (as always).
> 
> On Tue, Oct 27, 2015 at 10:41 AM, Vicente Olivert Riera
> <Vincent.Riera@imgtec.com> wrote:
>>> +     help
>>> +       YAD (yet another dialog) is a tool for create graphical dialogs from
>>
>> That line exceeds the 72 characters limit considering 8-characters-wide
>> tabs.
> 
> Sorry, I'm confused. I only moved the word "from" into the lined based
> off your previous suggestion:

You are totally right and I apologize for that. This is totally my
fault. It seems that in the Buildroot project tabs are considered as 8
characters wide. I didn't know that when I made that suggestion to you,
but in my defense I have to say that is not documented anywhere.

Regards,

Vincent.

> On Wed, Oct 7, 2015 at 6:08 AM, Vicente Olivert Riera
> <Vincent.Riera@imgtec.com> wrote:
>>> +     select BR2_PACKAGE_LIBGLIB2
>>> +     help
>>> +       YAD (yet another dialog) is a tool for create graphical dialogs
>>> +       from shell scripts.
>>
>> The word "from" still fits in the first line as it will not exceed the
>> 72 characters limit.
> 
> Am I missing something?
>