[PATCH] Add EXEEXT to gendis

[PATCH] Add EXEEXT to gendis

[Ulf Hermann]

Otherwise the build will fail on systems that actually need file
extension for executables.

Signed-off-by: Ulf Hermann <ulf.hermann@qt.io>
---
 libcpu/ChangeLog   | 4 ++++
 libcpu/Makefile.am | 6 +++---
 2 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/libcpu/ChangeLog b/libcpu/ChangeLog
index e59e876..7fe4345 100644
--- a/libcpu/ChangeLog
+++ b/libcpu/ChangeLog
@@ -1,5 +1,9 @@
 2017-04-20  Ulf Hermann  <ulf.hermann@qt.io>
 
+	* Makefile.am: Add EXEEXT to gendis.
+
+2017-04-20  Ulf Hermann  <ulf.hermann@qt.io>
+
 	* i386_parse.y: Eliminate comparison_fn_t.
 
 2016-11-02  Mark Wielaard  <mjw@redhat.com>
diff --git a/libcpu/Makefile.am b/libcpu/Makefile.am
index b98b583..31fc906 100644
--- a/libcpu/Makefile.am
+++ b/libcpu/Makefile.am
@@ -58,10 +58,10 @@ endif
 if MAINTAINER_MODE
 noinst_HEADERS = memory-access.h i386_parse.h i386_data.h
 
-noinst_PROGRAMS = i386_gendis
+noinst_PROGRAMS = i386_gendis$(EXEEXT)
 
-$(srcdir)/%_dis.h: %_defs i386_gendis
-	$(AM_V_GEN)./i386_gendis $< > $@T
+$(srcdir)/%_dis.h: %_defs i386_gendis$(EXEEXT)
+	$(AM_V_GEN)./i386_gendis$(EXEEXT) $< > $@T
 	$(AM_V_at)mv -f $@T $@
 
 else
-- 
2.1.4

Re: [PATCH] Add EXEEXT to gendis

[Mark Wielaard]

On Thu, 2017-04-20 at 17:02 +0200, Ulf Hermann wrote:
> Otherwise the build will fail on systems that actually need file
> extension for executables.

Thanks, pushed to master.
We seem to add it in other cases. Surprising this seems to be the only
case that got forgotten. I would have expected more issues.

Cheers,

Mark