Re: build problems

[Ludovic Courtès]

Hi Ken,

Ken Raeburn <address@hidden> 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" > "address@hidden"
> +         "$(snarf_doc).scm" > "address@hidden"

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’.