[PATCH libinput] tools: ship the event-debug tool as an installed libinput-debug-events tool

[Hans de Goede]

Hi,

On 30-04-15 10:28, Peter Hutterer wrote:
> Rebuild the same binary but without the special LDFLAG. The event-debug tool
> is left as-is to allow for easy debugging with gdb, the new tool is now
> libtool-enabled and can't be run directly in gdb without installing it first.
>
> Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>

Looks good to me:

Reviewed-by: Hans de Goede <hdegoede at redhat.com>

Regards,

Hans


> ---
> I find myself asking more bugreporters to git clone and run the debug tool,
> it'd be easier in some cases for them to run it directly. So we might as
> well install it, but I like not having to work around libtool, hence the
> double-build.
>
>   tools/.gitignore                |  1 +
>   tools/Makefile.am               |  7 ++++++-
>   tools/libinput-debug-events.man | 31 +++++++++++++++++++++++++++++++
>   3 files changed, 38 insertions(+), 1 deletion(-)
>   create mode 100644 tools/libinput-debug-events.man
>
> diff --git a/tools/.gitignore b/tools/.gitignore
> index e58dba9..cb93429 100644
> --- a/tools/.gitignore
> +++ b/tools/.gitignore
> @@ -2,3 +2,4 @@ event-debug
>   event-gui
>   ptraccel-debug
>   libinput-list-devices
> +libinput-debug-events
> diff --git a/tools/Makefile.am b/tools/Makefile.am
> index 792f310..6fb797f 100644
> --- a/tools/Makefile.am
> +++ b/tools/Makefile.am
> @@ -1,5 +1,5 @@
>   noinst_PROGRAMS = event-debug ptraccel-debug
> -bin_PROGRAMS = libinput-list-devices
> +bin_PROGRAMS = libinput-list-devices libinput-debug-events
>   noinst_LTLIBRARIES = libshared.la
>
>   AM_CPPFLAGS = -I$(top_srcdir)/include \
> @@ -26,6 +26,11 @@ libinput_list_devices_LDADD = ../src/libinput.la libshared.la $(LIBUDEV_LIBS)
>   libinput_list_devices_CFLAGS = $(LIBUDEV_CFLAGS)
>   dist_man1_MANS = libinput-list-devices.man
>
> +libinput_debug_events_SOURCES = $(event_debug_SOURCES)
> +libinput_debug_events_LDADD = $(event_debug_LDADD)
> +libinput_debug_events_CFLAGS = $(event_debug_CFLAGS)
> +dist_man1_MANS = libinput-debug-events.man
> +
>   if BUILD_EVENTGUI
>   noinst_PROGRAMS += event-gui
>
> diff --git a/tools/libinput-debug-events.man b/tools/libinput-debug-events.man
> new file mode 100644
> index 0000000..bb39399
> --- /dev/null
> +++ b/tools/libinput-debug-events.man
> @@ -0,0 +1,31 @@
> +.TH LIBINPUT-DEBUG-EVENTS "1"
> +.SH NAME
> +libinput-debug-events \- debug helper for libinput
> +.SH SYNOPSIS
> +.B libinput-debug-events [--help]
> +.SH DESCRIPTION
> +.PP
> +The
> +.I libinput-debug-events
> +tool creates a libinput context and prints all events from these devices.
> +.PP
> +This is a debugging tool only, its output may change at any time. Do not
> +rely on the output.
> +.PP
> +This tool usually needs to be run as root to have access to the
> +/dev/input/eventX nodes.
> +.SH OPTIONS
> +.TP 8
> +.B --help
> +Print help
> +.PP
> +For all other options, see the output from --help. Options may be added or
> +removed at any time.
> +.SH NOTES
> +.PP
> +Events shown by this tool may not correspond to the events seen by a
> +different user of libinput. This tool initializes a separate context.
> +.PP
> +Events shown by this tool include key codes in plain text. Anything you type
> +while this tool is running will show up in the output, including your
> +passwords.
>