Use -std=c99 when building with CMake

Disable the GNU extensions in CMake which is ON by default
for some reason. This is the same as in Makefile builds.
Enable GNU extensions where required.

Signed-off-by: Björn Svensson <[email protected]>

CMakeLists.txt

@@ -25,8 +25,9 @@ OPTION(ENABLE_EXAMPLES "Enable building valkey examples" OFF)
 option(ENABLE_IPV6_TESTS "Enable IPv6 tests requiring special prerequisites" OFF)
 OPTION(ENABLE_RDMA "Build valkey_rdma for RDMA support" OFF)
 
-# valkey requires C99
+# Libvalkey requires C99 (-std=c99)
 SET(CMAKE_C_STANDARD 99)
+set(CMAKE_C_EXTENSIONS OFF)
 SET(CMAKE_DEBUG_POSTFIX d)
 
 # Set target-common flags

examples/async-libsdevent.c

@@ -1,3 +1,4 @@
+#define _XOPEN_SOURCE 600 /* Required by libsdevent (CLOCK_MONOTONIC) */
 #include <valkey/async.h>
 #include <valkey/valkey.h>
 

examples/async-libuv.c

@@ -1,3 +1,4 @@
+#define _XOPEN_SOURCE 600 /* Required by libuv (pthread_rwlock_t) */
 #include <valkey/async.h>
 #include <valkey/valkey.h>
 

examples/blocking.c

@@ -1,3 +1,4 @@
+#define _XOPEN_SOURCE 600 /* For strdup() */
 #include <valkey/valkey.h>
 
 #include <stdio.h>

src/cluster.c

@@ -30,7 +30,7 @@
  * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
  * POSSIBILITY OF SUCH DAMAGE.
  */
-#define _XOPEN_SOURCE 600
+#include "fmacros.h"
 #include "win32.h"
 
 #include "cluster.h"

tests/client_test.c

@@ -1,6 +1,5 @@
-#if !defined(__FreeBSD__)
-#define _POSIX_C_SOURCE 200112L
-#endif
+#include "fmacros.h"
+
 #include "sockcompat.h"
 
 #include <stdio.h>

tests/ct_async_libuv.c

@@ -1,3 +1,4 @@
+#define _XOPEN_SOURCE 600 /* Required by libuv (pthread_rwlock_t) */
 #include "adapters/libuv.h"
 #include "cluster.h"
 #include "test_utils.h"

tests/ct_commands.c

@@ -1,3 +1,4 @@
+#define _XOPEN_SOURCE 600 /* For strdup() */
 #include "cluster.h"
 #include "test_utils.h"
 

tests/ut_parse_cmd.c

@@ -1,5 +1,6 @@
 /* Some unit tests that don't require Valkey to be running. */
 
+#include "fmacros.h"
 #include "win32.h"
 
 #include "cluster.h"