[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#356098: FTBFS with G++ 4.1: extra qualification



Package: oonsoo
Version: 1.2-9
Severity: important
Tags: patch

Your package fails to build with G++ 4.1.  I'm filing this bug as
important for now, but when 4.1 will be the default compiler in
unstable (probably in a few weeks) I'll upgrade this to serious.

A patch is below.


> Automatic build of oonsoo_1.2-9 on bigsur by sbuild/mips 1.80
...
> ------ Building: UI
> make[2]: Entering directory `/build/tbm/oonsoo-1.2/UI'
> g++  -I../Tools -I../Commands -I../Sound -c -O -ansi Text.cxx
> Sprite.hxx:29: error: extra qualification 'Sprite::' on member 'Sprite'
> ../Sound/Sample.hxx:29: error: extra qualification 'Sample::' on member 'Sample'
> ../Sound/Sound.hxx:34: error: extra qualification 'Sound::' on member 'Sound'
> make[2]: *** [Text.o] Error 1
> make[2]: Leaving directory `/build/tbm/oonsoo-1.2/UI'
> make[1]: *** [subdirectories] Error 1
> make[1]: Leaving directory `/build/tbm/oonsoo-1.2'


--- ./Sound/Sample.hxx~	2006-03-09 18:01:02.000000000 +0000
+++ ./Sound/Sample.hxx	2006-03-09 18:01:31.000000000 +0000
@@ -26,7 +26,7 @@
  
   public:
     // Constructor
-    Sample::Sample(unsigned char* sampleData);
+    Sample(unsigned char* sampleData);
 
     // Destructor 
     virtual ~Sample();
--- ./Sound/Sound.hxx~	2006-03-09 18:01:16.000000000 +0000
+++ ./Sound/Sound.hxx	2006-03-09 18:01:24.000000000 +0000
@@ -31,7 +31,7 @@
 
   public:
     // Constructor
-    Sound::Sound(SampleCollection* sampleCollection);
+    Sound(SampleCollection* sampleCollection);
 
     // Destructor 
     virtual ~Sound();
--- ./UI/Sprite.hxx~	2006-03-09 18:01:49.000000000 +0000
+++ ./UI/Sprite.hxx	2006-03-09 18:01:53.000000000 +0000
@@ -26,7 +26,7 @@
  
   public:
     // Constructor
-    Sprite::Sprite(unsigned char* spriteData);
+    Sprite(unsigned char* spriteData);
 
     // Destructor 
     virtual ~Sprite();
 

And please hit upstream for making the files non-writeable...
-- 
Martin Michlmayr
http://www.cyrius.com/



Reply to: