emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: "nmake info" Windows patch


From: Eli Zaretskii
Subject: Re: "nmake info" Windows patch
Date: Sat, 22 Feb 2003 11:21:55 +0200

> From: "Peter 'Luna' Runestig" <address@hidden>
> Date: 22 Feb 2003 00:14:04 +0100
> 
> "Peter 'Luna' Runestig" <address@hidden> writes:
> > Here is another thing I mentioned on bug-gnu-emacs some time ago. This
> > patch that adds the `info' target to the Windows build. Tested with
> > MSVC6/CygWin 1.3.20.
> > ftp://ftp.runestig.com/pub/emacs/emacs-20030219-makeinfo.patch
> > http://ftp.runestig.com/pub/emacs/emacs-20030219-makeinfo.patch
> 
> By request, I'm actually posting the patch here. I take it it's the
> preferred way in this forum then. You never now, sometimes people bitch
> over a sig > four lines...
> [...]
> +SHELL = cmd
> [...]
> +MAKEINFO = makeinfo
> +TEXI2DVI = texi2dvi
> +DVIPS = dvips
> +ENVADD = TEXINPUTS="$(srcdir):$(TEXINPUTS)" MAKEINFO="$(MAKEINFO) 
> -I$(srcdir)"
> [...]
> +emacs-lisp-intro.dvi: $(INFO_SOURCES)
> +     $(ENVADD) $(TEXI2DVI) $(srcdir)/emacs-lisp-intro.texi

Does this really work in native Windows environment (i.e. without
Cygwin being installed)?  I don't think CMD.EXE can run Unix shell
scripts such as texi2dvi, and neither I think that a native Windows
port of makeinfo will DTRT with the value of TEXINPUTS where
directories are separated by colons (instead of semi-colons).

If this only works with Cygwin tools, we should at least tell that
somewhere.  What would be _really_ nice if at least some targets would
work with native Windows tools, not only with Cygwin.  For example:

> +mostlyclean:
> +     rm -f  *.log *.cp *.fn *.ky *.pg *.vr *.tp
> +
> +clean: mostlyclean
> +     rm -f *.dvi

There's no reason not to use CMD's built-in DEL command here (perhaps
with a FOR loop).

> +++ emacs-20030219/nt/makefile.w32-in 2003-02-21 21:23:01.000000000 +0100
> @@ -214,6 +214,15 @@
>       $(MAKE) $(MFLAGS) -C ../lisp install
>       $(MAKE) $(MFLAGS) -C ../leim install
>  
> +force-info:
> +# Note that man/makefile knows how to
> +# put the info files in $(srcdir),
> +# so we can do ok running make in the build dir.
> +info: force-info
> +     (cd ..\man && $(MAKE) $(MFLAGS) info)
> +     (cd ..\lispref && $(MAKE) $(MFLAGS) info)
> +     (cd ..\lispintro && $(MAKE) $(MFLAGS) info)

This also won't work except with Cygwin Bash (or some other ported
Unix shell) as your shell.

Apologies if I missed some of this thread earlier, which stated that
the intent is indeed to make this work with Cygwin only.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]