[Buildroot] [PATCH v2 1/1] linuxconsoletools: new package

[Thomas Petazzoni <thomas.petazzoni@free-electrons.com>]

Hello,

On Mon, 19 Jun 2017 18:55:06 +0200, Koen Martens wrote:
> Linuxconsoletools contains the inputattach utility
> to attach legacy serial devices to the Linux kernel
> input layer and joystick utilities to calibrate and
> test joysticks and joypads.
> 
> The buildroot package adds options to build only certain
> tools.
> 
> website: http://sf.net/projects/linuxconsole/
> 
> Signed-off-by: Koen Martens <gmc@sonologic.nl>

Thanks, I've applied, after doing a few minor tweaks. See below.

> diff --git a/package/linuxconsoletools/Config.in b/package/linuxconsoletools/Config.in
> new file mode 100644
> index 0000000..08a4af3
> --- /dev/null
> +++ b/package/linuxconsoletools/Config.in
> @@ -0,0 +1,34 @@
> +config BR2_PACKAGE_LINUXCONSOLETOOLS
> +	bool "linuxconsoletools"
> +	default n

There was still a useless "default n" here, so I've dropped.

> +	help
> +	  Linuxconsoletools contains the inputattach utility
> +	  to attach legacy serial devices to the Linux kernel
> +	  input layer and joystick utilities to calibrate and
> +	  test joysticks and joypads.
> +	  

There was a trailing tab character on this line, which I dropped.

> +	  http://sf.net/projects/linuxconsole/
> +ifeq ($(BR2_PACKAGE_LINUXCONSOLETOOLS_FORCEFEEDBACK),y)
> +LINUXCONSOLETOOLS_MAKE_OPTS += ENABLE_FORCEFEEDBACK=1
> +LINUXCONSOLETOOLS_MAKE_OPTS += SDL_CONFIG=$(STAGING_DIR)/usr/bin/sdl-config

I've put these in a single assignment:

+LINUXCONSOLETOOLS_MAKE_OPTS += \
+       ENABLE_FORCEFEEDBACK=1 \
+       SDL_CONFIG=$(STAGING_DIR)/usr/bin/sdl-config


> +LINUXCONSOLETOOLS_DEPENDENCIES += sdl
> +endif
> +
> +LINUXCONSOLETOOLS_MAKE_OPTS += CC=$(TARGET_CC)
> +LINUXCONSOLETOOLS_MAKE_OPTS += CFLAGS="$(TARGET_CFLAGS)"

Both of those lines are not needed: CC and CFLAGS are already part of
$(TARGET_CONFIGURE_OPTS), which you pass in the build and install
commands.

Committed with those minor issues fixed. Thanks a lot for this
contribution!

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com