Opened 14 years ago

Closed 13 years ago

#628 closed task (worksforme)

would you like the warning messages from a build?

Reported by: david Owned by:
Priority: If Time Permits Milestone: Alpha 4
Component: Non-game systems Keywords:
Cc: Patch:

Description

I have notice a lot of warnings when compiling on mandriva linux, I was wondering if you would like the warning output of the build. Of course next time I build, or are warnings a necessary evil? David

Change History (5)

comment:1 by Philip Taylor, 14 years ago

Are they basically the same warnings as in this log? There's a few groups:

  • Warnings in third-party libraries (before the "/usr/bin/make CONFIG=Release -j4" line, and ones in wxJS directories) aren't ours to fix.
  • unused parameter warnings are intentional (they're pointing out code that hasn't been written yet).
  • ignoring return value warnings are boring (they're for error handling in code that isn't even used) but probably should be fixed.
  • archive_zip.cpp warnings are bogus (the code is correct but the compiler can't prove it); r8429 should stop them.
  • The part saying
    /usr/include/bits/string3.h: In function 'void TestMD5::T.881()':
    /usr/include/bits/string3.h:52: warning: array subscript is above array bounds
    /usr/include/bits/string3.h:52: warning: array subscript is above array bounds
    
    looks like it might just be a compiler bug - I can reproduce it with GCC 4.4.4 (it warns first from TestMD5::test_chunks first calling m.Update) but not 4.3.4 or 4.5.0, and the code looks correct as far as I can tell.

Do you get any different warnings to these?

in reply to:  1 ; comment:2 by david, 14 years ago

I didn't mean "Of course next time I build, or are warnings a necessary evil?" in any derogatory way. It's not easy making cross compiling applications.
Basically everything is being built it's just pyrogenesis_dbg segment faults after the first two speed tests. A list of the required header files would make it easier for me to find the packages I need to compile it, because packages change on a daily basis and mandriva does not utilize version packaging on every package, I don't know why they don't just for the sake of LAMP.

The thing is I cannot have both packages libJpeg62 and libjpeg version 8 installed at the same time, but I don't think that should matter. Anywho YOu can probably make more sense of this than I can.


here's the output of update-workspace.sh:


Updating bundled third-party dependencies...

bash -c 'mkdir -p output/{debug,release,test}/{FCollada/{FCDocument,FMath,FUtils,FColladaTest/{FCTestAssetManagement,FCTestExportImport,FCTestXRef}},FColladaPlugins/FArchiveXML}'
cp output/libFColladaSD.a ../lib/libFColladaSD.a
cp output/libFColladaSR.a ../lib/libFColladaSR.a

SpiderMonkey is already up to date

Building NVTT...

-- Setting optimal options
--   Processor: i686
--   Compiler Flags:  -march=i686
-- Looking for OpenGL - found
-- Looking for GLUT - found
-- Looking for DirectX - not found
-- Looking for GLEW - found
-- Looking for Cg - not found
-- Looking for CUDA - not found
-- Looking for Maya - not found
-- Looking for JPEG - found
-- Looking for PNG - found
-- Looking for TIFF - found
-- Found OpenEXR: /usr/lib/libIlmImf.so
-- Looking for OpenEXR - found
-- Use thread library: -lpthread
-- Configuring done
-- Generating done
-- Build files have been written to: /home/david/Downloads/A-svn/trunk/libraries/nvtt/src/build
[ 16%] Built target nvcore
[ 18%] Built target posh
[ 32%] Built target nvmath
[ 58%] Built target nvimage
[ 72%] Built target squish
[100%] Built target nvtt

make: Entering directory `/home/david/Downloads/A-svn/trunk/build/premake/src'
==== Building Premake ====
make[1]: `../bin/premake' is up to date.
make: Leaving directory `/home/david/Downloads/A-svn/trunk/build/premake/src'

Generating GNU makefiles:
...pyrogenesis
...network
...simulation2
...scriptinterface
...engine
...graphics
...i18n
...atlas
...gui
...lowlevel
...mocks_real
...mocks_test
...AtlasObject
...AtlasScript
...wxJS
...AtlasUI
...ActorEditor
...ColourTester
...Collada
...test_gen
...test

Here are the warnings I am getting


../../../source/simulation2/components/CCmpObstructionManager.cpp:99: warning: unused parameter ‘context’
../../../source/simulation2/components/CCmpObstructionManager.cpp:120: warning: unused parameter ‘serialize’
../../../source/simulation2/components/CCmpObstructionManager.cpp:127: warning: unused parameter ‘deserialize’
../../../source/simulation2/components/CCmpObstructionManager.cpp:661: warning: unused parameter ‘r’
../../../source/simulation2/components/CCmpMinimap.cpp:102: warning: unused parameter ‘serialize’
../../../source/simulation2/components/CCmpMinimap.cpp:107: warning: unused parameter ‘deserialize’
../../../source/ps/CConsole.cpp:719: warning: unused parameter ‘pText’
../../../source/tools/atlas/GameInterface/Handlers/ObjectHandlers.cpp:237: warning: unused parameter ‘selections’
/usr/bin/ld: warning: libjpeg.so.8, needed by /usr/lib/libtiff.so.3, may conflict with libjpeg.so.62
In file included from /usr/include/wx-2.8/wx/dcgraph.h:17,
                 from /usr/include/wx-2.8/wx/dc.h:887,
                 from /usr/include/wx-2.8/wx/wx.h:48,
                 from ../../../source/tools/atlas/AtlasScript/ScriptInterface.cpp:32:
/usr/include/wx-2.8/wx/geometry.h:91: warning: redundant redeclaration of ‘wxPoint2DInt operator*(wxInt32, const wxPoint2DInt&)’ in same scope
/usr/include/wx-2.8/wx/geometry.h:90: warning: previous declaration of ‘wxPoint2DInt operator*(wxInt32, const wxPoint2DInt&)’
/usr/include/wx-2.8/wx/geometry.h:93: warning: redundant redeclaration of ‘wxPoint2DInt operator*(const wxPoint2DInt&, wxInt32)’ in same scope
/usr/include/wx-2.8/wx/geometry.h:92: warning: previous declaration of ‘wxPoint2DInt operator*(const wxPoint2DInt&, wxInt32)’
/usr/include/wx-2.8/wx/geometry.h:96: warning: redundant redeclaration of ‘wxPoint2DInt operator/(const wxPoint2DInt&, wxInt32)’ in same scope
/usr/include/wx-2.8/wx/geometry.h:95: warning: previous declaration of ‘wxPoint2DInt operator/(const wxPoint2DInt&, wxInt32)’
In file included from /usr/include/wx-2.8/wx/dcgraph.h:17,
                 from /usr/include/wx-2.8/wx/dc.h:887,
                 from /usr/include/wx-2.8/wx/wx.h:48,
                 from /usr/include/wx-2.8/wx/wxprec.h:68,
                 from ../../../source/tools/atlas/wxJS/precompiled.h:31,
                 from ../../../source/tools/atlas/wxJS/precompiled.cpp:1:
/usr/include/wx-2.8/wx/geometry.h:91: warning: redundant redeclaration of ‘wxPoint2DInt operator*(wxInt32, const wxPoint2DInt&)’ in same scope
/usr/include/wx-2.8/wx/geometry.h:90: warning: previous declaration of ‘wxPoint2DInt operator*(wxInt32, const wxPoint2DInt&)’
/usr/include/wx-2.8/wx/geometry.h:93: warning: redundant redeclaration of ‘wxPoint2DInt operator*(const wxPoint2DInt&, wxInt32)’ in same scope
/usr/include/wx-2.8/wx/geometry.h:92: warning: previous declaration of ‘wxPoint2DInt operator*(const wxPoint2DInt&, wxInt32)’
/usr/include/wx-2.8/wx/geometry.h:96: warning: redundant redeclaration of ‘wxPoint2DInt operator/(const wxPoint2DInt&, wxInt32)’ in same scope
/usr/include/wx-2.8/wx/geometry.h:95: warning: previous declaration of ‘wxPoint2DInt operator/(const wxPoint2DInt&, wxInt32)’
../../../source/tools/atlas/wxJS/gui/control/radioboxit.cpp: In static member function ‘static bool wxjs::gui::RadioBoxItem::GetProperty(wxjs::Index*, JSContext*, JSObject*, int, jsval*)’:
../../../source/tools/atlas/wxJS/gui/control/radioboxit.cpp:68: warning: comparison between signed and unsigned integer expressions
../../../source/tools/atlas/wxJS/gui/control/radioboxit.cpp:79: warning: comparison between signed and unsigned integer expressions
../../../source/tools/atlas/wxJS/gui/control/radioboxit.cpp: In static member function ‘static bool wxjs::gui::RadioBoxItem::SetProperty(wxjs::Index*, JSContext*, JSObject*, int, jsval*)’:
../../../source/tools/atlas/wxJS/gui/control/radioboxit.cpp:109: warning: comparison between signed and unsigned integer expressions
../../../source/tools/atlas/wxJS/gui/control/item.cpp: In static member function ‘static bool wxjs::gui::ControlItem::GetProperty(wxjs::Index*, JSContext*, JSObject*, int, jsval*)’:
../../../source/tools/atlas/wxJS/gui/control/item.cpp:84: warning: comparison between signed and unsigned integer expressions
../../../source/tools/atlas/wxJS/gui/control/item.cpp: In static member function ‘static bool wxjs::gui::ControlItem::SetProperty(wxjs::Index*, JSContext*, JSObject*, int, jsval*)’:
../../../source/tools/atlas/wxJS/gui/control/item.cpp:108: warning: comparison between signed and unsigned integer expressions
../../../source/tools/atlas/wxJS/gui/control/item.cpp: In static member function ‘static JSBool wxjs::gui::ControlItem::remove(JSContext*, JSObject*, uintN, jsval*, jsval*)’:
../../../source/tools/atlas/wxJS/gui/control/item.cpp:145: warning: comparison between signed and unsigned integer expressions
../../../source/tools/atlas/wxJS/gui/control/item.cpp: In static member function ‘static JSBool wxjs::gui::ControlItem::select(JSContext*, JSObject*, uintN, jsval*, jsval*)’:
../../../source/tools/atlas/wxJS/gui/control/item.cpp:172: warning: comparison between signed and unsigned integer expressions
../../../source/tools/atlas/wxJS/gui/misc/rect.cpp: In static member function ‘static JSBool wxjs::gui::Rect::inside(JSContext*, JSObject*, uintN, jsval*, jsval*)’:
../../../source/tools/atlas/wxJS/gui/misc/rect.cpp:464: warning: ‘bool wxRect::Inside(int, int) const’ is deprecated (declared at /usr/include/wx-2.8/wx/gdicmn.h:490)
../../../source/tools/atlas/wxJS/gui/misc/rect.cpp:477: warning: ‘bool wxRect::Inside(const wxPoint&) const’ is deprecated (declared at /usr/include/wx-2.8/wx/gdicmn.h:491)
../../../source/tools/atlas/wxJS/gui/misc/sizer.cpp: In static member function ‘static JSBool wxjs::gui::Sizer::remove(JSContext*, JSObject*, uintN, jsval*, jsval*)’:
../../../source/tools/atlas/wxJS/gui/misc/sizer.cpp:534: warning: ‘virtual bool wxSizer::Remove(wxWindow*)’ is deprecated (declared at /usr/include/wx-2.8/wx/sizer.h:513)
../../../source/tools/atlas/wxJS/io/textline.cpp: In static member function ‘static bool wxjs::io::TextLine::GetProperty(wxjs::Index*, JSContext*, JSObject*, int, jsval*)’:
../../../source/tools/atlas/wxJS/io/textline.cpp:102: warning: comparison between signed and unsigned integer expressions
../../../source/tools/atlas/wxJS/io/textline.cpp: In static member function ‘static bool wxjs::io::TextLine::SetProperty(wxjs::Index*, JSContext*, JSObject*, int, jsval*)’:
../../../source/tools/atlas/wxJS/io/textline.cpp:134: warning: comparison between signed and unsigned integer expressions
../../../source/tools/atlas/wxJS/io/textline.cpp: In static member function ‘static bool wxjs::io::TextLine::Enumerate(wxjs::Index*, JSContext*, JSObject*, JSIterateOp, jsval*, jsid*)’:
../../../source/tools/atlas/wxJS/io/textline.cpp:186: warning: comparison between signed and unsigned integer expressions
In file included from /usr/include/wx-2.8/wx/dcgraph.h:17,
                 from /usr/include/wx-2.8/wx/dc.h:887,
                 from /usr/include/wx-2.8/wx/wx.h:48,
                 from /usr/include/wx-2.8/wx/wxprec.h:68,
                 from ../../../source/tools/atlas/AtlasUI/Misc/precompiled.h:43,
                 from ../../../source/tools/atlas/AtlasUI/Misc/precompiled.cpp:18:
/usr/include/wx-2.8/wx/geometry.h:91: warning: redundant redeclaration of ‘wxPoint2DInt operator*(wxInt32, const wxPoint2DInt&)’ in same scope
/usr/include/wx-2.8/wx/geometry.h:90: warning: previous declaration of ‘wxPoint2DInt operator*(wxInt32, const wxPoint2DInt&)’
/usr/include/wx-2.8/wx/geometry.h:93: warning: redundant redeclaration of ‘wxPoint2DInt operator*(const wxPoint2DInt&, wxInt32)’ in same scope
/usr/include/wx-2.8/wx/geometry.h:92: warning: previous declaration of ‘wxPoint2DInt operator*(const wxPoint2DInt&, wxInt32)’
/usr/include/wx-2.8/wx/geometry.h:96: warning: redundant redeclaration of ‘wxPoint2DInt operator/(const wxPoint2DInt&, wxInt32)’ in same scope
/usr/include/wx-2.8/wx/geometry.h:95: warning: previous declaration of ‘wxPoint2DInt operator/(const wxPoint2DInt&, wxInt32)’
/usr/bin/ld: warning: libjpeg.so.8, needed by /usr/lib/libtiff.so.3, may conflict with libjpeg.so.62

comment:3 by Kieran P, 14 years ago

Milestone: Backlog

in reply to:  2 comment:4 by Philip Taylor, 13 years ago

(Sorry, forgot to reply earlier.)

Replying to dsims:

Basically everything is being built it's just pyrogenesis_dbg segment faults after the first two speed tests.

Hmm, segmentation faults don't sound good! Can you run it in gdb to find where it's failing? (Run "gdb ./pyrogenesis_dbg" then enter "r" to run, and hopefully it'll crash, then enter "bt full" to see the location.)

A list of the required header files would make it easier for me to find the packages I need to compile it, because packages change on a daily basis and mandriva does not utilize version packaging on every package

The BuildInstructions page has a list of the tools and libraries that are required. In general, any versions that aren't more than a few years old ought to work.

The thing is I cannot have both packages libJpeg62 and libjpeg version 8 installed at the same time, but I don't think that should matter.

Given the warning:

/usr/bin/ld: warning: libjpeg.so.8, needed by /usr/lib/libtiff.so.3, may conflict with libjpeg.so.62

it sounds like libtiff and the game are being compiled with inconsistent versions of libjpeg. I think you shouldn't need to have multiple versions installed - everything should use the same one. Maybe something is a bit broken in the Mandriva packages you're using, but I'm afraid I don't know any details that would help :-(

comment:5 by Philip Taylor, 13 years ago

Milestone: BacklogAlpha 4
Resolution: worksforme
Status: newclosed

Closing due to lack of activity; feel free to reopen if there's still a problem. (Warnings all seem to be either fixed or unfixable, and the segmentation fault looks like a system configuration error.)

Note: See TracTickets for help on using tickets.