You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by William A Rowe Jr <wr...@rowe-clan.net> on 2017/04/13 04:12:24 UTC

Re: svn commit: r1791192 - /httpd/httpd/branches/2.4.x/support/abs.mak

On Wed, Apr 12, 2017 at 5:31 PM,  <gs...@apache.org> wrote:
> Author: gsmith
> Date: Wed Apr 12 22:31:15 2017
> New Revision: 1791192
>
> URL: http://svn.apache.org/viewvc?rev=1791192&view=rev
> Log:
> Add another include since applink.c has been moved in
> the OpenSSL source. More info:
> http://marc.info/?t=149192864300001&r=1&w=2
>
> Modified:
>     httpd/httpd/branches/2.4.x/support/abs.mak
>
> Modified: httpd/httpd/branches/2.4.x/support/abs.mak
> URL: http://svn.apache.org/viewvc/httpd/httpd/branches/2.4.x/support/abs.mak?rev=1791192&r1=1791191&r2=1791192&view=diff
> ==============================================================================
> --- httpd/httpd/branches/2.4.x/support/abs.mak (original)
> +++ httpd/httpd/branches/2.4.x/support/abs.mak Wed Apr 12 22:31:15 2017
> @@ -28,7 +28,7 @@ NULL=nul
>  !IF "$(_HAVE_OSSL110)" == "1"
>  SSLCRP=libcrypto
>  SSLLIB=libssl
> -SSLINC=/I ../srclib/openssl/include
> +SSLINC=/I ../srclib/openssl/include /I ../srclib/openssl/ms

Question (untested)... does this work for local builds, since ms is not
in the include <opensll/appfoo> scope?

Re: svn commit: r1791192 - /httpd/httpd/branches/2.4.x/support/abs.mak

Posted by Gregg Smith <gl...@gknw.net>.
On 4/12/2017 9:12 PM, William A Rowe Jr wrote:
> On Wed, Apr 12, 2017 at 5:31 PM,  <gs...@apache.org> wrote:
>> Author: gsmith
>> Date: Wed Apr 12 22:31:15 2017
>> New Revision: 1791192
>>
>> URL: http://svn.apache.org/viewvc?rev=1791192&view=rev
>> Log:
>> Add another include since applink.c has been moved in
>> the OpenSSL source. More info:
>> http://marc.info/?t=149192864300001&r=1&w=2
>>
>> Modified:
>>     httpd/httpd/branches/2.4.x/support/abs.mak
>>
>> Modified: httpd/httpd/branches/2.4.x/support/abs.mak
>> URL: http://svn.apache.org/viewvc/httpd/httpd/branches/2.4.x/support/abs.mak?rev=1791192&r1=1791191&r2=1791192&view=diff
>> ==============================================================================
>> --- httpd/httpd/branches/2.4.x/support/abs.mak (original)
>> +++ httpd/httpd/branches/2.4.x/support/abs.mak Wed Apr 12 22:31:15 2017
>> @@ -28,7 +28,7 @@ NULL=nul
>>  !IF "$(_HAVE_OSSL110)" == "1"
>>  SSLCRP=libcrypto
>>  SSLLIB=libssl
>> -SSLINC=/I ../srclib/openssl/include
>> +SSLINC=/I ../srclib/openssl/include /I ../srclib/openssl/ms
>
> Question (untested)... does this work for local builds, since ms is not
> in the include <opensll/appfoo> scope?
>
No, will revert.

It will have to be done as a prebuild step, placing a copy into 
include/openssl.