Hi Ken, Ken Raeburn <raeb...@raeburn.org> writes:
> On Feb 15, 2010, at 08:41, Ken Raeburn wrote: >> #1: c-tokenize.lex declares yyget_leng() as returning int, but the flex >> template defines it as returning yy_size_t (which is size_t, a.k.a. unsigned >> long), so c-tokenize.c doesn't compile. Changing the declaration in >> c-tokenize.lex to return size_t works for me, but the file hasn't been >> changed in some time, so if "int" was working for other platforms, changing >> it could be a problem. >> >> We don't actually reference yyget_leng elsewhere explicitly; can we just get >> rid of the declaration? > > This patch has been working fine for me on my Mac; we may be able to delete > more of the declarations, depending what other lex implementations do Keep in mind that ‘c-tokenize.c’ is part of the distribution, so we shouldn’t worry much about what Flex implementations do, as long as the file we ship compiles fine. I don’t have any problem with Flex 2.5.35. Which version do you use? >> #2: [...] I found that load-module in boot-9.scm tries to interpret a >> relative path as relative to the filename for the current-load-port, >> if there is one. Since the currently loading file was >> "../../../doc/ref/make-texinfo.scm" and the supplied filename (given >> on the command line) was "../../../doc/ref/standard-library.scm", >> load-module jammed the two prefixes together and came up with a >> pathname pointing off to nowhere. [...] > Path for snarf_doc input is treated relative to make-texinfo.scm file. > > diff --git a/doc/ref/Makefile.am b/doc/ref/Makefile.am > index a587343..1da5cde 100644 > --- a/doc/ref/Makefile.am > +++ b/doc/ref/Makefile.am > @@ -150,7 +150,7 @@ include standard-library.am > $(snarf_doc).texi: $(standard_library_scm_files) > GUILE_AUTO_COMPILE=0 \ > "$(preinstguile)" "$(srcdir)/make-texinfo.scm" \ > - "$(srcdir)/$(snarf_doc).scm" > "$...@.tmp" > + "$(snarf_doc).scm" > "$...@.tmp" This one seems wrong to me since $(snarf_doc).scm is in $(srcdir), not $(builddir). Instead I suggest changing $(srcdir) to $(abs_srcdir), which should solve the problem. Can you confirm and commit? Thanks! Ludo’.
