https://bz.apache.org/bugzilla/show_bug.cgi?id=64429
--- Comment #7 from Mark Thomas ---
The directives I added needed to be consistent with the directives already in
place to avoid the errors.
--
You are receiving this mail because:
You are the assignee for the bug.
https://bz.apache.org/bugzilla/show_bug.cgi?id=64429
--- Comment #6 from Michael Osipov ---
(In reply to Mark Thomas from comment #5)
> It was another inconsistent directive.
Does your fix apply to every version of LibreSSL? While working on other
LibreSSL related issues I used this pattern:
https://bz.apache.org/bugzilla/show_bug.cgi?id=64429
Mark Thomas changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
https://bz.apache.org/bugzilla/show_bug.cgi?id=64429
--- Comment #4 from Michael Osipov ---
(In reply to Mark Thomas from comment #3)
> The fix looks to be fairly simple and I have this committed locally. The
> various #if preprocessor directives are not consistent.
>
> #if
https://bz.apache.org/bugzilla/show_bug.cgi?id=64429
--- Comment #3 from Mark Thomas ---
The fix looks to be fairly simple and I have this committed locally. The
various #if preprocessor directives are not consistent.
#if OPENSSL_VERSION_NUMBER < 0x1010L
vs
#if OPENSSL_VERSION_NUMBER <
https://bz.apache.org/bugzilla/show_bug.cgi?id=64429
--- Comment #2 from Michael Osipov ---
(In reply to Christopher Schultz from comment #1)
> I think a solution is desired, but there is no need to revert.
>
> Support for LibreSSL is a goal, not a requirement.
I agree, but if it worked before
https://bz.apache.org/bugzilla/show_bug.cgi?id=64429
--- Comment #1 from Christopher Schultz ---
I think a solution is desired, but there is no need to revert.
Support for LibreSSL is a goal, not a requirement.
--
You are receiving this mail because:
You are the assignee for the bug.