-
Notifications
You must be signed in to change notification settings - Fork 5
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
lua: Make building more generic (build for any arch) #81
base: master
Are you sure you want to change the base?
Conversation
Default version changed from 5.3.5 to 5.3.6. Added port build options: - PORTS_LUA_VERSION - version to download (must be supported by respective patches); - PORTS_LUA_STACK_SIZE - size of Lua's internal stack; - PORTS_LUA_CSTACK_SIZE - size of stack (when using linked lua binary); - PORTS_LUA_COMPAT_5_2 - backward compatibility with Lua 5.2; - PORTS_LUA_DEBUG - enable debug features, e.g. API checks; - PORTS_LUA_INSTALL_TESTS - download and install external Lua test scripts; - PORTS_LUA_RESTRAIN - disable Lua features which may be used to crash the interpreter or perform side-effects in the operating system. JIRA: NIL-595
fi | ||
|
||
mycflags=( | ||
${CFLAGS} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fi | ||
|
||
mycflags=( | ||
${CFLAGS} | ||
-DLUAI_MAXSTACK=${PORTS_LUA_STACK_SIZE:-2000} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if [ "${PORTS_LUA_COMPAT_5_2}" = "y" ]; then mycflags+=("-DLUA_COMPAT_5_2"); fi | ||
if [ "${PORTS_LUA_DEBUG}" = "y" ]; then mycflags+=("-DLUA_USE_APICHECK"); fi | ||
|
||
myldflags=( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if [ "${PORTS_LUA_DEBUG}" = "y" ]; then mycflags+=("-DLUA_USE_APICHECK"); fi | ||
|
||
myldflags=( | ||
${LDFLAGS} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
||
myldflags=( | ||
${LDFLAGS} | ||
-Wl,-z,stack-size=${PORTS_LUA_CSTACK_SIZE:-4096} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fi | ||
|
||
mycflags=( | ||
${CFLAGS} | ||
-DLUAI_MAXSTACK=${PORTS_LUA_STACK_SIZE:-2000} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LUAI_MAXSTACK
passed to lualib should be compatible with the one that is given to applications (as it is used in one of the header files of Lua API). The value of PORTS_LUA_STACK_SIZE
will be used outside of ports too.
Default version changed from 5.3.5 to 5.3.6.
Added port build options:
JIRA: NIL-595
Description
Motivation and Context
Types of changes
How Has This Been Tested?
Checklist:
Special treatment