From 788583c2dde4d8eeff6afca9c23bde567f27473b Mon Sep 17 00:00:00 2001 From: John Ellson Date: Tue, 16 Aug 2016 12:03:03 -0400 Subject: [PATCH] remove -D_POSIX_C_SOURCE=200112L as it breaks ./configure test for php.h --- configure.ac | 12 +++++++----- graphviz.spec.in | 3 +-- 2 files changed, 8 insertions(+), 7 deletions(-) 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 -- 2.40.0