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

List:       kde-devel
Subject:    Re: [PATCH2] Re: KSocks on non-linux systems.
From:       Rob Napier <rnapier () employees ! org>
Date:       2001-03-21 20:50:39
[Download RAW message or body]

On Tue, Mar 20, 2001 at 02:54:49PM -0800, Waldo Bastian wrote:
> On Tuesday 20 March 2001 14:22, Rob Napier wrote:
> > OK, here's the corrected patch with David's comments. Someone else
> > will need to commit it.
> 
> I hate to bring the bad news, but this isn't going to work. ksize_t is nice 
> but we can't use it in a header file that gets installed cause it depends on 
> config.h and we can't install header files that need config.h

Didn't know that. Obvious in retrospect :)


> > BTW, I understand that we're using ksize_t instead of socklen_t. But
> > why does ksockaddr.h define a ksocklen_t (unsigned, which is not the
> > same as ksize_t).
> 
> ksockaddr is a new class part of the IPv6 stuff. I think ksocks should be 
> integrated with this new ksocket class so that we have one portable KDE 
> socket API with support for proxies.
> 
> Is this patch needed to fix compilation? In that case I will apply it but 
> will remove ksocks.h from the headers that get installed.

This patch (or something like it) is required for solaris compilation.
I don't know if ksocks.h is needed anywhere else. I like the idea of
getting it over into ksocket, though. I'm assuming that this will
still show up on machies without IPv6 (eg solaris)?

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

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