Fix build on macOS if /usr/local/include/db.h exists

lichray / nvi2

A multibyte fork of the nvi editor for BSD

Other

144 stars 34 forks source link

Fix build on macOS if /usr/local/include/db.h exists #91

Closed arichardson closed 3 years ago

arichardson commented 3 years ago

Otherwise #include will end up including /usr/local/include/db.h instead of the file from the sysroot. Work around this by passing the path to db.h using a DB_H_ABS_PATH define and including that.

Also don't add -Wsystem-headers on macOS since this produces 180 -Wnullability-completeness warnings for each file and makes it impossible to see real warnings.

CC: @jrtc27

lichray commented 3 years ago

Is #69 addressed by this?

arichardson commented 3 years ago

I works for me on 10.15, so I would say yes.