Re: [Csql-development] CVS repository imported into git and my fixes to CSQL

[Przemysław 'P. P. <pr...@gm...>]

Hi Prabakaran,

> Thanks for x86_64 porting.

Exaggerated words. I made CSQL only compilable on x86_64.
The simplest way was using CPUTYPE=i686 for x86_64 (only one change in
configure.in) and effect would be the same.
The right way is adding new CPUTYPE and handling it properly in
Mutex.* files and I've done it.
Currently defined(x86_64) alternative uses i686 code, but maybe in
future that will change somehow.

Using unsigned int for Lock type (in Mutex.h) on x86 architectures
should be reconsidered in favour of unsigned long, because long size
equals processor word size (in general, there are exceptions but not
in x86 CPUs). This change may impact (or not -- I'm not sure, because
I haven't investigate the whole source code yet) some other areas, but
fixing will be rather trivial.

> Shall I know more about you and your assignments?

I'm member of collegiate team committed to project based on CSQL and
working on Ubuntu distribution. My current task is making CSQL
"available" for the rest of the team.

I sent two new commits:
f2cf58 -- fix for GCC 4.3
8a221e -- fix for more recent autotools

They are crucial for successful build in Ubuntu 8.10. This apply to
Ubuntu 8.04 if you remove ltmain.sh (and recreate it).

Side notes: libtool generated from ./configure script with old
ltmain.sh on Ubuntu 8.10 was causing something that looked like fork
bomb. On Ubuntu 8.04 there was no problem. Strange, really strange.

Regards.

-- 
Przemysław 'Przemoc' Pawełczyk
http://przemoc.net/