Opened 18 years ago

Closed 18 years ago

Last modified 2 years ago

#7985 closed patch (outdated)

Patch for sharp zaurus.

Reported by: (none) Owned by:
Priority: normal Component: Ports
Keywords: Cc:
Game:

Description

a patch to allow scummvm to work on the sharp zaurus pda.

Thorsten Scheuermann did most of the initial work and released his modified source on his web page. I wrapped his changes around ifdef QTOPIA's so it wont affect other compile targets. I also made the input more zaurus friendly, I remapped the following buttons:

Menu key(f9) ---> f5 (scumm menu) button in the center of the arrow keys --> tab (default action) keyboard tab --> space (pause) Fn+backspace --> quit

summary per file:

main.cpp :

dont undef main for qtopia. someone fix my or error.

scummsys.h

define scumm_need_alignment for qtopia

sdl.cpp

SDL for the zaurus has no cd support compile in so all references to SDL_CD* are wrapped in ifndef QTOPIA

input remapped for qtopia. someone check this out to see if it can be less ugly.

####

aside from the change near the top of main.cpp this shouldn't affect other compile targets.

Ticket imported from: #584678. Ticket imported from: patches/90.

Attachments (1)

scumm-zaurus-0.2.0.patch (7.2 KB ) - added by (none) 18 years ago.
the patch!, applies to 0.2.0 release

Download all attachments as: .zip

Change History (6)

by (none), 18 years ago

Attachment: scumm-zaurus-0.2.0.patch added

the patch!, applies to 0.2.0 release

comment:1 by (none), 18 years ago

Logged In: NO

btw, i can be emailed at "dewt at kc dot rr dot com" i just didn't want to register with sourceforge

comment:2 by fingolfin, 18 years ago

There are a lot of things that need to be changed for this to be acceptable.

1) Make the patch against the latest CVS of ScummVM, not the 0.2.0 release

2) Do not modify the main Makefile. Rather create a Makefile.zaurus (or something like that) similar to Makefile.irix and Makefile.mingw

3) Why do you use QTOPIA and not ZAURUS? This patch is for the Zaurus after all...

4) What is the reason for the weird changes to config-file.cpp - either comment them, or find a better fix for whatever you are trying to do.

5) +//#elif !defined(QTOPIA) || !defined(__MORPHOS__) +// someone please explain to me why the above doesn't work

Simple, because you should be using && and not ||

6) As for the CDROM changes, you mean they didn't even compile in the dummy ("Do Nothing") CD-ROM driver for SDL?

There are possibly more issues, but I guess that's enough for now to fix :-) Looking forward to your revised patch!

comment:3 by SF/mncl, 18 years ago

The reason for using QTOPIA instead of ZAURUS is that the could *should* work as is on qtopia/opie for the ipaq.

as for the weird stuff in config-file.cpp, that was done by Thorsten Scheuermann, i dont know his reasons for commenting them out, i just wrapped them up in ifndefs.

On the cdroms, Yes cdrom support is disablefd for most(or all, i dont know if it works or not) SDL lib packages for the zaurus

I'll work on moving this into the latest from cvs and the makefile thing

comment:4 by fingolfin, 18 years ago

Resolution: outdated
Status: newclosed

comment:5 by digitall, 2 years ago

Component: Ports
Note: See TracTickets for help on using tickets.