[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-devel
Subject:    Re: [PATCH] Re: KSocks on non-linux systems.
From:       Carsten Griwodz <griff () ifi ! uio ! no>
Date:       2001-03-20 7:33:42
[Download RAW message or body]

Hi !

On Mon, 19 Mar 2001, Rob Napier wrote:

> On Mon, Mar 19, 2001 at 02:48:17PM -0500, Rob Napier wrote:
> > On Thu, Mar 15, 2001 at 12:54:01PM -0800, Waldo Bastian wrote:
> > > >From George's CVS message for those who run non-linux but don't read kde-cvs 
> > >
> > > Please:  anyone on non-linux/glibc2 platforms could you check this code and
> > > see what needs to be done to make it compile for you?  I am sure that there
> > > will be subtle libc and #include differences that need to be addressed.
> > 
> > Solaris fails because socklen_t doesn't exist. The following patch
> > should fix it (compiles on Solaris, but untested; haven't tried to
> > compile on Linux).
> 
> Ah crude. Doesn't quite work. The problem is on Solaris if XPG4 isn't
> defined, then this needs to be int instead of size_t (which is uint,
> so it probably "matters"). If I just set it to 'int', though, is this
> going to break other platforms? Any AIX testers?

I don't have time to test right now, but on AIX 4.2 and later you get
size_t with sys/types.h and socklen_t with sys/socket.h. Both are
typedefs, so you can't undef them. Both are used in the function
prototypes so it is very likely that they are in fact defined. Both are
uint32 at this time (but you never know) and gcc doesn't like int instead.

Regards,
   Carsten

 
>> Visit http://master.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<

[prev in list] [next in list] [prev in thread] [next in thread]