Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix docker build error on MacOS Ventura 13.4.1 #20

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

prawwtocol
Copy link

@prawwtocol prawwtocol commented Aug 23, 2023

Fixes this error:

7.129 Error: Could not find header file for OPENSSL
7.129   No file openssl/ssl.h in /usr/local/include
7.129   No file openssl/ssl.h in /include
7.129 You may have to install OPENSSL in your system and/or pass OPENSSL_DIR or OPENSSL_INCDIR to the luarocks command.
7.129 Example: luarocks install luasec OPENSSL_DIR=/usr/local
------
Dockerfile:29
--------------------
  28 |     # libssl.* are in /usr/lib/x86_64-linux-gnu on Travis Ubuntu precise
  29 | >>> RUN luarocks install --verbose luasocket \
  30 | >>>     && luarocks install luasec OPENSSL_LIBDIR=/usr/lib/x86_64-linux-gnu \
  31 | >>>     && luarocks install redis-lua \
  32 | >>>     && luarocks install busted \
  33 | >>>     && rm -rf /tmp/*
  34 |
--------------------

This change is Reviewable

@@ -27,7 +27,7 @@ RUN openssl req -x509 -nodes -days 365 -newkey rsa:2048 \

# libssl.* are in /usr/lib/x86_64-linux-gnu on Travis Ubuntu precise
RUN luarocks install --verbose luasocket \
&& luarocks install luasec OPENSSL_LIBDIR=/usr/lib/x86_64-linux-gnu \
&& OPENSSL_LIBDIR=/usr/lib/x86_64-linux-gnu OPENSSL_DIR=/usr/ luarocks install luasec \
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you try this instead?

Suggested change
&& OPENSSL_LIBDIR=/usr/lib/x86_64-linux-gnu OPENSSL_DIR=/usr/ luarocks install luasec \
&& luarocks install luasec \

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants