darklegion / tremulous

Other
111 stars 41 forks source link

Changes from /dev/humancontroller #9

Closed jkent closed 9 years ago

jkent commented 9 years ago

I'm not sure if these changes are wanted; the future of darklegion/tremulous is unclear. Perhaps a gpp-compat branch should be created? Feedback is appreciated.

IRC log:

<DevHC> p0 fixes a major usability issue: r_width and r_height get
        practically ignored
<DevHC> the rest r more or less [resolve a few gpp] compatibility issues
        with the latest build
<DevHC> p1 is just in case someone mixes server/client binaries and QVMs
        of different versions -- GPP1 vs GitHub-latest
<DevHC> p2, at least in my testing, is required to get a server listing
        working
<DevHC> u'd think that the QVM should say "globalservers %d 71 full 
        empty" (instead of 70 there); here's how this works: old GPP 
        binaries use the 70 (correct, the GPP release uses protocol 70), 
        while the (newly-patched) GitHub-latest builds ignore the number 
        and use 71 (also correct)
<DevHC> p3: to try to avoid loading incompatible QVMs, and to stop 
        fooling the stats aggregators
<DevHC> p4: meh...
jkent commented 9 years ago

poke

timangus commented 9 years ago

I'll put a05281f on master, but I think the others belong on a gpp branch, as you suggest. Could you split into two pull requests? I'm not really sure what the future of darklegion/tremulous is either, FWIW. But it isn't much work to keep it from rotting, so I might as well do so.

jkent commented 9 years ago

Sure thing.