cspublic.h BLK_VERSION_XXX - should this patch be applied?

"Reid, Roger L." <[email protected]>
Newsgroups gmane.comp.db.tds.freetds
Message-ID <[email protected]>
This is one of those "not sure it belongs here" patches.   I think it does, but I'm not unanimous in that.
I know when linking with Sybase's ct, I'll be getting them from Sybase's includes.

I modified include/cspublic.h so that sqsh-2.1.5 would compile with 0.82.   In March, http://lists.ibiblio.org/pipermail/freetds/2008q1/023098.html dealt with about the same issue 
(except not including Sybase 15) by putting the defines in the sqsh code instead.

I have only tested this on Solaris 8.  

----------- diff -e cspublic.h.orig cspublic.h -----------------
449c
#define BLK_VERSION_110 BLK_VERSION_100
#define BLK_VERSION_110 BLK_VERSION_100
#define BLK_VERSION_120 BLK_VERSION_100
#define BLK_VERSION_125 CS_VERSION_125
#define BLK_VERSION_150 CS_VERSION_150
.
----------- diff cspublic.h.orig cspublic.h -----------------
449c449,453
< #define BLK_VERSION_110 CS_VERSION_100
---
> #define BLK_VERSION_110 BLK_VERSION_100
> #define BLK_VERSION_110 BLK_VERSION_100
> #define BLK_VERSION_120 BLK_VERSION_100
> #define BLK_VERSION_125 CS_VERSION_125
> #define BLK_VERSION_150 CS_VERSION_150
--------------------------------------------------------------------

Roger Reid (advocate of Nonfunctional Programming)
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.