HI Frodo,
This all works great! Can you prepare one more big patch that
includes all your changes to abi/src/*?
Sam can you commit this patch and the new psiconv module? I'm sufficiently
CVS challenged that I think I would screw this up. Espcially since
I'm working from home behind a firewall without direct CVS access.
I think we need someone on the spot ready to fix things on tinderbox when
psiconv goes in. This is just being cautious however. Since psiconv
is straigt C and compiles without warnings there is a good chance it will
go in cleanly.
Thanks!
Martin
On Fri, 29 Dec 2000, Frodo Looijaard wrote:
> Martin Sevior wrote:
> >
> > HI Frodo,
> > I tried this just now. The code comlies all the way to ie_imp.cpp
> > where it fails becaase it can't find
> > <psiconv/data.h>
>
> Yeah. I though about this when lying in bed, but I did not have the
> courage to get up and fix it in the middle of the night :-)
>
> Add the following line to src/config/abi_defs.mk, after the EXTRA_LIBS
> declaration for psiconv:
> CFLAGS += -I$(ABI_ROOT)/../psiconv
> Note that there is one `psiconv'-level directory less here than in
> the EXTRA_LIBS declaration: this is on purpose! And it seems there is
> something iffy going on with libtool too. So I have attached a new
> patch that must be used *instead of* the previous one I sent.
>
> > You'd better rename all your own psiconv includes in /usr/include
> > /usr/local/include directories to make sure the includes from the peer
> > directory get found.
>
> Done so, and it now compiles and links properly. Sorry for the confusion.
>
> Frodo
>
> --
> Frodo Looijaard <[EMAIL PROTECTED]> PGP key and more: http://huizen.dds.nl/~frodol
> Defenestration n. (formal or joc.):
> The act of removing Windows from your computer in disgust, usually followed
> by the installation of Linux or some other Unix-like operating system.
>
