On 2015/11/02 22:58, Jérémie Courrèges-Anglas wrote:
> Michael McConville <[email protected]> writes:
> 
> > They added some new SSL conditions that don't compile. I took the simple
> > route in the attached patches and defaulted to SSL23.
> 
> Well, simple but a bit intrusive...
> 
> > That uses the best
> > available cipher, right?
> 
> That uses the default cipher suite.
> 
> > This approach is a little iffy because I think
> > it ignores the user's cipher prefs. I wanted to get a working WiP so
> > that people could review it because I haven't worked with SSL/TLS APIs
> > before.
> >
> > What's attached builds and runs fine for me.
> 
> Here's an less intrusive diff that should be easy to push upstream.

yep, I much prefer this.

> (Except in special cases, please send diffs instead of tarballs for
> existing ports).

cvs add $file
cvs rm $file
cvs diff -uNp

for openbsd devs: do your adds against the master repo unless you are
sure that you're not re-adding a file from the Attic.

Reply via email to