aMule Forum

Please login or register.

Login with username, password and session length
Advanced search  

News:

We're back! (IN POG FORM)

Author Topic: Build error in Statistics.cpp  (Read 4241 times)

gnazio

  • Approved Newbie
  • *
  • Karma: 5
  • Offline Offline
  • Posts: 18
  • Live long and prosper
Build error in Statistics.cpp
« on: February 13, 2012, 12:10:08 AM »

Moving from 10708 to 10756, I received an error during the build. Looking at the changes in the git repository, I think that
 the problem has been introduced in 10754.
Quote
c++ -DHAVE_CONFIG_H -I. -I..  -DUSE_WX_EXTENSIONS   -I/usr/local/lib/wx/include/gtk2-unicode-release-2.8 -I/usr/local/include/wx-2.8 -D_FILE_OFFSET_BITS=64 -D_LARGE_FILES -D__WXGTK__ -DENABLE_IP2COUNTRY=1 -I./libs -Ilibs -I./include -I/usr/local/include -D__WEAK_CRYPTO__ -I/usr/local/include -W -Wall -Wshadow -Wundef -pthread -D_THREAD_SAFE  -O2 -pipe -fno-strict-aliasing -march=prescott -MT amule-Statistics.o -MD -MP -MF .deps/amule-Statistics.Tpo -c -o amule-Statistics.o `test -f 'Statistics.cpp' || echo './'`Statistics.cpp
Statistics.cpp: In static member function 'static void CStatistics::Load()':
Statistics.cpp:323: error: 'AddLogLineN' was not declared in this scope

Can you please review the attached patch?
Logged

Schuttwegraeumer

  • Sr. Member
  • ****
  • Karma: 2
  • Offline Offline
  • Posts: 421
Re: Build error in Statistics.cpp
« Reply #1 on: February 13, 2012, 03:06:14 AM »

Just built with R10757 (patched) und it works now.

Logged

x_x_kamikaze_x_x

  • Jr. Member
  • **
  • Karma: 18
  • Offline Offline
  • Posts: 83
Re: Build error in Statistics.cpp
« Reply #2 on: February 13, 2012, 11:34:59 AM »

Hi again!

Just built with R10757 (patched) und it works now.

Sorry Schuttwegraeumer, but at revision 10757, it's still crashing...

Code: [Select]

kami@maria2:~/Desktop/aMule SVN$ svn up
At revision 10757.

kami@maria2:~/Desktop/aMule SVN$ ./configure --enable-optimize --enable-cas --enable-wxcas --enable-alc --enable-alcc --enable-xas --enable-geoip --enable-mmap --enable-fileview --with-zlib --enable-ccache --enable-debug

.....

kami@maria2:~/Desktop/aMule SVN$ make

.....

/usr/bin/ccache g++ -DHAVE_CONFIG_H -I. -I..  -D__DEBUG__ -DUSE_WX_EXTENSIONS -I/usr/include/upnp   -DENABLE_UPNP=1 -pthread   -I/usr/lib/wx/include/gtk2-unicode-release-2.8 -I/usr/include/wx-2.8 -D_FILE_OFFSET_BITS=64 -D_LARGE_FILES -D__WXGTK__ -DENABLE_IP2COUNTRY=1 -I./libs -Ilibs -I./include -I/usr/include -D__WEAK_CRYPTO__  -g -W -Wall -Wshadow -Wundef -ggdb -fno-inline -fmessage-length=0 -O2 -pthread   -MT amule-Statistics.o -MD -MP -MF .deps/amule-Statistics.Tpo -c -o amule-Statistics.o `test -f 'Statistics.cpp' || echo './'`Statistics.cpp
Statistics.cpp: In static member function ‘static void CStatistics::Load()’:
Statistics.cpp:323:23: error: ‘AddLogLineN’ was not declared in this scope
make[3]: *** [amule-Statistics.o] Error 1
make[3]: Leaving directory `/home/kami/Desktop/aMule SVN/src'
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory `/home/kami/Desktop/aMule SVN/src'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/home/kami/Desktop/aMule SVN'
make: *** [all] Error 2

Thanks for your wonderful work! :)

Kami
Logged

Schuttwegraeumer

  • Sr. Member
  • ****
  • Karma: 2
  • Offline Offline
  • Posts: 421
Re: Build error in Statistics.cpp
« Reply #3 on: February 13, 2012, 06:54:39 PM »

I mean the crash during the compiling.
Logged

GonoszTopi

  • The current man in charge of most things.
  • Administrator
  • Hero Member
  • *****
  • Karma: 169
  • Offline Offline
  • Posts: 2685
Re: Build error in Statistics.cpp
« Reply #4 on: February 13, 2012, 07:22:28 PM »

Moving from 10708 to 10756, I received an error during the build. Looking at the changes in the git repository, I think that
 the problem has been introduced in 10754.
Quote
c++ -DHAVE_CONFIG_H -I. -I..  -DUSE_WX_EXTENSIONS   -I/usr/local/lib/wx/include/gtk2-unicode-release-2.8 -I/usr/local/include/wx-2.8 -D_FILE_OFFSET_BITS=64 -D_LARGE_FILES -D__WXGTK__ -DENABLE_IP2COUNTRY=1 -I./libs -Ilibs -I./include -I/usr/local/include -D__WEAK_CRYPTO__ -I/usr/local/include -W -Wall -Wshadow -Wundef -pthread -D_THREAD_SAFE  -O2 -pipe -fno-strict-aliasing -march=prescott -MT amule-Statistics.o -MD -MP -MF .deps/amule-Statistics.Tpo -c -o amule-Statistics.o `test -f 'Statistics.cpp' || echo './'`Statistics.cpp
Statistics.cpp: In static member function 'static void CStatistics::Load()':
Statistics.cpp:323: error: 'AddLogLineN' was not declared in this scope

Can you please review the attached patch?

You're right. Thanks for the patch, applied in rev. 10758.
Logged
concordia cum veritate

Stu Redman

  • Administrator
  • Hero Member
  • *****
  • Karma: 214
  • Offline Offline
  • Posts: 3739
  • Engines screaming
Re: Build error in Statistics.cpp
« Reply #5 on: February 13, 2012, 09:17:45 PM »

Another precompiled headers problem, in case you wonder how this slipped through. Sometimes stuff looks innocent enough so I don't bother booting Linux, and sometimes that's wrong. Thanks for patching, folks.
Logged
The image of mother goddess, lying dormant in the eyes of the dead, the sheaf of the corn is broken, end the harvest, throw the dead on the pyre -- Iron Maiden, Isle of Avalon