#gemrb@irc.freenode.net logs for 10 Feb 2012 (GMT)

Archive Today Yesterday Tomorrow
GemRB homepage

[00:48:32] <-- kettuz has left IRC (Quit: Leaving)
[01:06:22] <-- Yoshimo has left IRC (Ping timeout: 265 seconds)
[01:13:46] <brad_a> im severely annoyed that the engine ini file doesnt have the same name for every game :-/
[02:32:23] <CIA-41> GemRB: 03bradallred * r89bc001fbaab 10gemrb/gemrb/core/ (4 files in 2 dirs): Implement "Keyboard Scroll Speed" setting available in baldur.ini
[03:22:42] <-- brad_a has left IRC (Ping timeout: 248 seconds)
[03:38:54] --> brad_a has joined #gemrb
[04:41:46] <brad_a> tomprince: http://dl.dropbox.com/u/13866402/params.patch
[04:42:00] <brad_a> still testing but im not sure if that is really the best way to process args
[04:48:37] <brad_a> heh pay no mind to the fact that it doesnt work :-/
[04:58:27] <brad_a> i need some advice on how to shuffle things about such that the paths default to useable values without a config...
[04:59:02] <brad_a> is it really as simple as moving code from loadConfig into init?
[05:04:22] <-- edheldil has left IRC (Remote host closed the connection)
[05:05:44] <tomprince> Well, I don't have a direct answer to your question.
[05:06:41] <tomprince> I'd suggest moving LoadCongig and LoadINI to a new class Config, and passing argc/argv to it.
[05:07:12] <tomprince> I'd also move the passing of argc and argv to Interface::Init.
[05:10:25] <brad_a> the core is just begging for refactoring isn't it?
[05:11:10] <tomprince> Yes. Yes. Yes. YEs.
[05:12:31] <tomprince> Working on Config now.
[05:13:14] <brad_a> what are you doing with the config?
[05:13:27] <tomprince> Just refactoring.
[05:14:07] <brad_a> i was afraid i was going to have to do it myself :)
[05:22:59] <-- brad_a has left IRC (Quit: brad_a)
[05:26:29] --> brad_a has joined #gemrb
[05:36:33] <-- joneirik has left IRC (Remote host closed the connection)
[06:23:46] <-- brad_a has left IRC (Quit: brad_a)
[07:03:18] <-- fuzzie has left IRC (*.net *.split)
[07:03:34] <-- harijan_ has left IRC (*.net *.split)
[07:03:34] <-- nutron has left IRC (*.net *.split)
[07:03:34] <-- tomprince has left IRC (*.net *.split)
[07:03:34] <-- |Cable| has left IRC (*.net *.split)
[07:03:34] <-- gembot has left IRC (*.net *.split)
[07:03:34] <-- Kiranos has left IRC (*.net *.split)
[07:03:34] <-- basse has left IRC (*.net *.split)
[07:03:34] <-- CIA-41 has left IRC (*.net *.split)
[07:03:34] <-- xrogaan has left IRC (*.net *.split)
[07:03:34] <-- yoann_ has left IRC (*.net *.split)
[07:03:34] <-- cj_schnell has left IRC (*.net *.split)
[07:03:34] <-- kamui has left IRC (*.net *.split)
[07:03:34] <-- Gekz has left IRC (*.net *.split)
[07:03:34] <-- DrMcCoy has left IRC (*.net *.split)
[07:04:47] <-- Calchan has left IRC (Ping timeout: 245 seconds)
[07:06:27] <-- exultbot has left IRC (Ping timeout: 245 seconds)
[07:06:27] <-- exultbot has left IRC (signing off...)
[07:07:41] --> exultbot has joined #gemrb
[07:07:41] --- Topic for #gemrb is: GemRB 0.7.0 | http://gemrb.org | Be wary of your words for there are Modron sensors in this channel: http://log.usecode.org/gemrblog.php | Hey <CHARNAME>, we need some awesome screenshots! | import pdb; pdb.set_trace()
[07:07:41] --- Topic for #gemrb set by lynxlynxlynx!~quassel@sourcemage/warlock/lynxlynxlynx at Fri Dec 30 14:46:00 2011
[07:07:49] --> Shinkamui has joined #gemrb
[07:07:49] --> wjp has joined #gemrb
[07:08:37] --> kamui has joined #gemrb
[07:15:54] <-- yoann_ has left IRC (Ping timeout: 378 seconds)
[07:16:05] --> yoann_ has joined #gemrb
[07:17:14] <-- xrogaan has left IRC (Write error: Broken pipe)
[07:17:20] --> xrogaan has joined #gemrb
[07:17:20] <-- xrogaan has left IRC (Changing host)
[07:17:20] --> xrogaan has joined #gemrb
[07:40:03] <tomprince> WIP: https://gist.github.com/1787460
[07:47:59] --> Kiranos_ has joined #gemrb
[07:50:19] <-- Kiranos has left IRC (Ping timeout: 276 seconds)
[08:04:28] <-- xrogaan has left IRC (Ping timeout: 244 seconds)
[08:06:11] --> xrogaan has joined #gemrb
[08:06:11] <-- xrogaan has left IRC (Changing host)
[08:06:11] --> xrogaan has joined #gemrb
[08:36:39] --> edheldil has joined #gemrb
[08:36:42] --- ChanServ gives channel operator status to edheldil
[08:51:11] <-- nutron has left IRC (*.net *.split)
[08:52:49] --> basse_ has joined #gemrb
[08:54:05] --> CIA-127 has joined #gemrb
[08:54:39] --> nutron has joined #gemrb
[08:54:45] <-- basse has left IRC (Write error: Connection reset by peer)
[08:54:45] <-- CIA-41 has left IRC ()
[08:59:47] <edheldil> brad_a: if you read this ... 1) I would make GamePath option-less argument. 2) If you process options with args, check that there really is an arg. 3) What about using getopt?
[09:01:08] --> CIA-125 has joined #gemrb
[09:01:18] <edheldil> oh, and if you change config or cli, update the relevant documentation as well, e.g. the man page
[09:01:32] <edheldil> in the same commit :)
[09:02:15] <-- nutron has left IRC (Ping timeout: 252 seconds)
[09:02:15] <-- CIA-127 has left IRC (Ping timeout: 252 seconds)
[09:12:38] --> nutron has joined #gemrb
[09:12:42] --> kamui_ has joined #gemrb
[09:12:44] <-- kamui has left IRC (Ping timeout: 245 seconds)
[09:13:22] --> lynxlynxlynx has joined #gemrb
[09:13:22] <-- lynxlynxlynx has left IRC (Changing host)
[09:13:22] --> lynxlynxlynx has joined #gemrb
[09:13:22] --- ChanServ gives channel operator status to lynxlynxlynx
[09:17:19] --> Kiranos has joined #gemrb
[09:21:35] <-- Kiranos_ has left IRC (Ping timeout: 248 seconds)
[09:26:31] --> gembot_ has joined #gemrb
[09:30:46] <-- gembot has left IRC (Ping timeout: 276 seconds)
[09:45:08] <-- kamui_ has left IRC (Ping timeout: 240 seconds)
[09:45:44] --> kamui has joined #gemrb
[10:05:12] --> kamui_ has joined #gemrb
[10:10:34] <-- kamui has left IRC (Ping timeout: 245 seconds)
[11:08:53] --> kalimann has joined #gemrb
[11:17:24] <kalimann> my log says it couldn't open the Chitin.key, and that I should change the directory path in my gamepath config variable correctly... which variable does it mean? the path to my planescape torment game itself? cause im like 99% sure its correct
[11:18:01] <fuzzie> yes, chitin.key should be in the game dir
[11:27:49] <kalimann> and of course it is :s
[11:28:00] <kalimann> and my game paths are: GamePath=/mnt/sdcard/app-data/net.sourceforge.gemrb/Torment/ CD1=/mnt/sdcard/app-data/net.sourceforge.gemrb/Torment/CD1 CD2=/mnt/sdcard/app-data/net.sourceforge.gemrb/Torment/CD2/ CD3=/mnt/sdcard/app-data/net.sourceforge.gemrb/Torment/CD3/ CD4=/mnt/sdcard/app-data/net.sourceforge.gemrb/Torment/CD4/
[11:28:09] <kalimann> so i cant see whats wrong
[11:28:55] <fuzzie> paste your entire config into paste.debian.net or similar?
[11:29:56] <kalimann> here http://paste.debian.net/155806/
[11:31:53] <fuzzie> hm, looks ok
[11:33:32] <fuzzie> i am not expert on android version, but since /mnt/sdcard should be world-readable, and you haven't fiddled with any of the options which shouldn't be changed, that looks just fine
[11:34:12] <kalimann> grrrrr
[11:34:16] <kalimann> thx for the info tho
[11:34:21] <fuzzie> are you using a version from the last few days?
[11:34:27] <kalimann> think so
[11:34:29] <kalimann> 2 sec
[11:34:47] <kalimann> yea i am
[11:37:15] <fuzzie> hmm
[11:37:46] <kalimann> you know maybe whos an expert on the droid version?
[11:39:20] <fuzzie> usually the forum is a better place to ask
[11:39:37] <fuzzie> it is pretty simple code though, it just adds chitin.key to the end of your GamePath and tries to open it
[11:39:56] <kalimann> yeah thats why i dont get it
[11:40:06] <kalimann> the chitin.key is in there
[11:40:21] <kalimann> and it did exactly the same when i tried installing BG
[11:40:26] <fuzzie> which device is it?
[11:40:52] <kalimann> samsung google nexus ice cream sandwich
[11:41:11] <kalimann> its pretty much the newest, so that might be why
[11:48:02] <kalimann> how do i create an account @ g3?! >:(
[11:48:46] <fuzzie> it looks like /mnt/sdcard is still the fuse mount on the galaxy nexus so it should be fine. and forum confuses me. lynxlynxlynx?
[11:52:15] <lynxlynxlynx> nope
[11:52:30] <lynxlynxlynx> so you're using the weekly builds, not the one from market?
[11:52:51] <lynxlynxlynx> ask on the forum, al3xapps will answer you and knows best
[11:53:13] <lynxlynxlynx> the paths indeed look fine
[11:54:58] <kalimann> ohhh no its the one from the market
[11:55:21] <kalimann> yeah ill try on the forum but i cant figure out how to create a login so i can post
[12:00:05] <lynxlynxlynx> try the weekly builds first
[12:00:15] <lynxlynxlynx> bbl
[12:00:32] <kalimann> can i just make the weekly build override the one i already got?
[12:04:50] <-- lynxlynxlynx has left IRC (Ping timeout: 240 seconds)
[12:05:25] <kalimann> someone help me ;____;
[12:05:34] <kalimann> i already got the 0.7.0 version available
[12:05:43] <kalimann> how the hell do i post on the forum?
[12:16:51] --> SiENcE has joined #gemrb
[12:18:29] --> lynxlynxlynx has joined #gemrb
[12:18:29] <-- lynxlynxlynx has left IRC (Changing host)
[12:18:29] --> lynxlynxlynx has joined #gemrb
[12:18:29] --- ChanServ gives channel operator status to lynxlynxlynx
[12:20:20] <-- kalimann has left IRC (Quit: Page closed)
[12:20:49] --> kalimann has joined #gemrb
[12:20:54] <kalimann> lynxlynxlynx, can you help me out here?
[12:21:13] <kalimann> i think the version i dl'ed is the same as you mentioned, 0.7.0
[12:23:52] <lynxlynxlynx> eh? that's the market version
[12:24:40] <lynxlynxlynx> i see there was an update though, so it's not really 0.7.0
[12:24:47] <lynxlynxlynx> you can still try the weekly
[12:26:16] <kalimann> hmm maybe i didnt find the correct one
[12:26:22] <kalimann> can you link me to the weekly..?
[12:35:33] <kalimann> is it GemRB-nmake-vc10-v0.7.0-2-g8b1f3fa.zip?
[12:41:36] <lynxlynxlynx> err, aren't you on android?
[12:53:16] <kalimann> yeah
[12:53:20] <kalimann> im so confused..
[12:53:29] <kalimann> i downloaded for windows phone?
[12:59:02] <kalimann> i dunno if you can find it, but i cant find anything for droid thats over 0.7.0
[13:01:15] <edheldil> 0.7.0 is the last release
[13:01:21] <edheldil> latest
[13:01:44] <kalimann> yeah what i said, thanks
[13:01:55] <kalimann> so maybe you can help me post this on the forum now?
[13:14:57] <edheldil> kalimann: so .. /mnt/sdcard/app-data/net.sourceforge.gemrb/Torment//Chitin.key does exist? I.e. "ls /mnt/sdcard/app-data/net.sourceforge.gemrb/Torment//Chitin.key" lists the file?
[13:22:52] <kalimann> yes it does
[13:23:59] <tomprince> Any thoughts ib https://gist.github.com/1787460
[13:26:26] <fuzzie> tomprince: well I'd really prefer we not have it in main()
[13:27:13] <tomprince> Yes, I wasn't sure about that.
[13:27:40] <fuzzie> especially if it's a 'Config config' in Interface anyway..?
[13:28:36] <tomprince> Oh, yes. I had one and then swtiched to the other, and missed removing the other.
[13:28:38] <fuzzie> and I think I'm not really convinced that it's a good idea to be passing this kind of thing around
[13:29:07] <fuzzie> but Config.h doesn't seem to be in the diff so that is about all I can comment on
[13:30:12] <CIA-125> GemRB: 03avenger_teambg * r9aa710cab159 10gemrb/gemrb/override/iwd2/ (dsclass.2da dsrace.2da dstable.2da): detect secret door tables for iwd2
[13:31:55] <CIA-125> GemRB: 03avenger_teambg * r07e01e2dea83 10gemrb/gemrb/override/iwd2/dirty.spl: spell block for dirty fighting feat
[13:32:12] <tomprince> https://gist.github.com/1789653
[13:32:38] <tomprince> fuzzie: Well, I want to get rid of some of the global data.
[13:33:41] <tomprince> I don't want to be passing it around everywhere, just at startup, hopefully.
[13:33:56] <tomprince> Haven't figured out how to do that for everything yet, though.
[13:34:03] <fuzzie> but the config *is* global data, essentially
[13:34:38] <fuzzie> well, I guess it depends on what you intend to put in it.
[13:35:37] <CIA-125> GemRB: 03avenger_teambg * r5d6f07a4d43f 10gemrb/gemrb/core/Scriptable/ (Actor.cpp Door.cpp): more 3rd ed ability differences
[13:37:00] <fuzzie> but most of the configuration is going to be needed in far too many places, I would think.
[13:41:32] <CIA-125> GemRB: 03avenger_teambg * r8d0badec70a9 10gemrb/gemrb/core/Scriptable/Actor.cpp: implemented the dirty fighting feat
[13:43:26] <-- kalimann has left IRC (Quit: Page closed)
[13:43:43] <tomprince> Well, I've found a couple of place where we currently use config at an arbitrary point, but could just as easily grab it once at the begining.
[13:43:54] <tomprince> The config branch in my github.
[13:43:56] <fuzzie> but you don't intend to include any of the mutable config?
[13:45:03] <tomprince> I don't know yet. Not necesarily in the config object.
[13:45:19] <fuzzie> maybe it might be best to call it StartupConfig or similar, in that case
[13:45:22] <tomprince> Perhaps I'll pass it places that need it, and they can keep a reference.
[13:45:49] <edheldil> startup config and running config? Where have I heard it? :o)))
[13:45:51] <fuzzie> and a lot of the uses in the diff make me think that really there should be a 'add this to game path' function
[13:46:25] <fuzzie> but I don't see the advantage of passing around objects to be *stored*, over storing them in a global object.
[13:46:55] <edheldil> fuzzie: there used to be PathAppend()
[13:46:58] <fuzzie> g_engine.getConfig().GamePath or whatever.
[13:47:10] <fuzzie> edheldil: i mean, without the caller needing to know the game path.
[13:48:26] <fuzzie> (although I have no issue with e.g. passing the engine object to *everything* and getting rid of the global, I should maybe note)
[13:49:19] <tomprince> Well, I was thinking of moving in that direction.
[13:49:40] <tomprince> When I had a version of the config object that needed access to interface, I passed it in explicitly.
[13:49:56] <edheldil> btw: http://www.metzgercartoons.com/uploads/1/5/6/3/1563197/6630590_orig.jpg :-)
[13:50:25] <fuzzie> I spent some time today poking through the AGS source code, which is essentially a single .cpp file with a few hundred global structures, so I am feeling more charitable towards GemRB's code than usual today. :p
[13:51:50] <edheldil> probably did not want to bother with build systems ;-)
[13:52:37] <fuzzie> yes, it does come plastered with huge warnings about how it is awful code and should never be looked at
[13:52:51] <fuzzie> but there are *little* bits in other .cpp files (and headers..) :)
[14:01:36] <-- Gekz has left IRC (Remote host closed the connection)
[14:01:42] --> Gekz has joined #gemrb
[14:01:42] <-- Gekz has left IRC (Changing host)
[14:01:42] --> Gekz has joined #gemrb
[14:03:34] <-- Gekz has left IRC (Read error: Connection reset by peer)
[15:24:40] --> Yoshimo has joined #gemrb
[16:42:56] <-- SiENcE has left IRC (Quit: @all: cya)
[16:56:13] --> Baldurer has joined #gemrb
[17:03:06] <-- Baldurer has left IRC ()
[17:19:14] --> brad_a has joined #gemrb
[17:49:55] --> edheldil_ has joined #gemrb
[18:08:14] <-- edheldil_ has left IRC (Ping timeout: 272 seconds)
[19:25:54] --> SiENcE has joined #gemrb
[20:23:42] <tomprince> Does the game data use <GamePath>/sounds or is that just for pcs?
[20:24:03] <tomprince> i.e. can I remove it from search path?
[20:25:31] <fuzzie> i think probably it shouldn't be in search path
[20:25:40] --> brad_a_ has joined #gemrb
[20:27:38] <-- brad_a has left IRC (Ping timeout: 240 seconds)
[20:27:38] --- brad_a_ is now known as brad_a
[20:28:31] <tomprince> https://github.com/tomprince/gemrb/compare/master...config
[20:30:32] <fuzzie> looks like a clear improvement?
[20:30:38] <brad_a> i think so
[21:04:56] <CIA-125> GemRB: 03bradallred * r3d60c339ad35 10gemrb/gemrb/core/Interface.cpp: Interface: add ability to run GemRB without a config via new -g GamePath parameter.
[21:04:56] <CIA-125> GemRB: 03bradallred * r7fbcad7d3cac 10gemrb/apple/CocoaWrapper/ios/GEM_ConfController.m: GEM_ConfController: finally figured out how to unarchive to a different directory than CWD. iOS game data installation should be thread safe now that i got rid of the ugly hack of changing directory.
[21:04:56] <CIA-125> GemRB: 03bradallred * rb4e74660f766 10gemrb/gemrb/ (6 files in 2 dirs): Merge branch 'master' of ssh://gemrb.git.sourceforge.net/gitroot/gemrb/gemrb
[21:05:02] <brad_a> damn it
[21:05:09] <brad_a> forgot to undo that :(
[21:07:52] <CIA-125> GemRB: 03bradallred * rebe6d2ffe649 10gemrb/gemrb/core/Interface.cpp:
[21:07:52] <CIA-125> GemRB: Revert accidentially pushed commit 3d60c339ad352c32279d95fc5e10aede8ff180fa
[21:07:52] <CIA-125> GemRB: Revert "Interface: add ability to run GemRB without a config via new -g GamePath parameter."
[21:07:52] <CIA-125> GemRB: This reverts commit 3d60c339ad352c32279d95fc5e10aede8ff180fa.
[21:08:29] <brad_a> thought i had been working on a diffrent branch...
[21:15:55] <tomprince> hmmm... how to thread Config through to KEYImporter, without also threading it to DirectortImporter.
[21:22:55] <CIA-125> GemRB: 03avenger_teambg * rbb4e02e58912 10gemrb/apple/CocoaWrapper/ios/GEM_ConfController.m: Merge branch 'master' of ssh://gemrb.git.sourceforge.net/gitroot/gemrb/gemrb
[21:23:03] <CIA-125> GemRB: 03avenger_teambg * rc9e25ac7edac 10gemrb/gemrb/ (5 files in 3 dirs): implemented day blindness for drow/duergar
[21:26:15] --> Avenger has joined #gemrb
[21:26:43] <Avenger> Sounds are on the search path in the original engine, though we might survive if you remove it. No one exploits that feature, i guess.
[21:29:28] <-- Avenger has left IRC (Client Quit)
[22:03:19] <-- Yoshimo has left IRC (Quit: Yoshimo)
[23:03:49] <-- SiENcE has left IRC (Quit: cya)
[23:05:25] <tomprince> Update config patch: https://gist.github.com/1793835 <-- this is on top of https://github.com/tomprince/gemrb/compare/master...config
[23:06:03] <-- brad_a has left IRC (Ping timeout: 244 seconds)
[23:07:00] <tomprince> Just need to move var.var handling earlier, and the bif_path selection.
[23:07:35] <tomprince> For the latter, I'm pondering using a resource manager.
[23:08:54] <fuzzie> well, again, I don't see the value of passing a Config struct around
[23:09:27] <tomprince> Well, you did say you would be happy to pass core around everywhere.
[23:09:51] <tomprince> Certainly, at this point, with a single variable in it, it isn't that useful.
[23:10:11] <fuzzie> Yes, but passing individual bits in piecemeal fashion..?
[23:11:28] <tomprince> Well, do you think the github branch is a net win? All those cleanups were directly inspired by getting the patch to where it is.
[23:12:09] <tomprince> So, perhaps the benefit is more in gettin other code in better shape to accomadte this. Even if we don't actually use it. :)
[23:12:50] <tomprince> I'd actually like it if less things talke to core at all, and so just passing this bit of static config around, rather than the whole object seems to make sense.
[23:13:09] <tomprince> Also, seems like it might eventually be eaiser to unit test, if we don't need to pass everything an Interface.
[23:13:21] <fuzzie> um
[23:14:00] <fuzzie> i think, no, not entirely happy with branch any more
[23:14:10] <fuzzie> moving ReadAutonoteINI to plugin init time is not good
[23:14:49] <tomprince> No?
[23:15:28] <fuzzie> you can't tell the game version there
[23:15:48] <tomprince> My moved plugin init time later, so we do know.
[23:15:53] <fuzzie> no
[23:16:04] <fuzzie> unless you reordered plugin init?
[23:16:07] <tomprince> We need to do that anyway for GF_MAGICBIT in CREImporter
[23:16:11] <fuzzie> you don't know game version until GUIScript is initialised
[23:16:37] <fuzzie> so you can't load gemrb.ini until after plugin init, no?
[23:16:59] <fuzzie> or do we let plugins be used without their initializers having been run first? if so, that seems .. not good :)
[23:17:18] <tomprince> Well, a few are.
[23:17:35] --> brad_a has joined #gemrb
[23:17:42] <tomprince> But, yes.
[23:17:48] * tomprince goes to proctor.
[23:17:49] <fuzzie> otherwise it all looks great
[23:18:13] <fuzzie> and obviously if we're already doing it for GF_MAGICBIT then it's no worse you committing what you have..
[23:18:18] <fuzzie> certainly not NACKing anything.
[23:18:39] <fuzzie> (which includes Config stuff..)
[23:22:30] <-- lynxlynxlynx has left IRC (Remote host closed the connection)
[23:25:09] <brad_a> i'm hoping this new config interface i haven't looked at will allow me to simply map an NSUserDefaults context to the config instead of requiring a text file :)
[23:26:10] <brad_a> on another topic I think i really need to make an SDL 2.0 version of SDLVideo sooner rather than later
[23:27:08] <brad_a> should i do video -> SDLVideo -> SDL2Video/SDLLegacyVideo class structure?
[23:27:40] <fuzzie> um
[23:27:42] <brad_a> or even just video -> SDLVideo -> SDL2video
[23:27:45] <fuzzie> it depends how much they deviated in SDL 2.0?
[23:28:45] <brad_a> http://wiki.libsdl.org/moin.cgi/MigrationGuide
[23:28:49] <brad_a> if you havent read it
[23:29:31] <brad_a> i feel confidant i can do it
[23:29:42] <brad_a> i jsut would like to do it right the first time :)
[23:30:09] <brad_a> not like the TTF font debacle :)
[23:32:16] <brad_a> id rather just rewrite the touch code while making a 2.0 driver also
[23:32:36] <brad_a> part of the reason i havent done that yet
[23:43:04] <brad_a> im thinking there is enough overlap to do video -> SDLVideo -> SDL2Video
[23:43:47] <fuzzie> well I would like to move the non-opengl stuff into a superclass someday
[23:43:52] <fuzzie> so if you are thinking of design..
[23:45:02] <brad_a> well right now i mostly just want an SDL 2.0 compatible video driver :)
[23:45:14] <brad_a> but im willing to listen to what you have in mind