Skip to content

Commit

Permalink
clar: define feature macros in "clar.c"
Browse files Browse the repository at this point in the history
Compiling clar requires several different extensions which are not part
of ISO C90, like strdup(3P), lstat(3P) and mkdtemp(3P). And while we
define these via our CMake build instructions, downstream users do not
benefit from this and will have to make sure that they set these.

Let's instead pull the definitions into "clar.c" such that the macros
are defined for everyone.
  • Loading branch information
pks-t committed Oct 11, 2024
1 parent f065214 commit 4f746f4
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
3 changes: 3 additions & 0 deletions clar.c
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,9 @@
* For full terms see the included COPYING file.
*/

#define _DARWIN_C_SOURCE
#define _POSIX_C_SOURCE 200809L

#include <errno.h>
#include <setjmp.h>
#include <stdlib.h>
Expand Down
2 changes: 0 additions & 2 deletions test/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,6 @@ target_sources(clar_test PRIVATE
)
target_compile_definitions(clar_test PRIVATE
CLAR_FIXTURE_PATH="${CMAKE_CURRENT_SOURCE_DIR}/resources/"
_DARWIN_C_SOURCE
_POSIX_C_SOURCE=200809L
)
target_compile_options(clar_test PRIVATE
$<IF:$<CXX_COMPILER_ID:MSVC>,/W4,-Wall>
Expand Down

0 comments on commit 4f746f4

Please sign in to comment.