From: John Ellson <ellson@research.att.com>
Date: Tue, 16 Aug 2016 16:03:03 +0000 (-0400)
Subject: remove -D_POSIX_C_SOURCE=200112L as it breaks ./configure test for php.h
X-Git-Tag: untagged-d177ed11e0540b07902a
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=788583c2dde4d8eeff6afca9c23bde567f27473b;p=graphviz

remove -D_POSIX_C_SOURCE=200112L as it breaks ./configure test for php.h
---

diff --git a/configure.ac b/configure.ac
index 04013fd4e..72833ba31 100644
--- a/configure.ac
+++ b/configure.ac
@@ -330,14 +330,16 @@ then
   CFLAGS="${CFLAGS} -Wmissing-include-dirs -Wswitch-default -Wfloat-equal -Wundef -Wshadow -Wpointer-arith -Wbad-function-cast -Wcast-qual -Wconversion -Wstrict-prototypes -Wmissing-prototypes -Wnested-externs"
 
  # if C99, then enable additional warnings
- if [test "${ac_cv_prog_cc_c99}" != "no"]
- then
+# if [test "${ac_cv_prog_cc_c99}" != "no"]
+# then
     # When enabling c99 on this codebase, this POSIX version should be defined
-    CFLAGS="${CFLAGS} -D_POSIX_C_SOURCE=200112L"
+
+    # FIXME - enabling this flag breaks configure test for php.h
+    # CFLAGS="${CFLAGS} -D_POSIX_C_SOURCE=200112L"
 
     # FIXME - not available on centos[56]: gcc-4.1.2, gcc-4.4.7
-    # -Wdouble-promotion -Wtrampolines -Wlogical-op
-  fi
+    # CFLAGS="${CFLAGS} -Wdouble-promotion -Wtrampolines -Wlogical-op"
+#  fi
 fi
 # Workaround for native compilers
 #  HP  : http://bugs.gnome.org/db/31/3163.html
diff --git a/graphviz.spec.in b/graphviz.spec.in
index 8a5d371a1..c3735818c 100644
--- a/graphviz.spec.in
+++ b/graphviz.spec.in
@@ -166,10 +166,9 @@ BuildRequires: libwebp-devel
 %define _GO 1
 BuildRequires: golang >= 1.2 gcc-go swig >= 3.0.2
 %endif
-%if %fedora <= 23
+%if %fedora <= 24
 %define PHP 1
 BuildRequires: php-devel
-# disable PHP5 in fc24 until we can figure out what broke
 # disable PHP7 in fc25 until swig supports it
 %endif
 %endif