unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Espen Wiborg <espenhw@grumblesmurf.org>
To: David Kastrup <dak@gnu.org>
Cc: emacs-pretest-bug@gnu.org, kifer@cs.sunysb.edu
Subject: Re: make bootsrap problems
Date: Tue, 27 May 2008 15:36:56 +0200	[thread overview]
Message-ID: <874p8jvpfr.fsf@grumblesmurf.org> (raw)
In-Reply-To: <85bq2suivg.fsf@lola.goethe.zz> (David Kastrup's message of "Tue,  27 May 2008 12:44:03 +0200")

David Kastrup <dak@gnu.org> writes:
> Michael Kifer <kifer@cs.sunysb.edu> writes:
>> In the latest cvs version under Ubuntu 8.04, I am getting:
>>
>> make maintainer-clean
>> configure
>> make bootstrap
>>
<-snip->
>> sed -f temp.sed /home/users/kifer/gnu/emacs/lisp/Makefile.in > temp-elcfiles || rm temp-elcfiles
>> sed: file temp.sed line 3: unknown command: `('
>> rm temp.sed
>> chmod +w /home/users/kifer/gnu/emacs/lisp/Makefile.in
>> mv -f temp-elcfiles /home/users/kifer/gnu/emacs/lisp/Makefile.in || echo "Maintainer warning: failed to update Makefile.in"
>> mv: cannot stat `temp-elcfiles': No such file or directory
>> Maintainer warning: failed to update Makefile.in
>
> Me too.  The culprit would likely be
>
> ## Construct a sed command file that operates on lines between
> ## ^ELCFILES and the next blank line.  Convert spaces between members
> ## of COMPILE_FIRST to "\|", and escape directory "/".  List the .elc
> ## files, exclude the members of COMPILE_FIRST, convert ^$(lisp) to a
> ## literal " $(lisp)", add trailing " \\\" to the end of every line
> ## (the last \ continues the line in sed, the other \\ result in a
> ## single \ at the end of the replacement text).  For the last line,
> ## just use a single "\", for sed.  Finally, run sed using the command
> ## so constructed and update Makefile.in.  chmod +w is for CVSREAD=1.
> ## Errors in final sed are non-fatal, since they have no effect on
> ## building Emacs.
> update-elclist:
<-snip->
>
> in lisp/Makefile.in.  In particular, "Finally, run sed using the command
> so constructed and update Makefile.in." since the preceding recipe does
> not actually construct a command, but just a part of it.

The problem is that the generated sed script is missing a line
continuation backslash.

The following patch fixes:

Index: lisp/Makefile.in
===================================================================
RCS file: /cvsroot/emacs/emacs/lisp/Makefile.in,v
retrieving revision 1.123
diff -c -r1.123 Makefile.in
*** lisp/Makefile.in	27 May 2008 02:14:46 -0000	1.123
--- lisp/Makefile.in	27 May 2008 13:21:27 -0000
***************
*** 164,170 ****
  ## building Emacs.
  update-elclist:
  	echo "/^ELCFILES/,/^$$/c\\" > temp.sed
! 	echo "ELCFILES = \\\\\\" >> temp.sed
  	exclude=`echo $(COMPILE_FIRST) | sed -e 's, ,\\\\|,g' -e 's,\/,\\\\/,g'`; \
  	ls $(lisp)/*.elc $(lisp)/*/*.elc | sed -e "/$$exclude/d" -e "s|^$(lisp)|	\$$(lisp)|" -e 's/$$/ \\\\\\/' -e '$$ s/ \\\\//' >> temp.sed
  	echo "" >> temp.sed
--- 164,170 ----
  ## building Emacs.
  update-elclist:
  	echo "/^ELCFILES/,/^$$/c\\" > temp.sed
! 	echo "ELCFILES = \\\\\\\\\\" >> temp.sed
  	exclude=`echo $(COMPILE_FIRST) | sed -e 's, ,\\\\|,g' -e 's,\/,\\\\/,g'`; \
  	ls $(lisp)/*.elc $(lisp)/*/*.elc | sed -e "/$$exclude/d" -e "s|^$(lisp)|	\$$(lisp)|" -e 's/$$/ \\\\\\/' -e '$$ s/ \\\\//' >> temp.sed
  	echo "" >> temp.sed


-- 
Espen Wiborg <espenhw@grumblesmurf.org>
Thermodynamics in a nutshell:
1st Law:  You can't win.  (Energy is conserved)
2nd Law:  You can't break even.  (Entropy)
0th Law:  You can't even quit the game.  (Closed systems) -- Taki Kogoma




  reply	other threads:[~2008-05-27 13:36 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-27  9:56 make bootsrap problems Michael Kifer
2008-05-27 10:44 ` David Kastrup
2008-05-27 13:36   ` Espen Wiborg [this message]
2008-05-27 19:29     ` Glenn Morris
2008-05-28  0:44       ` Glenn Morris
2008-05-28  1:04         ` Glenn Morris
2008-05-28  7:08           ` Espen Wiborg
  -- strict thread matches above, loose matches on Subject: below --
2008-05-28  7:57 Angelo Graziosi
2008-05-28  8:02 ` Glenn Morris

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

  List information: https://www.gnu.org/software/emacs/

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

  git send-email \
    --in-reply-to=874p8jvpfr.fsf@grumblesmurf.org \
    --to=espenhw@grumblesmurf.org \
    --cc=dak@gnu.org \
    --cc=emacs-pretest-bug@gnu.org \
    --cc=kifer@cs.sunysb.edu \
    /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 public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).