News:

Come Chat with us live! Learn how HERE!

Main Menu

How do you submit a patch?

Started by i3enedek, July 18, 2007, 09:04:12 PM

i3enedek

There is a line in vm_ppc_new.c that came about with Apples switch to PowerPC, it makes sure that data in the instruction and data cache is synced after they moved away from the unified cache in the 601, anyway...

// On Mac OS X, the following library routine clears the instruction cache for generated code
          MakeDataExecutable(vm->codeBase, vm->codeLength);

Spotted after I saw this in the Makefile;

   # Carbon is required on PPC only to make a call to MakeDataExecutable
   # in the PPC vm (should be a better non-Carbon way).
   #LDFLAGS += -framework Carbon

Well there is a better non-Carbon way;
msync( vm->codeBase, vm->codeLength, MS_INVALIDATE );

It compiles but im having trouble building the entire project since I updated to Quicktime 7.2.

ld: Undefined symbols:
_HIPointConvert referenced from QuickTime expected to be defined in Carbon
make[1]: *** [build/release-darwin-ppc/tremulous.ppc] Error 1
make: *** [build_release] Error 2

Probably to do with the bindings.
color=red]l[/color]angsam

tehOen

contact ioquake3 devs (zakk is mostly online)
#ioquake3 irc.freenode.net
edit:
also check the svn version it is updated (since tremulous is using ioquake3 engine you should check ioquake3 not tremulous)

kevlarman

or ioq3 bugzilla (bugzilla.icculus.org), in general client/server changes go to ioq3 bugzilla and vm changes to tremulous bugzilla, any changes to the client/server that can't also be included in ioq3 tend to have a much lower chance of making it into svn.
Quote from: Asvarox link=topic=8622.msg169333#msg169333Ok let's plan it out. Asva, you are nub, go sit on rets, I will build, you two go feed like hell, you go pwn their asses, and everyone else camp in the hallway, roger?
the dretch bites.
-----
|..d| #
|.@.-##
-----

i3enedek

Ok it works but something is asking for a Gestalt ID so thats defo a legacy Carbon function, so until I find that the Carbon framework is still needed.
color=red]l[/color]angsam


i3enedek

Ahh well, this is terribly ironic.
color=red]l[/color]angsam

tjw

Quote from: i3enedekAhh well, this is terribly ironic.

Not really.  I just opened the bug in response to your posting ;)

i3enedek

Quote from: tjw
Quote from: i3enedekAhh well, this is terribly ironic.

Not really.  I just opened the bug in response to your posting ;)

:P

(and it is, as I file bugs all day long on Adobe apps and berate the other Engineers for not filing the bugs on the corporate DB).
color=red]l[/color]angsam