On 04/10/15 02:10 pm, Erik de Castro Lopo wrote: > Tristan Matthews wrote: > >> --- >> configure.ac | 4 +++- >> 1 file changed, 3 insertions(+), 1 deletion(-) >> >> diff --git a/configure.ac b/configure.ac >> index eb9b0cc..e7d68c3 100644 >> --- a/configure.ac >> +++ b/configure.ac >> @@ -399,9 +399,11 @@ if test x$ac_cv_c_compiler_gnu = xyes ; then >> >> if test "x$asm_optimisation$sse_os" = "xyesyes" ; then >> XIPH_ADD_CFLAGS([-msse2]) >> - XIPH_ADD_CFLAGS([-mstackrealign]) >> fi >> + fi >> >> +if test "$host_os" = "mingw32" ; then >> + XIPH_ADD_CFLAGS([-mstackrealign]) >> fi >> >> XIPH_ADD_CFLAGS([-Wextra]) > > Everyone happy with this patch? >
Should it be a case statement so other OSes can be added or just use || ? OS/2 should also have the -mstackrealign option added as well. Dave _______________________________________________ flac-dev mailing list flac-dev@xiph.org http://lists.xiph.org/mailman/listinfo/flac-dev