all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Glenn Morris <rgm@gnu.org>
To: Eli Zaretskii <eliz@gnu.org>
Cc: nickrob@snap.net.nz, emacs-devel@gnu.org, ulm@gentoo.org,
	monnier@iro.umontreal.ca, jasonr@gnu.org
Subject: Re: Emacs can't be built without recent makeinfo
Date: Mon, 05 May 2008 14:31:21 -0400	[thread overview]
Message-ID: <bq4p9cbo1y.fsf@fencepost.gnu.org> (raw)
In-Reply-To: <uk5iawwmy.fsf@gnu.org> (Eli Zaretskii's message of "Sun, 04 May 2008 06:55:49 +0300")

Eli Zaretskii wrote:

> I agree with Jason: for distributed tarballs, makeinfo should not be
> an obstacle.  I don't see what's confusing, either; care to explain?

Patch with explanatory comments.

*** configure.in	3 May 2008 20:16:15 -0000	1.526
--- configure.in	5 May 2008 18:28:14 -0000
***************
*** 896,916 ****
     MAKEINFO=no
  fi
  
! if test "$MAKEINFO" = "no" && test "x${with_makeinfo}" != "xno"; then
! 
!    if test -e $srcdir/info/emacs; then
!       gotinfo="seems"
!    else
!       gotinfo="does NOT seem"
!    fi
! 
!   AC_MSG_ERROR( [You do not seem to have makeinfo >= 4.6.
! You will not be able to build the Emacs manuals.
! In Emacs releases, they are prebuilt, so this might not be a problem.
! Your source tree $gotinfo to have manuals in the `info' directory.
  Either install a suitable version of makeinfo, or re-run configure
! with the `--without-makeinfo' option.] )
! 
  fi
  
  dnl Add our options to ac_link now, after it is set up.
--- 896,919 ----
     MAKEINFO=no
  fi
  
! ## Makeinfo is unusual.  For a released Emacs, the manuals are
! ## pre-built, and not deleted by the normal clean rules.  makeinfo is
! ## therefore in the category of "special tools" not normally required, which
! ## configure does not have to check for (eg autoconf itself).
! ## In a CVS checkout on the other hand, the manuals are not included.
! ## So makeinfo is a requirement to build from CVS, and configure
! ## should test for it as it does for any other build requirement.
! ## We use the presence of $srcdir/info/emacs to distinguish a release,
! ## with pre-built manuals, from a CVS checkout.
! if test "$MAKEINFO" = "no"; then
!   if test "x${with_makeinfo}" = "xno"; then
!     MAKEINFO=off
!   elif ! test -e $srcdir/info/emacs; then
!     AC_MSG_ERROR( [You do not seem to have makeinfo >= 4.6, and your
! source tree does not seem to have pre-built manuals in the `info' directory.
  Either install a suitable version of makeinfo, or re-run configure
! with the `--without-makeinfo' option to build without the manuals.] )
!   fi
  fi
  
  dnl Add our options to ac_link now, after it is set up.


*** Makefile.in	3 May 2008 20:16:32 -0000	1.345
--- Makefile.in	5 May 2008 18:28:21 -0000
***************
*** 554,560 ****
  	   fi; \
  	   cd ${srcdir}/info ; \
  	   for elt in $(INFO_FILES); do \
! 	      test "$(MAKEINFO)" = "no" && ! test -e $$elt && continue; \
  	      for f in `ls $$elt $$elt-[1-9] $$elt-[1-9][0-9] 2>/dev/null`; do \
  	        ${INSTALL_DATA} $$f $(DESTDIR)${infodir}/$$f; \
  	        chmod a+r $(DESTDIR)${infodir}/$$f; \
--- 554,560 ----
  	   fi; \
  	   cd ${srcdir}/info ; \
  	   for elt in $(INFO_FILES); do \
! 	      test "$(MAKEINFO)" = "off" && ! test -e $$elt && continue; \
  	      for f in `ls $$elt $$elt-[1-9] $$elt-[1-9][0-9] 2>/dev/null`; do \
  	        ${INSTALL_DATA} $$f $(DESTDIR)${infodir}/$$f; \
  	        chmod a+r $(DESTDIR)${infodir}/$$f; \
***************
*** 566,572 ****
  	if [ `(cd ${srcdir}/info && /bin/pwd)` != `(cd $(DESTDIR)${infodir} && /bin/pwd)` ]; \
  	then \
  	  for elt in $(INFO_FILES); do \
! 	    test "$(MAKEINFO)" = "no" && ! test -e $$elt && continue; \
  	    (cd $${thisdir}; \
  	     ${INSTALL_INFO} --info-dir=$(DESTDIR)${infodir} $(DESTDIR)${infodir}/$$elt); \
  	  done; \
--- 566,572 ----
  	if [ `(cd ${srcdir}/info && /bin/pwd)` != `(cd $(DESTDIR)${infodir} && /bin/pwd)` ]; \
  	then \
  	  for elt in $(INFO_FILES); do \
! 	    test "$(MAKEINFO)" = "off" && ! test -e $$elt && continue; \
  	    (cd $${thisdir}; \
  	     ${INSTALL_INFO} --info-dir=$(DESTDIR)${infodir} $(DESTDIR)${infodir}/$$elt); \
  	  done; \
***************
*** 785,792 ****
  # put the info files in $(srcdir),
  # so we can do ok running make in the build dir.
  info: force-info
! 	@if test "$(MAKEINFO)" = "no"; then \
! 	  echo "Configured without makeinfo, not building manuals" ; \
  	else \
  	  $(MAKE) $(MFLAGS) info-real ; \
  	fi
--- 785,795 ----
  # put the info files in $(srcdir),
  # so we can do ok running make in the build dir.
  info: force-info
! 	@if test "$(MAKEINFO)" = "off"; then \
! 	  echo "Configured --without-makeinfo, not building manuals" ; \
! 	elif test "$(MAKEINFO)" = "no"; then \
! 	  echo "makeinfo is missing - cannot build manuals" ; \
! 	  exit 1 ; \
  	else \
  	  $(MAKE) $(MFLAGS) info-real ; \
  	fi




  reply	other threads:[~2008-05-05 18:31 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-04-25  4:45 Emacs can't be built without recent makeinfo Nick Roberts
2008-04-25 13:45 ` Stefan Monnier
2008-04-25 16:22   ` Ulrich Mueller
2008-04-25 18:48     ` Glenn Morris
2008-04-25 22:06       ` Nick Roberts
2008-04-25 23:26         ` Glenn Morris
2008-05-03 20:17           ` Glenn Morris
2008-05-03 21:07             ` Jason Rumney
2008-05-03 21:12               ` Glenn Morris
2008-05-03 21:38                 ` Jason Rumney
2008-05-04  1:58                   ` Glenn Morris
2008-05-04  3:55                 ` Eli Zaretskii
2008-05-05 18:31                   ` Glenn Morris [this message]
2008-05-04 21:15             ` Nick Roberts
2008-04-25 22:47 ` Magnus Henoch

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=bq4p9cbo1y.fsf@fencepost.gnu.org \
    --to=rgm@gnu.org \
    --cc=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=jasonr@gnu.org \
    --cc=monnier@iro.umontreal.ca \
    --cc=nickrob@snap.net.nz \
    --cc=ulm@gentoo.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.