1.6 psyntax.pp bootstrapping bug -- may have found when it started.

[Rob Browning]

After trying a bunch of CVS builds from different points, it looks
like I may have narrowed down the source of the bug down to a
particular day.

If you check out two trees and update each as follows:

  cvs -qz3 update -r branch_release-1-6 -D 2003/01/27 -Pd
  cvs -qz3 update -r branch_release-1-6 -D 2003/01/28 -Pd

Then you apply the diff below to both trees (turns out there are two
psyntax.pp generation issues) and run

  ./autogen.sh
  ./configure --prefix=whatever --enable-maintainer-mode
  make
  touch ice-9/psyntax.ss
  make

then at least here, these commands will run fine in first tree, but
will fail in the second like this:

  /home/rlb/src/guile/1.6-debug/pre-inst-guile -s ./compile-psyntax.scm \
          ./psyntax.ss ./psyntax.pp
  /home/rlb/src/guile/1.6-debug/ice-9/syncase.scm:131:16: In procedure
    scm-error in expression (scm-error (quote misc-error) who ...):
  /home/rlb/src/guile/1.6-debug/ice-9/syncase.scm:131:16: invalid syntax ()

  Some deprecated features have been used.  Set the environment
  variable GUILE_WARN_DEPRECATED to "detailed" and rerun the
  program to get more information.  Set it to "no" to suppress
  this message.
  make[2]: *** [psyntax.pp] Error 2

I may not have time to try to figure out the actual cause for a few
days, so if anyone else would like to try to reproduce this or track
it down, go right ahead.  I'd rather not release 1.6.5 until we can
see about fixing this.

Here's the diff you need to get around the other error relating to
apply.

Index: ice-9/syncase.scm
===================================================================
RCS file: /cvsroot/guile/guile/guile-core/ice-9/syncase.scm,v
retrieving revision 1.18.2.8
diff -u -r1.18.2.8 syncase.scm
--- ice-9/syncase.scm   27 Jan 2003 11:02:51 -0000      1.18.2.8
+++ ice-9/syncase.scm   14 Sep 2003 19:29:42 -0000
@@ -54,15 +54,16 @@
           syntax-dispatch syntax-error bound-identifier=?
           datum->syntax-object free-identifier=?
           generate-temporaries identifier? syntax-object->datum
-          void syncase))
+;;        void syncase))
+          void eval syncase))
 
 ;; This is to avoid a deprecation warning about re-exporting eval.
 ;; When the re-exporting behavior of export is removed, removed this
 ;; code and include 'eval' in the export clause of define-module,
 ;; above.
 
-(define eval #f)
-(export eval)
+;;(define eval #f)
+;;(export eval)
 
 
 
@@ -244,8 +245,8 @@
 
 
 ;;; The following lines are necessary only if we start making changes
-;; (use-syntax sc-expand)
-;; (load-from-path "ice-9/psyntax.ss")
+(use-syntax sc-expand)
+(load-from-path "ice-9/psyntax.ss")
 
 (define internal-eval (nested-ref the-scm-module '(app modules guile eval)))
 

-- 
Rob Browning
rlb @defaultvalue.org and @debian.org; previously @cs.utexas.edu
GPG starting 2002-11-03 = 14DD 432F AE39 534D B592  F9A0 25C8 D377 8C7E 73A4