URL: https://github.com/SSSD/sssd/pull/212 Author: lslebodn Title: #212: intg: fix configure failure with strict cflags Action: opened
PR body: """ The warning -Wstrict-prototypes is a part of AM_CFLAGS which was appended for CFLAGS in make target intgcheck-prepare. And combination with strict CFLAGS in environment variable (e.g. -Werror) caused failures.
sh$ CFLAGS="-Werror" make intgcheck-prepare
checking for gcc... gcc checking whether the C compiler works... no configure: error: in `/home/build/sssd/ci-build-debug/intg/bld': configure: error: C compiler cannot create executables
configure:3719: checking whether the C compiler works configure:3741: gcc -g3 -O2 -Werror -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -Wall -Wshadow -Wstrict-prototypes -Wpointer-arith -Wcast-qual -Wcast-align -Wwrite-s trings -Wundef -Werror-implicit-function-declaration -Winit-self -Wmissing-include-dirs -fno-strict-aliasing -std=gnu99 -DKCM_PEER_UID=1000 conftest.c >&5 conftest.c:11:1: error: function declaration isn't a prototype [-Werror=strict-prototypes] main () ^~~~ cc1: all warnings being treated as errors """
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/212/head:pr212 git checkout pr212
URL: https://github.com/SSSD/sssd/pull/212 Title: #212: intg: fix configure failure with strict cflags
lslebodn commented: """ @pbrezina it should fix your use-case. I am sorry for troubles. Jakub just squashed a change which I proposed. I do not notice any failure because I do not use -Werror in env variable but I pass it to configure as a parameters. `CFLAGS="-Werror" ./configure` vs `./configure CFLAGS="-Werror"`
But it is a bug. """
See the full comment at https://github.com/SSSD/sssd/pull/212#issuecomment-289728246
URL: https://github.com/SSSD/sssd/pull/212 Title: #212: intg: fix configure failure with strict cflags
pbrezina commented: """ This fixed my issue. Ack. """
See the full comment at https://github.com/SSSD/sssd/pull/212#issuecomment-289730474
URL: https://github.com/SSSD/sssd/pull/212 Title: #212: intg: fix configure failure with strict cflags
Label: +Accepted
URL: https://github.com/SSSD/sssd/pull/212 Title: #212: intg: fix configure failure with strict cflags
lslebodn commented: """ http://sssd-ci.duckdns.org/logs/job/65/95/summary.html """
See the full comment at https://github.com/SSSD/sssd/pull/212#issuecomment-289782747
URL: https://github.com/SSSD/sssd/pull/212 Author: lslebodn Title: #212: intg: fix configure failure with strict cflags Action: closed
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/212/head:pr212 git checkout pr212
URL: https://github.com/SSSD/sssd/pull/212 Title: #212: intg: fix configure failure with strict cflags
lslebodn commented: """ master: * 8e785c7478e1a79179842106a62f3f85118b6690 """
See the full comment at https://github.com/SSSD/sssd/pull/212#issuecomment-289784484
URL: https://github.com/SSSD/sssd/pull/212 Title: #212: intg: fix configure failure with strict cflags
Label: +Pushed
sssd-devel@lists.fedorahosted.org