[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: build problems
From: |
Ludovic Courtès |
Subject: |
Re: build problems |
Date: |
Tue, 02 Mar 2010 11:23:37 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.1 (gnu/linux) |
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’.