<div class="gmail_quote">On Tue, Jan 11, 2011 at 03:10, Maxim Dounin <span dir="ltr">&lt;<a href="mailto:mdounin@mdounin.ru">mdounin@mdounin.ru</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Hello!<br>
<div><div></div><div class="h5"><br>
On Mon, Jan 10, 2011 at 11:45:26PM +0300, Kirill A. Korinskiy wrote:<br>
<br>
&gt; Fix nginx for build with any version of openssl.<br>
&gt;<br>
&gt; Format of openssl version is MNNFFPPS: major minor fix patch status<br>
&gt; The status nibble has one of the values 0 for development, 1 to e for betas<br>
&gt; 1 to 14, and f for release.  The patch level is exactly that.<br>
<br>
</div></div>Missing part in your patch comment is when the interface change in<br>
question (&quot;const&quot; qualifier in SSL_get_current_cipher()&#39;s return<br>
type and other places) was introduced.<br>
<br>
It was introduced here:<br>
<br>
<a href="http://cvs.openssl.org/chngview?cn=17503" target="_blank">http://cvs.openssl.org/chngview?cn=17503</a><br>
2008-Oct-12 14:32:38 (UTC)<br>
<br>
at that time version in openssl&#39;s HEAD was 0x00909000L, and it was<br>
bumped to 0x10000000L here:<br>
<br>
<a href="http://cvs.openssl.org/chngview?cn=17927" target="_blank">http://cvs.openssl.org/chngview?cn=17927</a><br>
2009-Mar-28 17:12:50 (UTC)<br>
<br>
There were no version bump for the interface change in question,<br>
so you really can&#39;t make nginx buildable with *any* version with<br>
this aproach.  Though checking for 0x10000000L is indeed ok, it<br>
will make nginx buildable with openssl 1.0.0-beta[12345] without<br>
breaking anything.<br></blockquote><div><br></div><div><meta http-equiv="content-type" content="text/html; charset=utf-8">But any way I see one way: start use version of 0x009080a0L (it&#39;s a first release after interface change). Yes this is do broken build with version after revison 17503 and before 0.9.8f.</div>
<div><br></div><div>Update patches attached.</div></div><br>-- <br><div>wbr, Kirill</div><br>