The Xorg binary is missing the extension $(EXEEXT) in the makefile
authorGaetan Nadon <memsize@videotron.ca>
Sat, 28 Sep 2013 19:26:33 +0000 (15:26 -0400)
committerKeith Packard <keithp@keithp.com>
Fri, 4 Oct 2013 21:09:26 +0000 (14:09 -0700)
This is not a problem on UNIX platforms, but on CYGWIN it creates a broken
link to Xorg rather than a link to Xorg.exe.

From the CYGWIN log on tinderbox, we can see that the executable Xorg.exe is
installed correctly. We can see the command used to create the link:

(cd /jhbuild/install/[...]/install/bin && rm -f X && ln -s Xorg X)

Note that the "relink" makefile target correctly appends $(EXEEXT) to Xorg.

Reviewed-By: Matt Dew <marcoz@osource.org>
Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
Signed-off-by: Keith Packard <keithp@keithp.com>
hw/xfree86/Makefile.am

index d568e0d..6b66732 100644 (file)
@@ -90,7 +90,7 @@ install-data-local:
 
 
 install-exec-local: install-binPROGRAMS
-       (cd $(DESTDIR)$(bindir) && rm -f X && ln -s Xorg X)
+       (cd $(DESTDIR)$(bindir) && rm -f X && ln -s Xorg$(EXEEXT) X)
 if INSTALL_SETUID
        chown root $(DESTDIR)$(bindir)/Xorg
        chmod u+s $(DESTDIR)$(bindir)/Xorg