Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature Request] Revive/rewrite QVM PowerPC #232

Open
runlevel5 opened this issue Aug 16, 2023 · 6 comments
Open

[Feature Request] Revive/rewrite QVM PowerPC #232

runlevel5 opened this issue Aug 16, 2023 · 6 comments

Comments

@runlevel5
Copy link

Firstly, I have to say that I love quake3e source port. To me it is one of the BEST Q3 source port. I've been playing ioq3 with friends within the PowerPC community for quite awhile (all thanks to the great support for PPC in the original source code of ioq3). Is there any chance that quake3e could restore the support ppc64/ppc64le? (specifically restoring the qcommon/vm_powerpc_* files)? Many thanks

@runlevel5 runlevel5 changed the title [Feature Request] Restore support for ppc64/ppc64le [Feature Request] Restore support for VM ppc64/ppc64le Aug 16, 2023
@ec-
Copy link
Owner

ec- commented Aug 16, 2023

I rewrote QVM several times so 'restore' is incorrect term there - thus, PPC QVM it must be re-written too. It is a time-consuming task so no guarantees from my side

@ghost
Copy link

ghost commented Aug 16, 2023

@ec- Just asking out of curiosity, doesn't it work to include the files and compile them?
I own no PPC, and I know this isn't the right place to ask, but maybe adding the files from ioq3 will help others (with a PPC and some coding skills) to do the time-consuming job for you.
Just some thoughts.

@runlevel5
Copy link
Author

@ec I rewrote QVM several times so 'restore' is incorrect term there - thus, PPC QVM it must be re-written too. It is a time-consuming task so no guarantees from my side

Yes I am aware of this. I am more than happy to offer you access to hardware to work on if you have capacity.

@KuehnhammerTobias ..but maybe adding the files from ioq3 will help others (with a PPC and some coding skills) to do the time-consuming job for you.

That's what I thought too but initial attempt did not compile, some functions are reported missing, for example currentVM. In theory we could restore this legacy QVM PPC but I need to dig in the git history to work out what's changed

@ghost
Copy link

ghost commented Aug 17, 2023

@runlevel5

That's what I thought too but initial attempt did not compile, some functions are reported missing, for example currentVM. In theory we could restore this legacy QVM PPC but I need to dig in the git history to work out what's changed

This is unfortune. As far as the git history is concerned, ppc_vm files were dropped on Apr 21, 2017 but I think you found this out already by yourself. Well, assuming the PPC vm work until that point, it would be the starting point for you to check the code changes.
I'm not sure if that will help you, sorry!

@runlevel5 runlevel5 changed the title [Feature Request] Restore support for VM ppc64/ppc64le [Feature Request] Revive/rewrite QVM PowerPC Aug 17, 2023
@runlevel5
Copy link
Author

@ec- So if I could backport the vm_powerpc.c from the original ioq3, would you accept such patches given into consideration it is not yet rewritten?

@ec-
Copy link
Owner

ec- commented Aug 17, 2023

@runlevel5 sure it is ok if you can adopt it to new QVM structure and run some mod without errors

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants