Re: OpenSSL should define OPENSSL_THREADS?

2003-02-26 Thread Mark Murray
"Jacques A. Vidrine" writes:
> > Should the OpenSSL in FreeBSD be defining OPENSSL_THREADS?
> 
> I think you may be right.  OpenSSL 0.9.7's out-of-the box configure
> creates an opensslconf.h that would define OPENSSL_THREADS on FreeBSD.
> 
> Mark supplied the opensslconf.h's that are used in the FreeBSD build ...
> let's see if this is intentional or not. [cc'd]

Not intentional. If adding it breaks nothing, go ahead. I have Panic City
right now on my devboxes, so I'm working on fixing that.

M
--
Mark Murray
iumop ap!sdn w,I idlaH

To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message


Re: OpenSSL should define OPENSSL_THREADS?

2003-02-25 Thread Craig Rodrigues
On Tue, Feb 25, 2003 at 07:22:09AM -0600, Jacques A. Vidrine wrote:
> On Mon, Feb 24, 2003 at 09:58:21PM -0500, Craig Rodrigues wrote:
> > Hi,
> > 
> > I ran a cvsup of -CURRENT a few days ago.
> > 
> > I have some code which assumes that OPENSSL_THREADS is defined if
> > the OpenSSL version is greater than 0.9.7:
> > Should the OpenSSL in FreeBSD be defining OPENSSL_THREADS?
> 
> I think you may be right.  OpenSSL 0.9.7's out-of-the box configure
> creates an opensslconf.h that would define OPENSSL_THREADS on FreeBSD.
> 
> Mark supplied the opensslconf.h's that are used in the FreeBSD build ...
> let's see if this is intentional or not. [cc'd]


These kind of macro changes can break backwards compatibility
(OpenSSL's fault).  If the intent is to minimize breakage
of ports and such, I would not be opposed to do something like:

#define OPENSSL_THREADS 1
#define THREADS OPENSSL_THREADS/** deprecated */

Would this work?
 
-- 
Craig Rodrigues
http://home.attbi.com/~rodrigc
[EMAIL PROTECTED]

To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message


Re: OpenSSL should define OPENSSL_THREADS?

2003-02-25 Thread Jacques A. Vidrine
On Mon, Feb 24, 2003 at 09:58:21PM -0500, Craig Rodrigues wrote:
> Hi,
> 
> I ran a cvsup of -CURRENT a few days ago.
> 
> I have some code which assumes that OPENSSL_THREADS is defined if
> the OpenSSL version is greater than 0.9.7:
> 
> #define OPENSSL_THREAD_DEFINES
> #include 
> #include 
> #if OPENSSL_VERSION_NUMBER < 0x0090700fL
> #   if !defined(THREADS)  
> #  error "Thread support not enabled"
> #   endif
> #else
> #   if !defined(OPENSSL_THREADS)
> #  error "Thread support not enabled"
> #   endif
> #endif
> 
> 
> Should the OpenSSL in FreeBSD be defining OPENSSL_THREADS?

I think you may be right.  OpenSSL 0.9.7's out-of-the box configure
creates an opensslconf.h that would define OPENSSL_THREADS on FreeBSD.

Mark supplied the opensslconf.h's that are used in the FreeBSD build ...
let's see if this is intentional or not. [cc'd]

Cheers,
-- 
Jacques A. Vidrine <[EMAIL PROTECTED]>  http://www.celabo.org/
NTT/Verio SME  . FreeBSD UNIX .   Heimdal Kerberos
[EMAIL PROTECTED] .  [EMAIL PROTECTED]  .  [EMAIL PROTECTED]

To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message


Re: OpenSSL should define OPENSSL_THREADS?

2003-02-24 Thread Craig Rodrigues
On Mon, Feb 24, 2003 at 09:58:21PM -0500, Craig Rodrigues wrote:
> Hi,
> 
> I ran a cvsup of -CURRENT a few days ago.
> 
> I have some code which assumes that OPENSSL_THREADS is defined if
> the OpenSSL version is greater than 0.9.7:

I meant to say greater than or equal to 0.9.7. :)


> 
> #define OPENSSL_THREAD_DEFINES
> #include 
> #include 
> #if OPENSSL_VERSION_NUMBER < 0x0090700fL
> #   if !defined(THREADS)  
> #  error "Thread support not enabled"
> #   endif
> #else
> #   if !defined(OPENSSL_THREADS)
> #  error "Thread support not enabled"
> #   endif
> #endif
> 
> 
> Should the OpenSSL in FreeBSD be defining OPENSSL_THREADS?
> 
> Thanks.
> -- 
> Craig Rodrigues
> http://home.attbi.com/~rodrigc
> [EMAIL PROTECTED]
> 
> To Unsubscribe: send mail to [EMAIL PROTECTED]
> with "unsubscribe freebsd-current" in the body of the message

-- 
Craig Rodrigues
http://home.attbi.com/~rodrigc
[EMAIL PROTECTED]

To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message