Skip to content

Commit

Permalink
Fix build w/o OpenSSL by w/ MbedTLS
Browse files Browse the repository at this point in the history
In 78fa397 from
libevent#1241 (comment) a
new "if MBEDTLS" block was added to include/include.am. This is not
parsed by automake as it isn't on the first column, resulting in a
subsequent error in the build.

From [1]:

> The if, else, and endif statements should not be indented, i.e., start on column one.

  [1]: https://www.gnu.org/software/automake/manual/html_node/Usage-of-Conditionals.html

Fixes: libevent#1251
Fixes: google/oss-fuzz#7149
  • Loading branch information
saurik authored and azat committed Feb 12, 2022
1 parent 6d09efe commit 3da7185
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions include/include.am
Original file line number Diff line number Diff line change
Expand Up @@ -36,9 +36,9 @@ EVENT2_EXPORT = \
if OPENSSL
EVENT2_EXPORT += include/event2/bufferevent_ssl.h
else
if MBEDTLS
EVENT2_EXPORT += include/event2/bufferevent_ssl.h
endif
if MBEDTLS
EVENT2_EXPORT += include/event2/bufferevent_ssl.h
endif
endif

## Without the nobase_ prefixing, Automake would strip "include/event2/" from
Expand Down

0 comments on commit 3da7185

Please sign in to comment.