Re: [Monetdb-developers] MonetDB: default - Remove the C flag -Wno-uninitialized for Mac ...
Good point. Thanks! I don't expect any problems with this change, mostly because we don't have any MacOS X machine for testing, anymore; only one for buildbot --- once that's running again,we'll see ... Assuming that compilation works for you on a "modern" MacOS X (Snow Leopard), I assume that the mentioned bison.simple problems were related/specific to an older version of MacOS X, only. (MacOS X was never a primary develoment platform for us, that why disabling -Wuninitialized (by enabling -Wno-uninitialized) on MacOS X used to be - "acceptable" for us; obviously, with you using MacOS X as a primary development platform, -Wno-uninitialized should indeed not be used - (gloablly) ...) Stefan On Thu, May 20, 2010 at 06:20:02PM +0200, Jan Rittinger wrote:
Changeset: 892fc9b323aa for MonetDB URL: http://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=892fc9b323aa Modified Files: buildtools/conf/MonetDB.m4 Branch: default Log Message:
Remove the C flag -Wno-uninitialized for Mac operating systems. This helps to detect uninitialized variables on OS X.
(Please undo the fix in case it breaks anything.)
diffs (12 lines):
diff -r b181da69e3ba -r 892fc9b323aa buildtools/conf/MonetDB.m4 --- a/buildtools/conf/MonetDB.m4 Thu May 20 17:50:15 2010 +0200 +++ b/buildtools/conf/MonetDB.m4 Thu May 20 18:18:33 2010 +0200 @@ -811,7 +811,7 @@ dnl generates "X_CFLAGS =" in the generated Makefile. NO_X_CFLAGS='X_CFLAGS' ;; - *-solaris*|*-darwin*|*-aix*) + *-solaris*|*-aix*) dnl In some cases, there is a (possibly) uninitialized dnl variable in bison.simple ... |-( X_CFLAGS="$X_CFLAGS -Wno-uninitialized" _______________________________________________ Checkin-list mailing list Checkin-list@monetdb.org http://mail.monetdb.org/mailman/listinfo/checkin-list
-- | Dr. Stefan Manegold | mailto:Stefan.Manegold@cwi.nl | | CWI, P.O.Box 94079 | http://www.cwi.nl/~manegold/ | | 1090 GB Amsterdam | Tel.: +31 (20) 592-4212 | | The Netherlands | Fax : +31 (20) 592-4199 |
participants (1)
-
Stefan Manegold