Re: [RFA][patch 1/9] Yet another respin of the patch with initial Python support

Thiago Jung Bauermann
2008-07-21T16:47:56Z

On Mon, 2008-07-21 at 12:26 -0300, Thiago Jung Bauermann wrote:
> @@ -1283,7 +1303,11 @@ test-cp-name-parser$(EXEEXT): test-cp-na
> # duplicates. Files in the gdb/ directory can end up appearing in
> # COMMON_OBS (as a .o file) and CONFIG_SRCS (as a .c file).
>
> -INIT_FILES = $(COMMON_OBS) $(TSOBS) $(CONFIG_SRCS)
> +# NOTE: bauermann/2008-06-15: python.c needs to be explicitly included
> +# because the generation of init.c does not work for .c files which
> +# result in differently named objects (i.e., python/python -> python.o).
> +
> +INIT_FILES = $(COMMON_OBS) $(TSOBS) $(CONFIG_SRCS) python/python.c
> init.c: $(INIT_FILES)
> @echo Making init.c
> @rm -f init.c-tmp init.l-tmp

Please ignore this hunk, it shouldn't be there and won't be committed
(*sigh*). The rest of the patch is ok.

Thanks, Tromey, for sanity checking this post and spotting this.
--
[]'s
Thiago Jung Bauermann
Software Engineer
IBM Linux Technology Center