aMule Forum

English => Compilation problems => Topic started by: IhmSelbst on March 10, 2006, 07:52:26 AM

Title: 20060310 doesn't compile on suse9.3 and SuSE10.0
Post by: IhmSelbst on March 10, 2006, 07:52:26 AM

its last words before dying off:
...
In file included from CFile.h:29,
                 from PartFile.h:34,
                 from SharedFilesCtrl.cpp:32:
SafeFile.h:204:2: warning: #warning DOCS
In file included from SharedFilesCtrl.cpp:32:
PartFile.h:317:10: warning: #warning Kry - Is currently unused and must be fixed
SharedFilesCtrl.cpp:150:1: warning: "/*" within comment
SharedFilesCtrl.cpp:147:1: unterminated comment
SharedFilesCtrl.cpp: In member function `void
   CSharedFilesCtrl::OnRightClick(wxListEvent&)':
SharedFilesCtrl.cpp:146: error: syntax error at end of input
make[3]: *** [amule-SharedFilesCtrl.o] Error 1
make[3]: *** Waiting for unfinished jobs....
In file included from CFile.h:29,
                 from PartFile.h:34,
                 from TransferWnd.cpp:38:
SafeFile.h:204:2: warning: #warning DOCS
In file included from TransferWnd.cpp:38:
PartFile.h:317:10: warning: #warning Kry - Is currently unused and must be fixed
make[3]: Leaving directory `/usr/src/packages/BUILD/amule-cvs/src'
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory `/usr/src/packages/BUILD/amule-cvs/src'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/usr/src/packages/BUILD/amule-cvs'
make: *** [all] Error 2
error: Bad exit status from /var/tmp/rpm-tmp.56299 (%build)


RPM build errors:
    Bad exit status from /var/tmp/rpm-tmp.56299 (%build)


cu

Title: Re: 20060310 doesn't compile on suse9.3 and SuSE10.0
Post by: OldFrog on March 10, 2006, 09:17:06 AM

Same error on Mandriva 2006.0

Quote

In file included from CFile.h:29,
                 from PartFile.h:34,
                 from SharedFilesCtrl.cpp:32:
SafeFile.h:204:2: warning: #warning DOCS
In file included from SharedFilesCtrl.cpp:32:
PartFile.h:317:3: warning: #warning Kry - Is currently unused and must be fixed
SharedFilesCtrl.cpp:150:1: warning: "/*" within comment
SharedFilesCtrl.cpp:147:1: error: unterminated comment
SharedFilesCtrl.cpp: In member function 'void CSharedFilesCtrl::OnRightClick(wxL
istEvent&)':
SharedFilesCtrl.cpp:146: error: expected `}' at end of input
SharedFilesCtrl.cpp:146: error: expected `}' at end of input
make[3]: *** [amulegui-SharedFilesCtrl.o] Erreur 1
make[3]: Leaving directory `/usr/src/RPM/BUILD/amule-cvs/src'
make[2]: *** [all-recursive] Erreur 1
make[2]: Leaving directory `/usr/src/RPM/BUILD/amule-cvs/src'
make[1]: *** [all-recursive] Erreur 1
make[1]: Leaving directory `/usr/src/RPM/BUILD/amule-cvs'
make: *** [all] Erreur 2
erreur: Mauvais status de sortie pour /var/tmp/rpm-tmp.12114 (%build)

Title: Re: 20060310 doesn't compile on suse9.3 and SuSE10.0
Post by: Radek on March 10, 2006, 10:32:28 AM

Hi!

There's a little typo in SharedFilesCtrl.cpp - to correct it, change line 150 from "/*" to "*/". After that it should compile...

Title: Re: 20060310 doesn't compile on suse9.3 and SuSE10.0
Post by: Kry on March 10, 2006, 06:55:27 PM

I swear I test compilation before commiting, and yet this one passed the compilation test.

*sigh*

I was in a hurry for 2.1.1, you know...

Title: Re: 20060310 doesn't compile on suse9.3 and SuSE10.0
Post by: OldFrog on March 10, 2006, 07:09:50 PM

Hey, don't worry, this was an easy one ;) I guess IhmSelbst and I are used not to modify code written by other people, but rather alert about it (Actually this is what I do at work).

Just to be sure you're not sleeping :P


And great  news, we are on the way for 2.1.1  :baby: :baby:

Title: Re: 20060310 doesn't compile on suse9.3 and SuSE10.0
Post by: IhmSelbst on March 10, 2006, 11:13:36 PM

Quote

Originally posted by OldFrog
...
 I guess IhmSelbst and I are used not to modify code written by other people, but rather alert about it
...

yep, of course, you're right, and, of course I (usually) also never touch the code, too. - Although, this time, I couldn't resist, because it was only a small, not really code senstitive typo, so I indeed replaced it to build my daily rpm...  :]


cu