[slim-vim] progress - great!

[Larry Clapp]

On Sat, Jul 15, 2006 at 09:43:57AM -0400, Larry Clapp wrote:
> On Sat, Jul 15, 2006 at 11:17:45AM +0300, Sampo Vuori wrote:
> > > Here is a basic, off the top of my head guide though:
> > > 1) Get Darcs from http://abridgegame.org/darcs/
> > > 2) Get Vim+ECL and ecl-slime from here
> > >    http://wiki.alu.org/Slim-Vim-Repos
> > > 3) Get ECL from http://ecls.sourceforge.net/
> > I got the CVS-version of ECL.
> > 
> > > 4) Compile ECL following the instructions on the home page
> > > 5) Compile Vim+Async+ECL, on Darwin I configure with
> > >   ./configure --prefix=/opt/local --enable-eclinterp --disable-darwin
> > >   The important part is really the "--enable-eclinterp" part
> > 
> > I got here, then the ran into problems. The first complaint was
> > about ecl.h not found. I told configure where to find them
> > 
> > CFLAGS=-I/path/to/ecl-sources/src/h
> 
> This should be
> 
>   CFLAGS=-I/path/to/installed/ecl/files
> 
> e.g. on my machine:
> 
>   CFLAGS=-I/usr/lib/ecl/ecl

I've updated the Vim repository at theclapp.org to work with ECL 0.9i.
You should no longer need this.

> > Then next compiler error was:
> > 
> > if_ecl.c:160: error: 'union cl_lispunion' has no member named
> > 'string'
> > 
> > Looks like string is member of cl_lispunion only when ECL_UNICODE
> > is defined, so I did a dirty trick and added #define ECL_UNICODE
> > to src/if_ecl.c.
> 
> I'll look into this.  I doubt that an executable built this way
> would run.  But you probably knew that.  :)
> 
> > Next errors come from the linker:
> > objects/if_ecl.o: In function `intern_vim':if_ecl.c:(.text+0x14):
> > undefined reference to `make_string_copy'
> 
> I'll look at this too.  make_string_copy is a valid ECL function in
> my current copy.  But I'm not even running 0.9i, much less current
> CVS.

The new patch fixes both of these.  cl_object.string => base_string,
and make_string_copy => make_base_string_copy.

Changelog:

> Sat Jul 15 11:47:19 EDT 2006  Larry Clapp <larry at theclapp.org>
>   * Update to work with ECL 0.9i
> 
>   - Changed uses of cl_object.string to base_string.
>   - Changed uses of make_string_copy to make_base_string_copy.
>   - Changed include of ecl.h to ecl/ecl.h.  Now no longer need
>     CFLAGS=-I/usr/lib/ecl/ecl (or whatever) on the configure line.

Hope that does it!  Please keep reporting any problems you encounter!

-- Larry