Re: Any more infos about the build failure for my (small) patch

[James Mckenzie]

Octavian Voicu wrote:
> On Fri, Jul 23, 2010 at 12:32 PM, Uwe Bonnes
>  wrote:
> > Paul> SSL_CTX *SSL_CTX_new(SSL_METHOD *meth); 61557 88 -rw-r--r-- 1 root
> >    Paul> root 87238 Jun 2 11:11 ./usr/include/openssl/ssl.h
> >
> > Ah, Readhat defining in it's own way.
> > What about debian?
> >
> > Isn't the OpenSSL documentation way th way it should be?
>
> Ubuntu also has /usr/include/openssl/ssl.h.
> Linux manpages show the same thing: http://linux.die.net/man/3/ssl_ctx_new
>
> Maybe this should go in autoconf?

Have to check the effect on other UNIX variations, including MacOSX before doing this.

Looks like a good idea, however.

James McKenzie

Any more infos about the build failure for my (small) patch

[Austin English]

On Fri, Jul 23, 2010 at 8:50 AM, James Mckenzie
 wrote:
> Octavian Voicu wrote:
> > On Fri, Jul 23, 2010 at 12:32 PM, Uwe Bonnes
> >  wrote:
> > > Paul> SSL_CTX *SSL_CTX_new(SSL_METHOD *meth); 61557 88 -rw-r--r-- 1 root
> > >    Paul> root 87238 Jun 2 11:11 ./usr/include/openssl/ssl.h
> > >
> > > Ah, Readhat defining in it's own way.
> > > What about debian?
> > >
> > > Isn't the OpenSSL documentation way th way it should be?
> >
> > Ubuntu also has /usr/include/openssl/ssl.h.
> > Linux manpages show the same thing: http://linux.die.net/man/3/ssl_ctx_new
> >
> > Maybe this should go in autoconf?
>
> Have to check the effect on other UNIX variations, including MacOSX before doing this.
>
> Looks like a good idea, however.

FWIW, NetBSD has the same problem as Fedora, see
http://bugs.winehq.org/show_bug.cgi?id=18165

Any more infos about the build failure for my (small) patch

[James McKenzie]

James Mckenzie wrote:
> Octavian Voicu wrote:
> > On Fri, Jul 23, 2010 at 12:32 PM, Uwe Bonnes
> >  wrote:
> > > Paul> SSL_CTX *SSL_CTX_new(SSL_METHOD *meth); 61557 88 -rw-r--r-- 1 root
> > >    Paul> root 87238 Jun 2 11:11 ./usr/include/openssl/ssl.h
> > >
> > > Ah, Readhat defining in it's own way.
> > > What about debian?
> > >
> > > Isn't the OpenSSL documentation way th way it should be?
> >
> > Ubuntu also has /usr/include/openssl/ssl.h.
> > Linux manpages show the same thing: http://linux.die.net/man/3/ssl_ctx_new
> >
> > Maybe this should go in autoconf?
>
> Have to check the effect on other UNIX variations, including MacOSX before doing this.

Same will happen here and with Fink (checked the fink .info file and the 
openssl include directory has ssl.h.)

James McKenzie

Any more infos about the build failure for my (small) patch

[Marcus Meissner]

On Fri, Jul 23, 2010 at 06:37:11PM -0700, James McKenzie wrote:
> James Mckenzie wrote:
> > Octavian Voicu wrote:
> > > On Fri, Jul 23, 2010 at 12:32 PM, Uwe Bonnes
> > >  wrote:
> > > > Paul> SSL_CTX *SSL_CTX_new(SSL_METHOD *meth); 61557 88 -rw-r--r-- 1 root
> > > >    Paul> root 87238 Jun 2 11:11 ./usr/include/openssl/ssl.h
> > > >
> > > > Ah, Readhat defining in it's own way.
> > > > What about debian?
> > > >
> > > > Isn't the OpenSSL documentation way th way it should be?
> > >
> > > Ubuntu also has /usr/include/openssl/ssl.h.
> > > Linux manpages show the same thing: http://linux.die.net/man/3/ssl_ctx_new
> > >
> > > Maybe this should go in autoconf?
> >
> > Have to check the effect on other UNIX variations, including MacOSX before doing this.
>
> Same will happen here and with Fink (checked the fink .info file and the 
> openssl include directory has ssl.h.)

The const was added for the openssl 1.0 release I guess.

ciao, Marcus

Any more infos about the build failure for my (small) patch

[James McKenzie]

Marcus Meissner wrote:
> On Fri, Jul 23, 2010 at 06:37:11PM -0700, James McKenzie wrote:
> > James Mckenzie wrote:
> > > Octavian Voicu wrote:
> > > > On Fri, Jul 23, 2010 at 12:32 PM, Uwe Bonnes
> > > >  wrote:
> > > > > Paul> SSL_CTX *SSL_CTX_new(SSL_METHOD *meth); 61557 88 -rw-r--r-- 1 root
> > > > >    Paul> root 87238 Jun 2 11:11 ./usr/include/openssl/ssl.h
> > > > >
> > > > > Ah, Readhat defining in it's own way.
> > > > > What about debian?
> > > > >
> > > > > Isn't the OpenSSL documentation way th way it should be?
> > > >
> > > > Ubuntu also has /usr/include/openssl/ssl.h.
> > > > Linux manpages show the same thing: http://linux.die.net/man/3/ssl_ctx_new
> > > >
> > > > Maybe this should go in autoconf?
> > >
> > > Have to check the effect on other UNIX variations, including MacOSX before doing this.
> >
> > Same will happen here and with Fink (checked the fink .info file and the 
> > openssl include directory has ssl.h.)
>
> The const was added for the openssl 1.0 release I guess.

I had openssl 0.9.8h installed.  I'll be updating Fink tomorrow when I 
have WiFi access.

James McKenzie