[PATCH imake 1/4] Allow for EXEEXT in appman_DATA rule

Alan Coopersmith alan.coopersmith at oracle.com
Tue Apr 27 17:56:54 PDT 2010


Yaakov (Cygwin/X) wrote:
> From: Yaakov Selkowitz <yselkowitz at users.sourceforge.net>
> 
> As a result of commit 22578c3718e41da72c66ce6a4186ec7315d6e3ad:
> 
> make[1]: *** No rule to make target `imake.exe.1', needed by `all-am'.
> make[1]: *** No rule to make target `revpath.exe.1', needed by `all-am'.
> 
> On PE/COFF platforms EXEEXT is set to ".exe" and is appended by automake
> to all *_PROGRAMS.  For portability this must be accounted for in any
> custom rules which refer to any *_PROGRAMS.
> 
> Signed-off-by: Yaakov Selkowitz <yselkowitz at users.sourceforge.net>
> ---
>  Makefile.am |    3 +--
>  1 files changed, 1 insertions(+), 2 deletions(-)
> 
> diff --git a/Makefile.am b/Makefile.am
> index 1fe351d..d7048a3 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -125,8 +125,7 @@ appman_PRE = \
>  	mkhtmlindex.man
>  
>  # Only need to install man pages for programs/scripts being installed
> -appman_needed_for = $(bin_PROGRAMS) $(bin_SCRIPTS)
> -appman_DATA = $(appman_needed_for:%=%. at APP_MAN_SUFFIX@)
> +appman_DATA = $(bin_PROGRAMS:%$(EXEEXT)=%. at APP_MAN_SUFFIX@) $(bin_SCRIPTS:%=%. at APP_MAN_SUFFIX@)
>  
>  EXTRA_DIST += $(appman_PRE)
>  CLEANFILES += $(appman_DATA)

Oops, sorry about that.
Reviewed-by: Alan Coopersmith <alan.coopersmith at oracle.com>

-- 
	-Alan Coopersmith-        alan.coopersmith at oracle.com
	 Oracle Solaris Platform Engineering: X Window System



More information about the xorg-devel mailing list