Skip to content

add lua#6

Open
firejox wants to merge 1 commit intoc910335:masterfrom
firejox:lua
Open

add lua#6
firejox wants to merge 1 commit intoc910335:masterfrom
firejox:lua

Conversation

@firejox
Copy link

@firejox firejox commented Oct 14, 2017

No description provided.

- sudo apt-get install -y ruby default-jdk haskell-platform python3
- wget http://www.lua.org/ftp/lua-5.3.4.tar.gz
- tar -zxv -f lua-5.3.4.tar.gz
- pushd lua-5.3.4 && make CFLAGS="-O2 -fPIC -DLUA_COMPAT_5_2 -DLUA_COMPAT_5_1" linux && sudo make INSTALL_TOP=/usr install && popd
Copy link
Owner

Choose a reason for hiding this comment

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

Why not install lua with apt?

Copy link
Author

Choose a reason for hiding this comment

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

It requires lua5.3, and there are only lua5.0, lua 5.1, and lua5.2 in apt.

main.lua Outdated
while true do
local ranks = {}

function find_ranks(...)
Copy link
Owner

Choose a reason for hiding this comment

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

Why not just pass pts but table.unpack(pts) with ...?

Copy link
Author

Choose a reason for hiding this comment

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

It is bad idea to use ... ,and I will change it.

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