Re: [Warzone-dev] [Warzone-commits] r3986 - in /trunk: ./ lib/framework/ lib/gamelib/ lib/ivis_common/ lib/ivis_opengl/ lib/netplay/ lib/script/ lib/sequence/ lib/sound/ lib/sqlite3/ lib/widget/ src/

2008-03-10 Thread Giel van Schijndel
Dennis Schridde schreef:
 Author: devurandom
 Date: Mon Mar 10 03:01:46 2008
 New Revision: 3986
 
 URL: http://svn.gna.org/viewcvs/warzone?rev=3986view=rev
 Log:
 - Don't define CFLAGS anymore, export WZ_CFLAGS instead, which can be 
 overridden by make commandlines
 - Set -DYY_NO_INPUT only in those places where needed
 - Add some previously forgotten CFLAGS where needed
 - Remove extranous -lintl from WIN32_LIBS
 - Remove apparently unnecessary directory from Mac include (png_util doesn't 
 need it either)
 
 Modified:
 trunk/configure.ac
 trunk/lib/framework/Makefile.am
 trunk/lib/gamelib/Makefile.am
 trunk/lib/ivis_common/Makefile.am
 trunk/lib/ivis_common/png_util.c
 trunk/lib/ivis_opengl/Makefile.am
 trunk/lib/netplay/Makefile.am
 trunk/lib/script/Makefile.am
 trunk/lib/sequence/Makefile.am
 trunk/lib/sound/Makefile.am
 trunk/lib/sqlite3/Makefile.am
 trunk/lib/widget/Makefile.am
 trunk/src/Makefile.am

From this revision onward I get compiler warnings (which are fatal due
to -Werror):

 gcc -std=gnu99 -DHAVE_CONFIG_H -I. -I../..-D_GNU_SOURCE=1 -D_REENTRANT 
 -I/usr/include/SDL   -O0 -g -Wall -Werror -Wno-unused-label 
 -Wno-pointer-to-int-cast -Wmissing-field-initializers 
 -Wno-missing-field-initializers -g -O2 -MT sqlite3.o -MD -MP -MF 
 .deps/sqlite3.Tpo -c -o sqlite3.o sqlite3.c
 cc1: warnings being treated as errors
 sqlite3.c: In function 'sqlite3BtreeFactory':
 sqlite3.c:72670: warning: passing argument 2 of 'sqlite3BtreeOpen' discards 
 qualifiers from pointer target type
 make[3]: *** [sqlite3.o] Error 1
 make[3]: Leaving directory `/home/mortis/svn/warzone/wz-fresh/lib/sqlite3'

And if I circumvent sqlite3.o (by manually touching it):

 gcc -std=gnu99 -DHAVE_CONFIG_H -I. -I..  -DYY_NO_INPUT -DDEBUG 
 -DDATADIR=\/usr/local/share/warzone2100\ 
 -DLOCALEDIR=\/usr/local/share/locale\ -DYY_NO_INPUT -I..  -D_GNU_SOURCE=1 
 -D_REENTRANT -I/usr/include/SDL -DPNG_NO_MMX_CODE -I/usr/include/libpng12 
-O0 -g -Wall -Werror -Wno-unused-label -Wno-pointer-to-int-cast 
 -Wmissing-field-initializers -g -O2 -MT droid.o -MD -MP -MF .deps/droid.Tpo 
 -c -o droid.o droid.c
 cc1: warnings being treated as errors
 droid.c: In function 'checkDroidsBuilding':
 droid.c:4522: warning: dereferencing type-punned pointer will break 
 strict-aliasing rules
 droid.c: In function 'checkDroidsDemolishing':
 droid.c:4542: warning: dereferencing type-punned pointer will break 
 strict-aliasing rules
 droid.c: In function 'vtolReadyToRearm':
 droid.c:4911: warning: dereferencing type-punned pointer will break 
 strict-aliasing rules
 make[1]: *** [droid.o] Error 1
 make[1]: Leaving directory `/home/mortis/svn/warzone/wz-fresh/src'

-- 
Giel



signature.asc
Description: OpenPGP digital signature
___
Warzone-dev mailing list
Warzone-dev@gna.org
https://mail.gna.org/listinfo/warzone-dev


Re: [Warzone-dev] [Warzone-commits] r3986 - in /trunk: ./ lib/framework/ lib/gamelib/ lib/ivis_common/ lib/ivis_opengl/ lib/netplay/ lib/script/ lib/sequence/ lib/sound/ lib/sqlite3/ lib/widget/ src/

2008-03-10 Thread Dennis Schridde
Am Montag, 10. März 2008 12:07:44 schrieb Giel van Schijndel:
 Dennis Schridde schreef:
  Author: devurandom
  Date: Mon Mar 10 03:01:46 2008
  New Revision: 3986
 
  URL: http://svn.gna.org/viewcvs/warzone?rev=3986view=rev
  Log:
  - Don't define CFLAGS anymore, export WZ_CFLAGS instead, which can be
  overridden by make commandlines - Set -DYY_NO_INPUT only in those places
  where needed
  - Add some previously forgotten CFLAGS where needed
  - Remove extranous -lintl from WIN32_LIBS
  - Remove apparently unnecessary directory from Mac include (png_util
  doesn't need it either)
 
  Modified:
  trunk/configure.ac
  trunk/lib/framework/Makefile.am
  trunk/lib/gamelib/Makefile.am
  trunk/lib/ivis_common/Makefile.am
  trunk/lib/ivis_common/png_util.c
  trunk/lib/ivis_opengl/Makefile.am
  trunk/lib/netplay/Makefile.am
  trunk/lib/script/Makefile.am
  trunk/lib/sequence/Makefile.am
  trunk/lib/sound/Makefile.am
  trunk/lib/sqlite3/Makefile.am
  trunk/lib/widget/Makefile.am
  trunk/src/Makefile.am

 From this revision onward I get compiler warnings (which are fatal due
 to -Werror):
For sqlite3 I fixed them by disabling -Werror.
The rest is caused by autoconf defaulting to -O2 for CFLAGS... *doh!*
I forgot that that would happen...
But since the autoconf gurus strongly suggested not touching CFLAGS, I guess 
I'll go and fix the few remaining type-punning warnings, which are actually 
quite fatal. (type-punning can lead to incorrect generated code)

--Dennis


signature.asc
Description: This is a digitally signed message part.
___
Warzone-dev mailing list
Warzone-dev@gna.org
https://mail.gna.org/listinfo/warzone-dev