> On Feb. 21, 2015, 12:24 p.m., Corey Farrell wrote: > > /branches/11/configure.ac, line 1065 > > <https://reviewboard.asterisk.org/r/4370/diff/3/?file=71557#file71557line1065> > > > > I missed this before / thought Josh pointed it out. This one should be > > AC_MSG_ERROR as well. > > Matt Jordan wrote: > Replacing this with: > AC_MSG_ERROR("nested-functions is required for gcc") > > Caused configure to fail: > > checking for -Wshadow... yes > checking for -march=native support... yes > checking for gcc -fnested-functions... configure: error: > "nested-functions is required for gcc" > > Even though I did have nested functions.... > > > So that's weird. >
I'm actually not sure what -fnested-functions would even do. My google-foo is failing me on its use with GCC (which appears to not understand it), and since we only set that on the "false" path (that is, we failed to compile a program with a nested function), I'm not sure how it would help. As it is, I think I will put an AC_MSG_ERROR in the 'false' path of the compilation. - Matt ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4370/#review14519 ----------------------------------------------------------- On Feb. 20, 2015, 8:35 p.m., Matt Jordan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/4370/ > ----------------------------------------------------------- > > (Updated Feb. 20, 2015, 8:35 p.m.) > > > Review request for Asterisk Developers and Diederik de Groot. > > > Bugs: ASTERISK-20850 > https://issues.asterisk.org/jira/browse/ASTERISK-20850 > > > Repository: Asterisk > > > Description > ------- > > This is a continuation of the patch put up for review on r3488. It addresses > the issues found on that review. > > This patch *should* make Asterisk compile under clang. Note that compiling > with --enable-dev-mode will cause Asterisk to fail to compile under clang, as > it detects a number of warnings that aren't fixed under this patch. > > > Diffs > ----- > > /branches/11/makeopts.in 432053 > /branches/11/main/Makefile 432053 > /branches/11/include/asterisk/utils.h 432053 > /branches/11/include/asterisk/inline_api.h 432053 > /branches/11/configure.ac 432053 > /branches/11/configure UNKNOWN > /branches/11/Makefile 432053 > > Diff: https://reviewboard.asterisk.org/r/4370/diff/ > > > Testing > ------- > > * Compiled Asterisk with and without --enable-dev-mode using gcc. Asterisk > compiles correctly. > * Compiled Asterisk without --enable-dev-mode using clang. Asterisk compiles, > links, and executes. > > Note that you will need the BlocksRuntime to run Asterisk when it is compiled > with clang. > > > Thanks, > > Matt Jordan > >
-- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
