Package: bochs
Version: 2.2.6-1
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.  I think the use of PARANOID in this case is bogus
since the extra qualification is against the C++ standard.  However,
if upstream disagrees, PARANOID should not be set when GCC is used.


> Automatic build of bochs_2.2.6-1 on bigsur by sbuild/mips 1.94
...
>  mips-linux-gnu-g++ -c -I.. -I./.. -I../instrument/stubs 
> -I./../instrument/stubs -Wall -g -O2 -D_FILE_OFFSET_BITS=64 -D_LARGE_FILES 
> -I/usr/include/SDL -D_REENTRANT -I/usr/lib/wx/include/gtk-2.4 
> -DGTK_NO_CHECK_CASTS -D__WXGTK__ -D_FILE_OFFSET_BITS=64 -D_LARGE_FILES 
> -I/usr/include/gtk-1.2 -I/usr/include/glib-1.2 -I/usr/lib/glib/include 
> harddrv.cc  -fPIC -DPIC -o .libs/harddrv.o
> ../gui/siminterface.h:894: warning: 'class bx_param_c' has virtual functions 
> but non-virtual destructor
> ../gui/siminterface.h:935: warning: 'class bx_param_num_c' has virtual 
> functions but non-virtual destructor
> ../gui/siminterface.h:997: warning: 'class bx_shadow_num_c' has virtual 
> functions but non-virtual destructor
> ../gui/siminterface.h:1054: warning: 'class bx_param_bool_c' has virtual 
> functions but non-virtual destructor
> ../gui/siminterface.h:1070: warning: 'class bx_shadow_bool_c' has virtual 
> functions but non-virtual destructor
> ../gui/siminterface.h:1085: warning: 'class bx_param_enum_c' has virtual 
> functions but non-virtual destructor
> ../gui/siminterface.h:1407: warning: 'class bx_simulator_interface_c' has 
> virtual functions but non-virtual destructor
> hdimage.h:106: warning: 'class device_image_t' has virtual functions but 
> non-virtual destructor
> hdimage.h:134: warning: 'class default_image_t' has virtual functions but 
> non-virtual destructor
> hdimage.h:164: warning: 'class concat_image_t' has virtual functions but 
> non-virtual destructor
> hdimage.h:279: error: extra qualification 'sparse_image_t::' on member 
> 'get_physical_offset'
> hdimage.h:284: error: extra qualification 'sparse_image_t::' on member 
> 'set_virtual_page'
> hdimage.h:210: warning: 'class sparse_image_t' has virtual functions but 
> non-virtual destructor
> hdimage.h:354: warning: 'class growing_image_t' has virtual functions but 
> non-virtual destructor
> hdimage.h:384: warning: 'class undoable_image_t' has virtual functions but 
> non-virtual destructor
> hdimage.h:417: warning: 'class volatile_image_t' has virtual functions but 
> non-virtual destructor
> hdimage.h:456: warning: 'class z_ro_image_t' has virtual functions but 
> non-virtual destructor
> hdimage.h:488: warning: 'class z_undoable_image_t' has virtual functions but 
> non-virtual destructor
> hdimage.h:520: warning: 'class z_volatile_image_t' has virtual functions but 
> non-virtual destructor
> vmware3.h:29: warning: 'class vmware3_image_t' has virtual functions but 
> non-virtual destructor
> make[2]: *** [harddrv.lo] Error 1
> make[2]: Leaving directory `/build/tbm/bochs-2.2.6/iodev'


--- ./iodev/hdimage.h~  2006-03-15 15:08:02.000000000 +0000
+++ ./iodev/hdimage.h   2006-03-15 15:10:18.000000000 +0000
@@ -272,16 +272,8 @@
  off_t total_size;
 
  void panic(const char * message);
- off_t
-#ifndef PARANOID
-       sparse_image_t::
-#endif
-                       get_physical_offset();
- void
-#ifndef PARANOID
-       sparse_image_t::
-#endif
-                       set_virtual_page(Bit32u new_virtual_page);
+ off_t get_physical_offset();
+ void set_virtual_page(Bit32u new_virtual_page);
  void read_header();
  ssize_t read_page_fragment(Bit32u read_virtual_page, Bit32u read_page_offset, 
size_t read_size, void * buf);
 

-- 
Martin Michlmayr
http://www.cyrius.com/


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to