> On Jun 17, 2015, at 5:42 PM, John Ralls <[email protected]> wrote: > > >> On Jun 17, 2015, at 4:03 PM, Alex Aycinena <[email protected]> wrote: >> >> John, >> >> I just noticed that in the configure.ac <http://configure.ac/> file, line >> 332 says 'ac_cv_cstd=gnu99' but line 336 says 'ac_cv_cstd=c11'. If I change >> line 336 locally to also say 'ac_cv_cstd=gnu99', then autogen.sh does >> regenerate configure with CFLAGS set to gnu99 and make, make-check and >> make-install all work on the F22 machine. Should that be changed on master, >> and, if so, will you do it? Or would you like me to do it? > > Alex, > > That check is there because certain versions of GLib require C11 because > someone introduced a C11 feature into one of the header files. Ryan Lortie > took that out a few weeks ago, so if GC is building OK then the version of > GLib that F22 includes must be after Ryan’s commit or be patched to reflect > it. I guess that means we need to use AC_COMPILE_IFELSE instead of simply > checking the GLib version. > > http://stackoverflow.com/questions/19641460/using-strdup-in-c11 > <http://stackoverflow.com/questions/19641460/using-strdup-in-c11> suggests > that strdup isn’t part of C11 after all and that the correct solution is to > use gnu11 instead of c11. Can you test that? If it works go ahead and push > that change and I’ll do the compile check on the GLib header tomorrow.
Alex, I’ve pushed a commit that uses gnu11 instead of c11 and checks for being able to include gio.h to decide which to use. That should work better with F22, and is regardless a better approach than pinning on GLib version numbers. Regards, John Ralls _______________________________________________ gnucash-devel mailing list [email protected] https://lists.gnucash.org/mailman/listinfo/gnucash-devel
