Opened 16 years ago

Closed 16 years ago

Last modified 7 months ago

#808 closed defect (fixed)

ALL: Compilation error on BeOS

Reported by: SF/kreeblah Owned by: fingolfin
Priority: normal Component: Port: BeOS
Keywords: Cc:


Compiling ScummVM with the current CVS tree provides the
following error in the Sky module:

c++ -Wp,-MMD,"sky/.deps/text.d2" -g -O -Wall -Wstrict-
prototypes -Wuninitialized -Wno-long-long -Wno-multichar -Wno-
unknown-pragmas -DUNIX -I. -Icommon `sdl-config --cflags` -c
sky/text.cpp -o sky/text.o
/boot/home/ScummVMCVS/0716250503/scummvm/sky/text.cpp: In
method `struct displayText_t SkyText::displayText(char *,
uint8 *, bool, short unsigned int, unsigned char)':
initializing non-const `uint8 *&' with `byte *' will use a
make: *** [sky/text.o] Error 1

Disabling that in the makefile gets it to options.cpp where it
quits with this error:

c++ -Wp,-MMD,"gui/.deps/options.d2" -g -O -Wall -Wstrict-
prototypes -Wuninitialized -Wno-long-long -Wno-multichar -Wno-
unknown-pragmas -DUNIX -DDISABLE_SKY -I. -Icommon `sdl-config
--cflags` -c gui/options.cpp -o gui/options.o
/boot/home/ScummVMCVS/0716250503/scummvm/gui/options.cpp: In
method `GlobalOptionsDialog::GlobalOptionsDialog(class NewGui
*, class GameDetector &)':
implicit declaration of function `int getcwd(...)'
make: *** [gui/options.o] Error 1

Ticket imported from: #743138. Ticket imported from: bugs/808.

Change History (9)

comment:1 Changed 16 years ago by fingolfin

The first is a bogus warning / bogus compiler behaviour - byte and
uint8 are identical type (unsigned char), there should be no need for
any conversion.

For the second, try adding an
#include <unistd.h>
to the file - does that help?

comment:2 Changed 16 years ago by fingolfin

Summary: Compilation error on BeOSALL: Compilation error on BeOS

comment:3 Changed 16 years ago by SF/kreeblah

Hmmm. I was wondering what the difference was between an 8-bit
object and a byte. :^P I don't know whether somebody changed that
since I submitted the bug report or not, but that file compiles fine

Including unistd.h to options.cpp does allow it to compile
options.cpp and doesn't throw any more errors. Thanks.

BTW, would it be possible for the configure script to disable the -
Wno-unused-parameter flag on BeOS (the compiler doesn't support it)
and to disable MAD support on BeOS by default (no BeOS-compatible
port available)?

comment:4 Changed 16 years ago by fingolfin

Status: newclosed

comment:5 Changed 16 years ago by fingolfin

I added the unistd.h to options.cpp. For the configure script changes: the
configure script is still very experimental and not used by default. If you
want to see any changes in it, I think those are put best into feature request
items for now :-)
Please tell me if there are any other problems compiling on BeOS. We'd also
be interested in again shipping official BeOS binaries, if you are interested in
providing them, tell us :-)

comment:6 Changed 16 years ago by fingolfin

Owner: set to fingolfin
Resolution: fixed

comment:7 Changed 16 years ago by SF/kreeblah

Sure, I could provide binaries. How would I submit them once
they've been compiled? For the time being, I uploaded a compiled
copy of 0.4.1 to

comment:8 Changed 7 months ago by digitall

Component: Ports

comment:9 Changed 7 months ago by digitall

Component: PortsPort: BeOS
Note: See TracTickets for help on using tickets.