[Top][All Lists]
[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.