1016
93%

Ran 05 Oct 2017 11:56PM UTC

Jobs 3

Files 12

Run time 12min

Badge

Embed ▾

pending completion

Build # 1016

Build Type

Pull #109

travis-ci

Committed by

web-flow

Commit Message

Allow installing rocks locally - fix openssl include

/usr/local is not writable on my machine by purpose, so `make dev` fails
with permission error. I've changed the Makefile to allow installing
rocks in the user's home directory. This is optional, so by default `make
dev` will have the same behaviour as now.

I'm not super happy with this solution either, imo the right thing would
be to create a local tree inside this directory and install the
dependencies there instead of polluting the system. That's possible and
it works, however it required me to set the lua load paths in several
places and wasn't "backward compatible", so I chose to go with this
solution for now which should be easier to code-review.

Another issue that I encountered is that openssl header files on macOS
are not inside `/usr/local`, which mean I couldn't install luasec. To
fix this I've added an optional OPENSSL_DIR env variable that can be
used to configure luarocks. (This is the suggested procedure on luasec's
website).

I've also added luasec and luasocket as lua-cassandra dependencies since
without them the tests break. There doesn't seem to be a way to mark
luasec as optional (since it's only require if you enable ssl)
unfortunately.

Pull Request Pull Request #109: Allow installing rocks locally - fix openssl include

Run Details

1186 of 1560 relevant lines covered (76.03%)

736.8 hits per line

Uncovered Existing Lines

Lines	Coverage	File
3	100.0	./lib/cassandra/auth.lua
53	100.0	./lib/cassandra/init.lua
231	100.0	./lib/cassandra/cql.lua

Jobs

ID	Job ID	Ran	Coverage
5	1016.5 (OPENRESTY_TESTS=true CASSANDRA=3.9)	05 Oct 2017 11:56PM UTC	74.87	Travis Job 1016.5
6	1016.6 (OPENRESTY_TESTS=true CASSANDRA=2.2.8)	06 Oct 2017 12:04AM UTC	75.38	Travis Job 1016.6
7	1016.7 (OPENRESTY_TESTS=true CASSANDRA=2.1.16)	06 Oct 2017 12:08AM UTC	74.82	Travis Job 1016.7

Source Files on build 1016

Detailed source file information is not available for this build.

thibaultcha / lua-cassandra / 1016
93%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Uncovered Existing Lines

Jobs

Source Files on build 1016

thibaultcha / lua-cassandra / 1016 93%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Uncovered Existing Lines

Jobs

Source Files on build 1016

thibaultcha / lua-cassandra / 1016
93%

README BADGES
x