Closed Bug 241849 Opened 21 years ago Closed 21 years ago

compile error on linux platforms on tbird .6 branch

Categories

(Thunderbird :: Build Config, defect)

x86
Linux
defect
Not set
blocker

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: fredbezies, Assigned: mscott)

Details

User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8a) Gecko/20040427 Firefox/0.8.0+ Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8a) Gecko/20040427 Firefox/0.8.0+ Simple to reproduce. See details. Reproducible: Always Steps to Reproduce: 1.Get your source up to date 2.Launch build process and wait Actual Results: Crash in nsMsgDBView.cpp. Expected Results: Continues to build. Here is the log related to crash : Code: "nsMsgDBView.cpp c++ -o nsMsgDBView.o -c -DMOZ_LDAP_XPCOM -DOSTYPE=\"Linux2.6\" -DOSARCH=\"Linux\" -DHAVE_MOVEMAIL -I../../../dist/include/xpcom -I../../../dist/include/xpcom_obsolete -I../../../dist/include/string -I../../../dist/include/necko -I../../../dist/include/dom -I../../../dist/include/appshell -I../../../dist/include/uconv -I../../../dist/include/intl -I../../../dist/include/htmlparser -I../../../dist/include/widget -I../../../dist/include/docshell -I../../../dist/include/rdf -I../../../dist/include/gfx -I../../../dist/include/layout -I../../../dist/include/content -I../../../dist/include/mailnews -I../../../dist/include/locale -I../../../dist/include/unicharutil -I../../../dist/include/msgbaseutil -I../../../dist/include/webshell -I../../../dist/include/txmgr -I../../../dist/include/msgcompose -I../../../dist/include/msgdb -I../../../dist/include/uriloader -I../../../dist/include/pref -I../../../dist/include/msglocal -I../../../dist/include/msgimap -I../../../dist/include/mork -I../../../dist/include/msgnews -I../../../dist/include/addrbook -I../../../dist/include/prefmigr -I../../../dist/include/mime -I../../../dist/include/mimetype -I../../../dist/include/windowwatcher -I../../../dist/include/webbrwsr -I../../../dist/include/exthandler -I../../../dist/include/mozldap -I../../../dist/include/msgbase -I../../../dist/include -I/home/fred/logs/thun/mozilla/dist/include/nspr -I/usr/X11R6/include -fPIC -I/usr/X11R6/include -fno-rtti -fno-exceptions -Wall -Wconversion -Wpointer-arith -Wcast-align -Woverloaded-virtual -Wsynth -Wno-ctor-dtor-privacy -Wno-non-virtual-dtor -Wno-long-long -pedantic -fshort-wchar -pthread -pipe -DNDEBUG -DTRIMMED -Os -freorder-blocks -gstabs+ -I/usr/X11R6/include -DMOZILLA_CLIENT -include ../../../mozilla-config.h -Wp,-MD,.deps/nsMsgDBView.pp nsMsgDBView.cpp In file included from nsMsgDBView.cpp:53: ../../../dist/include/msgimap/nsImapCore.h:140: error: comma at end of enumerator list nsMsgDBView.cpp: In member function `virtual nsresult nsMsgDBView::Sort(int, int)': nsMsgDBView.cpp:3406: warning: invalid access to non-static data member `IdKey::key' of NULL object nsMsgDBView.cpp:3406: warning: (perhaps the `offsetof' macro was used incorrectly) nsMsgDBView.cpp: In member function `virtual nsresult nsMsgDBView::GetMsgToSelectAfterDelete(nsMsgViewIndex*)': nsMsgDBView.cpp:5400: warning: comparison between signed and unsigned integer expressions gmake[5]: *** [nsMsgDBView.o] Error 1 gmake[5]: Leaving directory `/home/fred/logs/thun/mozilla/mailnews/base/src' gmake[4]: *** [libs] Error 2 gmake[4]: Leaving directory `/home/fred/logs/thun/mozilla/mailnews/base' gmake[3]: *** [libs] Error 2 gmake[3]: Leaving directory `/home/fred/logs/thun/mozilla/mailnews' gmake[2]: *** [tier_97] Error 2 gmake[2]: Leaving directory `/home/fred/logs/thun/mozilla' make[1]: *** [default] Error 2 make[1]: Leaving directory `/home/fred/logs/thun/mozilla' make: *** [build] Error 2" I had noticed that before gcc warns, nsImapCore.h is listed, and line 141 pointed. When I look at last modification of this file, it was modified by bugfix for bug 200436. So it looks like this patch is guilty here. Here is the .mozconfig I used : Code: ". $topsrcdir/mail/config/mozconfig ac_add_options --enable-optimize="-Os -freorder-blocks -gstabs+' ac_add_options --disable-debug ac_add_options --enable-strip ac_add_options --enable-xft ac_add_options --enable-default-toolkit=gtk2 "
Christian Biesinger just add a fix. Closing bug. Sorry for not having noticed this before :[
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → INVALID
Sorry, I have to reopen it, I read a little to fast comment #88 on bug 200436 :]
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
Seems like this bug can be easily fixed. Just follow comment #88 and it works.
it's really a compile error, not a crash (the crash is because you couldn't recompile, I assume). I'll check in the fix, sorry for the inconvenience.
Summary: Crash in nsMsgDBView.cpp - side effect of bug 200436 ? → compile error on linux platforms on tbird .6 branch
Status: REOPENED → RESOLVED
Closed: 21 years ago21 years ago
Resolution: --- → FIXED
I mean of course crash while building thunderbird. Don't be sorry for the inconvenience, my "job" as a tester is to find bugs, even in build process. Thanks for your great work !
I think this is why there is no automated OS X build this morning either. Just an FYI in case folks are wondering where it is.
You need to log in before you can comment on or make changes to this bug.