| 01:11.36 | *** join/#bzflag Patlabor221 (~JeffM@2605:e000:7d02:c700:560:6fe7:cceb:44b7) |
| 02:39.53 | *** join/#bzflag zlsa (~zlsa@static-205-157-147-198.ded.unwiredbb.net) |
| 11:15.06 | *** join/#bzflag I_Died_Once (~I_Died_On@unaffiliated/idiedonce/x-1828535) |
| 11:38.29 | *** join/#bzflag I_Died_Once (~I_Died_On@69.136.32.149) |
| 11:38.29 | *** join/#bzflag I_Died_Once (~I_Died_On@unaffiliated/idiedonce/x-1828535) |
| 13:55.29 | *** join/#bzflag RAThomas (~chatzilla@r74-195-236-7.stl1cmta01.stwrok.ok.dh.suddenlink.net) |
| 13:56.27 | RAThomas | ran into a guy on a map claiming to have "hacked mofo" |
| 13:56.57 | RAThomas | IP was 177.234.14.150 but that appears to be a proxy in Mexico |
| 13:57.28 | RAThomas | said he found a new bug and wasn't going to help anyone figure it out |
| 13:57.49 | blast007 | just some DoS style attack.. far from "hacked" |
| 13:58.16 | blast007 | the guy thinks that any kind of cheat or attack is a "hack" :) |
| 13:58.21 | RAThomas | lovely |
| 13:58.31 | RAThomas | sounds like a child |
| 13:58.48 | blast007 | certainly does... |
| 14:02.36 | *** join/#bzflag AresHandler (~AresHandl@pool-74-109-244-171.pitbpa.fios.verizon.net) |
| 14:13.09 | *** join/#bzflag zlsa (~zlsa@static-205-157-147-198.ded.unwiredbb.net) |
| 14:35.07 | RAThomas | he's observer on a rabbit map right now. Says he has a python script that exercises a newly found teleport bug that crashes clients |
| 14:35.31 | RAThomas | refuses to demonstrate, says it's too boring |
| 14:38.10 | RAThomas | says he's the DR guy (dom. rep.) |
| 14:39.58 | therealtbe | yeah he was on hix ffa bragging about it too very annoying |
| 14:42.41 | *** join/#bzflag Patlabor221 (~JeffM@2605:e000:7d02:c700:a10a:6634:36df:c38b) |
| 14:44.17 | blast007 | or it's the same teleport bug he found before and he's just exploiting it on servers that aren't running the latest code from git |
| 14:46.04 | alezakos | seems so |
| 14:46.35 | alezakos | he's supporting he couldn't exploit it on mofo |
| 14:50.36 | *** join/#bzflag zlsa (~zlsa@static-205-157-147-198.ded.unwiredbb.net) |
| 14:56.51 | RAThomas | blast007: seems like both. Says tele bug doesn't work on mofo, does elsewhere. Also says he found a new bug. Won't tell what. |
| 14:57.01 | RAThomas | offered to sell it for $10 US |
| 14:58.03 | Patlabor221 | send him a t-shirt |
| 14:59.09 | RAThomas | heh. Offered an Amazon gift card (not sure I'd actually go through with it, though) |
| 14:59.34 | RAThomas | while it seems like blackmail, sort of, I'm actually a fan of putting bounties on bugs |
| 14:59.50 | Patlabor221 | the projet can't legaly pay people |
| 14:59.59 | RAThomas | I'm not a dev |
| 15:00.00 | Patlabor221 | and it's not cool if they demand payment |
| 15:00.21 | Patlabor221 | I know |
| 15:00.34 | RAThomas | Speaking of which... anyone fix BZ client so it saves the user options setting for gamma? |
| 15:00.52 | Patlabor221 | on what OS? |
| 15:01.09 | RAThomas | Linux at least |
| 15:01.26 | Patlabor221 | it saves it, it probably just can't restore it |
| 15:01.35 | Patlabor221 | what video card/driver? |
| 15:01.37 | RAThomas | It saves gamma, just not what the user set |
| 15:02.02 | Patlabor221 | is it not in the bzdb? |
| 15:02.52 | RAThomas | I looked into this a few months ago. I believe my fix was to manually edit the config file with the gamma I wanted, then never save options again :( |
| 15:03.17 | Patlabor221 | did you report it as a real bug or just mention it in chat? |
| 15:03.31 | RAThomas | I know, I know. Just in chat |
| 15:03.51 | Patlabor221 | put it on github with what value in the config you edited? |
| 15:04.04 | Patlabor221 | but no, there isn't much development on the game now |
| 15:04.23 | RAThomas | yeah, I've got time today. I'll try to refresh my understanding of the problem |
| 15:04.41 | RAThomas | and report properly and/or post a patch. |
| 15:07.23 | RAThomas | and no, alezakos, I did not actually write a fuzzer that breaks BZ. Trying to gain his confidence but it's not working |
| 15:07.25 | zlsa | what is the official bzflag repository? |
| 15:07.39 | Patlabor221 | github |
| 15:07.54 | alezakos | zlsa: https://github.com/BZFlag-Dev/bzflag-import-3/ |
| 15:08.13 | Patlabor221 | and then select the branch you want |
| 15:08.17 | alezakos | v2_4_x branch for the current version, v2_6_x for the next breaking version |
| 15:08.29 | alezakos | ~bzgit |
| 15:08.29 | infobot | well, bzgit is git clone --branch v2_4_x https://github.com/BZFlag-Dev/bzflag-import-3.git bzflag |
| 15:09.07 | zlsa | thanks everyone |
| 16:32.06 | *** join/#bzflag Patlabor221 (~JeffM@107-209-61-105.lightspeed.irvnca.sbcglobal.net) |
| 16:59.26 | *** join/#bzflag zlsa (~zlsa@static-205-157-147-198.ded.unwiredbb.net) |
| 17:25.59 | *** join/#bzflag zlsa (~zlsa@static-205-157-147-198.ded.unwiredbb.net) |
| 17:26.11 | *** join/#bzflag Patlabor (~JeffM@76-218-80-21.lightspeed.irvnca.sbcglobal.net) |
| 18:20.30 | *** join/#bzflag thrakattak (~user@unaffiliated/sharkey) |
| 18:44.06 | *** join/#bzflag zlsa (~zlsa@static-205-157-147-198.ded.unwiredbb.net) |
| 19:15.39 | RAThomas | can anyone else confirm the gamma save problem on a Linux distro besides mine? |
| 19:16.00 | RAThomas | mine: Fedora 20. Video driver: nvidia proprietary |
| 19:16.01 | therealtbe | what is the setting name exactly |
| 19:16.08 | RAThomas | gamma |
| 19:16.27 | blast007 | and I'm assuming it's an SDL build? (the output of bzflag -version would show -SDL near the end of the version if so) |
| 19:16.28 | therealtbe | I do not see such a setting |
| 19:16.35 | RAThomas | yes, SDL |
| 19:16.53 | therealtbe | what's the setting under, display ? |
| 19:17.08 | RAThomas | "Brightness" |
| 19:17.24 | therealtbe | display, brightness |
| 19:17.28 | alezakos | I can't confirm on Ubuntu 14.04 AMD proprietary |
| 19:17.28 | RAThomas | it's saved as "gamma" in the "resources" (config file) |
| 19:18.33 | therealtbe | set gamma 1.515732 |
| 19:18.37 | alezakos | https://github.com/BZFlag-Dev/bzflag-import-3/blob/v2_4_x/src/bzflag/bzflag.cxx#L588 |
| 19:19.08 | therealtbe | set gamma 0.500001 |
| 19:19.09 | therealtbe | works for me |
| 19:19.24 | therealtbe | you have to Actually quit to save the config, are you verifying other settings change and the timestamp on the file updates? |
| 19:20.07 | RAThomas | problem statement: the "brightness" that the user settings in Options, Display, Brightness, does not get saved to the config file. Some other value is saved. |
| 19:20.34 | therealtbe | Q1: Are you actually going to Quit from the menu, that is the only way it will save settings |
| 19:20.44 | therealtbe | Q2: Do you have Save Settings "On Exit" |
| 19:20.53 | therealtbe | Q3: Have you verified other settings in the file are changing |
| 19:20.56 | RAThomas | manually editing the config file for (a) not auto save settings and (b) gamma set to my value, then gamma is correct (until a save is done) |
| 19:21.01 | therealtbe | Q4: Have you verified the timestamp on the file is changing |
| 19:21.48 | RAThomas | The file changes each time BZ is exited, unless I have manually edited the file to disable auto-save |
| 19:22.11 | therealtbe | it works fine for me, ubuntu 9.10, bzflag 2.4.2 |
| 19:22.47 | blast007 | therealtbe: and you're running an SDL build as well? |
| 19:22.54 | therealtbe | fuck if i know |
| 19:23.14 | therealtbe | $ ./src/bzflag/bzflag -version |
| 19:23.15 | therealtbe | BZFlag client 2.4.2.20130824-MAINT-linux-gnu-SDL (protocol 0221) http://BZFlag.org/ |
| 19:23.15 | therealtbe | Copyright (c) 1993-2012 Tim Riker |
| 19:23.30 | Patlabor221 | therealtbe, language please |
| 19:23.48 | RAThomas | yep, SDL. same as here: |
| 19:23.51 | RAThomas | BZFlag client 2.4.2.20130803-MAINT-linux-gnu-SDL (protocol 0221) http://BZFlag.org/ |
| 19:23.52 | therealtbe | open client, go to Display, Brightness, turn to minimum, escape to main menu, Quit, restart client, is it real dark? |
| 19:24.00 | therealtbe | very simple test ^ |
| 19:25.00 | RAThomas | Heh. My problem condition is the opposite: start bz, screen is very dark. adjust "brightness", save (or just Quit). Restart bz, screen is very dark. |
| 19:25.56 | RAThomas | I've verified that BZ would like to save gamma = 1.018 on my machine every time. 1.5 or 1.6 is much more appropriate.... |
| 19:26.16 | RAThomas | in game settings changes always result in gamma = 1.018 being saved |
| 19:26.31 | Patlabor221 | it's possible you arn't reporting the correct gama back from the driver, so you get wierd stuff saved |
| 19:26.59 | Patlabor221 | all the values in the config are saved via the same system |
| 19:27.05 | RAThomas | OK, so I got the source and added some debug, but a straight ./configure; make yields a binary that can't find fonts, etc... |
| 19:27.08 | Patlabor221 | so if other data is saved, them it will save gama |
| 19:27.12 | blast007 | therealtbe: please watch the language here |
| 19:27.19 | RAThomas | so I got the build.spec from Fedora and am working on getting that to go |
| 19:27.24 | blast007 | oh, was already mentioned, sorry |
| 19:27.30 | Patlabor221 | more then likely the value is being changed before save |
| 19:28.24 | RAThomas | Patlabor221: nvidia-settings --version |
| 19:28.26 | RAThomas | nvidia-settings: version 331.89 |
| 19:28.39 | Patlabor221 | that doesn't tell us anything |
| 19:28.49 | Patlabor221 | I do not have a repository of all nvidia drivers in my head |
| 19:28.54 | RAThomas | you mentioned driver. Thought the rev would be apropos |
| 19:29.05 | RAThomas | it's pretty much the latest |
| 19:29.12 | Patlabor221 | then it may have a bug |
| 19:29.34 | Patlabor221 | have you tried building from github and debugging the gamma value before it's writen to the config file? |
| 19:29.46 | blast007 | could be an SDL bug too |
| 19:29.54 | Patlabor221 | yeah one or the other |
| 19:30.03 | Patlabor221 | debugging will show you where the number goes bad |
| 19:30.26 | blast007 | wonder if our native linux stuff still builds/works |
| 19:30.38 | RAThomas | Patlabor221: yes. however I can't get the github derived bin to find paths, etc in Fedora. That's why I'm firing up the Fedora distro build.spec for bz |
| 19:30.39 | Patlabor221 | maybe |
| 19:31.01 | Patlabor221 | run it from the bin dir, and use -directory to give it paths |
| 19:31.08 | Patlabor221 | you don't have to install it to run it |
| 19:32.14 | RAThomas | WTH. I ran it earlier and it complained of not finding a font path. Now it seems happy to run... Sheesh |
| 19:32.16 | blast007 | actually you should just be able to run src/bzflag/bzflag without any path |
| 19:32.26 | blast007 | data path* |
| 19:32.34 | Patlabor221 | yeah it knows how to look around itself for data |
| 19:32.49 | blast007 | if you have a system copy installed it might actually be looking there instead now |
| 19:33.16 | Patlabor221 | the data dir hasn't changed from 2.4.2 to branch, so ether should be fine |
| 19:33.29 | RAThomas | OK, I got some debug output, but will have to refine it. I'll post the output I have in a moment |
| 19:33.44 | Patlabor221 | goes to eat cows |
| 19:34.28 | RAThomas | http://pastebin.com/E2Jvw2tc |
| 19:34.58 | RAThomas | not much to glean from that except that everywhere I examined the gamma value, it was the seemedly fixed 1.018 |
| 19:35.35 | RAThomas | blast007: yes, that was the difference in running it |
| 19:36.02 | RAThomas | I'd copied the new bin to /usr/bin which is where Fedora puts it |
| 19:36.30 | RAThomas | or not. who knows. |
| 19:37.08 | blast007 | I would think it would be in /usr/local/bin or /usr/local/games |
| 19:37.39 | RAThomas | Well, this *is* Fedora after all ;) |
| 19:38.21 | RAThomas | [rthomas@localhost bzflag]$ which bzflag |
| 19:38.22 | RAThomas | /usr/bin/bzflag |
| 19:54.37 | *** join/#bzflag spldart (~spldart@bzflag/contributor/spldart) |
| 19:54.37 | *** mode/#bzflag [+v spldart] by ChanServ |
| 20:28.43 | RAThomas | http://pastebin.com/uM1esgiV |
| 20:29.12 | RAThomas | by line 21 in the debug output, I'm increasing the brightness in the options->display menu |
| 20:29.52 | RAThomas | the problem appears to be in SDL |
| 20:30.09 | RAThomas | gonna track it down further... |
| 20:35.30 | RAThomas | Yeah, SDLWindow::getGamma always calls CalcGamma() and I always get 1.018 as a result. |
| 20:38.26 | blast007 | SDLWindow::getGamma is one of our functions |
| 20:38.49 | blast007 | so, debug that function. it's in src/platform/SDLWindow.cxx |
| 20:42.41 | RAThomas | yeah, already have debug in there |
| 20:44.11 | Patlabor221 | IIRC, we compute a gamma, tell SDL to set it, then read back what it actualy did |
| 20:44.27 | Patlabor221 | makes us save what the card can support, not what we think it can support |
| 20:45.17 | RAThomas | I can try that explicitly, but that's pretty much what my debug output is showing.... from menu I "setGamma()" (increasing values in debug output)... |
| 20:45.37 | RAThomas | then once it's time to end (return from startPlaying()) |
| 20:45.49 | RAThomas | calls getGamma and gets the 1.018 value. |
| 20:46.15 | RAThomas | anyway, I'll add and explicit set/getGamma for debug purposes |
| 20:53.45 | therealtbe | nvidia-settings: version 295.53 |
| 20:53.46 | therealtbe | <PROTECTED> |
| 20:54.21 | Patlabor221 | you should be able to step in using the debugger |
| 20:54.30 | Patlabor221 | debugger your friend :) |
| 20:54.48 | therealtbe | pets old school system... just keep workin |
| 20:55.39 | Patlabor221 | DirectX always returns gamma for me just fine :) |
| 21:00.37 | therealtbe | this calculate gamma ramp is the same as the latest libsdl code |
| 21:00.52 | RAThomas | sometimes it's difficult to know which to to start with. "man gdb" or try tossing in a few printfs ? Though I used at least 3 different types of in-circuit debuggers in 1986 alone... |
| 21:01.16 | RAThomas | I have not yet had enough need for gdb to become intimately familiar |
| 21:01.24 | Patlabor221 | it's too bad GDB isn't more friendly |
| 21:01.40 | Patlabor221 | people realy need to be able to step into and over code |
| 21:01.47 | Patlabor221 | F5 FTW |
| 21:03.15 | therealtbe | break SDLWindow::getGamma() |
| 21:03.17 | therealtbe | step step step |
| 21:03.33 | therealtbe | most likely the return from SDL_GetGammaRamp doesn't know what kinda video device you have |
| 21:04.05 | RAThomas | I agree |
| 21:04.08 | RAThomas | http://pastebin.com/GL7idRw6 |
| 21:04.19 | Patlabor221 | like I started with, driver ;) |
| 21:04.39 | Patlabor221 | what chipset is it? |
| 21:04.56 | therealtbe | why not just umm call SDL_CalculateGammaFromRamp ? |
| 21:05.35 | therealtbe | you would have to pass it a value and then return that but whatev |
| 21:06.59 | therealtbe | altho this function does look identical to libsdl so... |
| 21:07.03 | therealtbe | why exactly 1.018 |
| 21:07.26 | RAThomas | $ lspci |
| 21:07.27 | RAThomas | [...] |
| 21:07.29 | RAThomas | 01:00.0 VGA compatible controller: NVIDIA Corporation G98 [GeForce 8400 GS Rev. 2] (rev a1) |
| 21:07.30 | RAThomas | [...] |
| 21:07.56 | blast007 | therealtbe: do you mean SDL_CalculateGammaRamp ? |
| 21:08.07 | therealtbe | no I mean what I said not what u said |
| 21:08.31 | therealtbe | libsdl contains a function for CalculateGammaFromRamp, why not use it, other than it would need a wrapper to pass in a float and return it |
| 21:08.44 | therealtbe | but the code appears identical, so I doubt that is the prob |
| 21:08.56 | blast007 | not seeing that in the SDL docs |
| 21:09.02 | Patlabor221 | are you looking at SDL 2 or sdl 1.x? |
| 21:09.09 | blast007 | but I do see SDL_CalculateGammaRamp |
| 21:09.25 | blast007 | oh, guess their wiki only has the 2.0 stuff now |
| 21:09.27 | RAThomas | in the bz source, "SDLDisplay.cxx": |
| 21:09.53 | RAThomas | <PROTECTED> |
| 21:09.54 | RAThomas | static float CalculateGammaFromRamp(Uint16 ramp[256]) { |
| 21:10.16 | therealtbe | yeah it's basically identical |
| 21:11.14 | RAThomas | OK, so maybe driver. Maybe Fedora, somehow. This may have occured on Fedora 18 too, but not on Fedora 14 |
| 21:11.27 | therealtbe | yes but one more line of debugging will tell |
| 21:11.32 | therealtbe | the question is why exatly 1.018 |
| 21:11.37 | therealtbe | can you print out sum & count |
| 21:11.41 | Patlabor221 | yeah that's why you need a real debugger |
| 21:11.43 | RAThomas | Fedora's package build.spec for bz is different from the one from github |
| 21:11.44 | Patlabor221 | step in |
| 21:12.09 | therealtbe | print sum |
| 21:12.10 | therealtbe | print count |
| 21:13.10 | therealtbe | count should be 255.. this means the sum has to be.... 250... |
| 21:13.53 | therealtbe | then print out all the ramp[i] values and they are probly stuck at some constant |
| 21:14.07 | therealtbe | which will lead you to SDL_GetGammaRamp is returning bad data |
| 21:14.56 | therealtbe | that whole function seems to hinge on "SDL_VideoDevice *video = current_video;" |
| 21:15.28 | therealtbe | when you startup bzflag does it say in purple OpenGL Driver: GeForce something something |
| 21:17.18 | RAThomas | on the OpenGL... yep |
| 21:18.51 | therealtbe | oh well, hardcode it to the value you like :-D |
| 21:19.36 | RAThomas | :) That's been my solution for the last few months. |
| 21:21.00 | RAThomas | anyway, I'll keep nibbling away at the problem as I have time. I'll try different distros and see what changes |
| 21:21.27 | Patlabor221 | at least you know the problem is with bzflag saving, it's further up the chain |
| 21:23.49 | therealtbe | +/- n't |
| 21:40.18 | *** join/#bzflag Slyther (49a4e6f8@gateway/web/freenode/ip.73.164.230.248) |
| 21:40.23 | Slyther | kie kee |
| 23:32.28 | *** join/#bzflag Foooey (5c16a0a8@gateway/web/freenode/ip.92.22.160.168) |
| 23:34.32 | Foooey | the game |
| 23:35.05 | *** join/#bzflag Patlabor221 (~JeffM@2605:e000:7d02:c700:8bf:c346:b2f8:b0db) |
| 23:36.41 | Foooey | Jolie |
| 23:38.43 | Foooey | howdy |
| 23:45.06 | blast007 | hello |
| 23:55.34 | Foooey | hello |
| 23:57.20 | kierra | Dooley! |
| 23:57.26 | kierra | Oops. |
| 23:57.34 | kierra | Foooey! |
| 23:59.19 | Foooey | kierraaaa |
| 23:59.31 | Foooey | how are you |
| 23:59.41 | kierra | O m good :) |