Skip to content

Commit dfd7e10

Browse files
authored
add SunOS to *BSD flags case (#338)
Slight change to get the .so building on OmniOS CE r151054 x86_64. It should work on OpenIndiana, SmartOS, other illumos distros that report `SunOS` from `uname -s` (helios?) The NIF is working in my application so far, both with the system `sqlite-3` pkg and without. However, to get the tests in this project to pass, I had to make changes to: * mindreframer/ex_sqlean@main...kenichi:ex_sqlean:sunos * https://github.com/mindreframer/sqlean (unpushed, curious about diffs w/ upstream) I had to allow the test script in sqlean (fork) to update expectations, then it passed. With the .so files built and those other changes all reflected locally: ``` Running ExUnit with seed: 614009, max_cases: 8 Excluding tags: [:slow_test] ............................................................................................................................... Finished in 5.9 seconds (0.00s async, 5.9s sync) 11 doctests, 117 tests, 0 failures, 1 excluded ```
1 parent 3aad0e9 commit dfd7e10

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ else
7777
LDFLAGS += -fPIC -shared
7878
LIB_NAME = $(PREFIX)/sqlite3_nif.dll
7979
endif
80-
ifeq ($(KERNEL_NAME), $(filter $(KERNEL_NAME),OpenBSD FreeBSD NetBSD))
80+
ifeq ($(KERNEL_NAME), $(filter $(KERNEL_NAME),OpenBSD FreeBSD NetBSD SunOS))
8181
CFLAGS += -fPIC
8282
LDFLAGS += -fPIC -shared
8383
endif

0 commit comments

Comments
 (0)