#nuvie@irc.freenode.net logs for 17 Aug 2012 (GMT)

Archive Today Yesterday Tomorrow
Nuvie homepage


[00:20:30] --> Kirben has joined #nuvie
[00:24:25] --> Kirben_ has joined #nuvie
[00:25:14] <-- Kirben has left IRC (Ping timeout: 260 seconds)
[01:15:24] <-- Malignant_Manor has left IRC (Quit: ChatZilla 0.9.88.2 [Firefox 14.0.1/20120713134347])
[06:01:48] <CIA-75> nuvie: 03malignantmanor * r1524 10/nuvie/trunk/ (15 files in 4 dirs): (log message trimmed)
[06:01:48] <CIA-75> nuvie: *Fix converse gump portrait when using a vehicle or loading a game with a
[06:01:48] <CIA-75> nuvie: different Avatar portrait
[06:01:48] <CIA-75> nuvie: *Fix clicking to equip items in containers being blocked (1518 regression)
[06:01:48] <CIA-75> nuvie: *Allow the torch to be equipped and used when inside a container
[06:01:48] <CIA-75> nuvie: *Fix objects in containers not stacking [3550551]
[06:01:49] <CIA-75> nuvie: *Allow container gump and doll widget to receive unreadied items from other
[06:23:50] --> Yuv422 has joined #nuvie
[06:29:40] --> Malignant_Manor has joined #nuvie
[06:30:04] <Malignant_Manor> I went to use my balloon and it turned into a skiff.
[06:30:46] <Malignant_Manor> It was broken before you even added the player movement stuff so I guess you never tested it with the balloon.
[07:03:39] <Yuv422> I think it doesn't handle transition from one vehicle type to another very well.
[07:03:59] <Malignant_Manor> It's seems to be an old bug.
[07:04:19] <Malignant_Manor> Probably 1367
[07:05:57] <Malignant_Manor> I'm going to build 1366
[07:06:35] <Yuv422> ah k
[07:07:25] <Malignant_Manor> I didn't think it had been that long since I went on a balloon.
[07:07:39] <Malignant_Manor> Maybe there is some issue with saving on the map.
[07:09:06] <Malignant_Manor> I mean 1368 is when I think the problem probably was introduced.
[07:10:13] <Malignant_Manor> 1366 still has this issue.
[07:10:27] <Malignant_Manor> A binarry from 6-22 doesn't
[07:22:15] <Malignant_Manor> 1355 broke it.
[07:24:17] <Malignant_Manor> That's a save file with a balloon that turns into a skiff.
[07:24:33] <Yuv422> cool I'll look into it
[07:24:49] <Malignant_Manor> I'm off to bed in a bit.
[07:29:34] <Yuv422> righto
[07:29:34] <Yuv422> cya
[07:30:06] <Yuv422> you've cracked triple digits on your save counter
[07:30:10] <Yuv422> very impressive
[07:30:48] <Malignant_Manor> They weren't all from the run.
[07:32:07] <Malignant_Manor> I think the saves started at around 50.
[07:33:35] <Malignant_Manor> Next release might be at the end of the month or the beginning of the next.
[07:34:43] <Malignant_Manor> It should (hopefully) be the first release where Nuvie is worth playing over the original for a decent amount of people.
[07:36:51] * Yuv422 is looking forward to that
[07:37:42] <Yuv422> Do you mind if I add you to the authors file?
[07:38:01] <Yuv422> I'll just add your details as per your change log entries
[07:41:32] <Malignant_Manor> I think I recently updated it.
[07:43:09] <Yuv422> cool
[07:43:12] <Malignant_Manor> I added it 1514
[07:43:28] <Yuv422> :-)
[07:44:45] <Malignant_Manor> I have to really test the interface. I messed with a whole lot.
[07:45:40] <Yuv422> :)
[07:46:20] <Malignant_Manor> It's so easy to mess up and I already have a lot.
[07:46:37] <Yuv422> it is a fragile thing
[07:48:20] <CIA-75> nuvie: 03wjpalenstijn * r1525 10/nuvie/trunk/ (keybinding/Keys.cpp views/PartyView.cpp): Fix build
[07:49:36] <Malignant_Manor> wjp: You should have done that a minute or two ago.
[07:49:59] <wjp> I'm pretty sure I did do that a minute or two ago?
[07:50:19] <Malignant_Manor> Right after I talked about messing up.
[07:51:23] <Yuv422> I think I've found the issue with the vehicle actor
[07:51:24] <Malignant_Manor> I though Dominus was using a case sensitive directory.
[07:51:47] <Yuv422> it is to do with the base_obj_n
[07:51:55] <Yuv422> not being reset properly
[07:57:29] <Malignant_Manor> It still happens with a cheat created balloon on first entering.
[07:57:42] <Malignant_Manor> Bye, I off
[07:57:47] <Yuv422> cya
[07:57:51] <-- Malignant_Manor has left IRC (Quit: ChatZilla 0.9.88.2 [Firefox 14.0.1/20120713134347])
[10:17:52] <-- Yuv422 has left IRC (Ping timeout: 246 seconds)
[12:08:08] <-- Dominus has left IRC (Read error: Connection reset by peer)
[12:08:54] --> Dominus has joined #nuvie
[13:47:29] <-- Kirben_ has left IRC ()
[13:56:50] <-- SugarCube has left IRC (*.net *.split)
[13:57:24] --> SugarCube has joined #nuvie
[13:57:26] <-- Dominus has left IRC (Read error: Connection reset by peer)
[14:01:46] --> Dominus has joined #nuvie
[16:28:04] <-- Dominus has left IRC (Quit: Leaving.)
[16:28:15] --> Dominus has joined #nuvie
[18:58:31] <-- Dominus has left IRC (Read error: Connection reset by peer)
[18:59:17] --> Dominus has joined #nuvie
[22:41:21] --> Malignant_Manor has joined #nuvie
[22:41:23] --> Yuv422 has joined #nuvie
[22:47:11] <Malignant_Manor> Do you know why MsgScroll.cpp line 433 is there for?
[22:47:13] <Malignant_Manor> if(msg_line->total_length == scroll_width) // add another line for cursor.
[22:47:37] <Malignant_Manor> It doesn't seem needed and adds an unwanted newline.
[22:49:11] <Yuv422> I think that was for problems with the line being exactly the width of the scroll
[22:49:20] <Yuv422> then you need a new line for the cursor
[22:49:40] <Yuv422> otherwise it rendered off the side of the scroll
[22:50:34] <Malignant_Manor> Is that problem fixed?
[22:50:51] <Yuv422> the vehicle prob?
[22:51:05] <Malignant_Manor> No, the cursor issue.
[22:51:22] <Yuv422> I think I fixed it with that logic you see there. ;-)
[22:51:46] <Yuv422> but I do have a fix for the balloon issue
[22:51:52] <Malignant_Manor> I'm not even sure what you mean by cursor.
[22:52:05] <Yuv422> just testing the horses to make sure they still work
[22:52:25] <Yuv422> the spinning ankh
[22:53:15] <Malignant_Manor> Ah, I see, input mode
[22:53:27] <Malignant_Manor> I guess only allow it for that case.
[22:53:57] <Yuv422> yeah that would work
[22:54:16] <Malignant_Manor> Is it needed in keyboard mode?
[22:55:26] <Yuv422> I'd caution against touching the msgscroll logic
[22:55:50] <Yuv422> it is quite fragile and changing that might muck up something else
[22:56:00] <Yuv422> maybe wait until after release
[22:56:24] <Yuv422> unless it is causing text to scroll off and never be read by the player
[22:56:45] <Yuv422> or some other issue that effects the game play
[22:57:08] <Malignant_Manor> If it was only added for the cursor, then why would it be needed for other things?
[22:57:45] <Malignant_Manor> It already checks to see if text can fit.
[22:58:12] --> Dominus1 has joined #nuvie
[22:59:58] <Yuv422> it might be ok in this case
[23:00:29] <-- Dominus has left IRC (Ping timeout: 240 seconds)
[23:00:32] <Malignant_Manor> I'll try and do some testing later on.
[23:00:39] <Yuv422> I'm going to commit my vehicle fix
[23:00:45] <Malignant_Manor> Okay
[23:01:53] <Malignant_Manor> Before the token is added, there is the line
[23:01:54] <Malignant_Manor> if(msg_line->total_length + token->length() > scroll_width)
[23:02:29] <Yuv422> that is if the new token goes over the width
[23:02:44] <Yuv422> the other was if the new token fix exactly in the width
[23:02:57] <Malignant_Manor> Yeah, the cursor should only show up in the two modes.
[23:04:46] <Malignant_Manor> Anyway, I'll make sure to do some testing.
[23:07:36] <Malignant_Manor> talking needs a case too
[23:07:36] <CIA-75> nuvie: 03efry * r1526 10/nuvie/trunk/ (4 files in 3 dirs):
[23:07:37] <CIA-75> nuvie: * Changed Actor::init_from_obj() to allow changing the base_obj_n.
[23:07:37] <CIA-75> nuvie: This is needed when using vehicle actors as they need to change their
[23:07:37] <CIA-75> nuvie: base_obj_n to allow changing from different vehicle types.
[23:09:18] <Dominus1> Malignant_Manor: I don't have a case sensitive fs on my mac, I only stumbled over case issues when I tried the (now defunct) iOS build of Exult
[23:09:56] <Malignant_Manor> I thought you had some issues with Nuvie and case sensitivity before.
[23:10:21] <Dominus1> hmm
[23:10:29] <Dominus1> don't remember
[23:10:34] <Dominus1> :)
[23:11:18] --- Dominus1 is now known as Dominus
[23:12:34] <Malignant_Manor> Oh well
[23:17:12] * Dominus is struggling with configure and why he can't produce a quoted AC_SUBST
[23:20:25] <wjp> quoted how exactly?
[23:22:15] <Dominus> this is what I've done so far http://pastebin.com/cUGP4q7Z
[23:23:39] <Dominus> the result is that I have OSX_CODE_SIGNATURE = Developer ID Application in the makefile
[23:23:53] <Malignant_Manor> Eric, can you look at this bug? https://sourceforge.net/tracker/index.php?func=detail&aid=3544896&group_id=76419&atid=547063
[23:23:56] <wjp> Dominus: yes
[23:24:28] <Malignant_Manor> I'm not sure why this hasn't bugged you since you play with the keyboard as your main control scheme.
[23:24:37] <Dominus> but it seems that the make is ignoring the quotes around $(OSX_CODE_SIGNATURE) later and thus fails to execute
[23:25:14] <wjp> the CODESIGN= line looks fishy
[23:25:22] <wjp> it has outer and inner double quotes
[23:25:24] <Dominus> it is
[23:25:31] <Dominus> the outer quotes are wrong
[23:25:51] <Dominus> bah, then it is working
[23:26:11] <wjp> I'd skip the CODESIGN variable entirely, by the way
[23:26:14] <wjp> just directly put
[23:26:18] <wjp> if WITH_OSX_CODE_SIGNATURE
[23:26:28] <wjp> codesign -f -s "$(OSX_CODE_SIGNATURE)" $(bundle_name)
[23:26:29] <wjp> endif
[23:26:31] <wjp> in the bundle target
[23:26:52] <wjp> (with proper whitespace in front of that codesign line)
[23:27:40] <wjp> and the test for setting the default is inverted, I think? Other than that it looks good
[23:28:28] <wjp> oh, one more nitpick: the codesign manpage called the -s option the identity, so maybe "--with-macosx-code-signature=identity" in the help string too?
[23:29:09] <wjp> and welcome to the wonderful world of mixing different scripting languages :-)
[23:29:22] * Dominus is following wjp's advice and not ignoring it
[23:29:27] <wjp> (m4 and bash in the case of configure, and make and bash for the Makefile... :-) )
[23:29:41] <wjp> with all kinds of fun quoting problems, some of which are _actually_ not solvable...
[23:29:42] <Dominus> yes, my head hurts :)
[23:31:17] <Dominus> wjp, about the test for default setting being inverted, I think I got it right
[23:32:20] <Dominus> when I just pass --with-macosx-code-signature to configure I get the default and when I use --with-macosx-code-signature="some sig" I get "some sig" in the makefile
[23:34:49] <wjp> ok :-)
[23:35:02] <wjp> I guess it's getting late :-)
[23:35:20] <Dominus> http://pastebin.com/G8ntQNkm is the up to date one now
[23:36:08] <wjp> is there a specific reason you remove trailing slashes, by the way?
[23:36:35] <wjp> ah, wait, I see, that's from the lib path
[23:36:45] <wjp> you can remove the sed 's/\/*$//' line
[23:36:57] <wjp> (it removes trailing slashes)
[23:37:13] <Dominus> thanks :)
[23:37:57] <Dominus> hmm, I think I should mention the default in the configure help but then it is getting awfully long...
[23:42:17] <wjp> hm, interesting
[23:42:30] <wjp> apparently the default value for $withval is "yes"
[23:42:48] <Dominus> hmm
[23:42:51] <Dominus> http://pastebin.com/Xdzv2Q74
[23:43:30] <wjp> so I think the test line should be: if test x$with_macosx_code_signature = xyes; then
[23:43:36] <Yuv422> I'm off now
[23:43:37] <Yuv422> cya
[23:43:38] <-- Yuv422 has left IRC (Quit: Yuv422)
[23:44:51] <Dominus> wjp, yes, that works
[23:45:07] <wjp> oh, didn't know that @<:@ syntax; nice
[23:45:21] <Dominus> I lifted that off the exult configure.ac :)
[23:45:56] <Malignant_Manor> R.I.P.
[23:46:17] <Malignant_Manor> Exult is dead.
[23:47:01] <Dominus> default value for $withval is "yes" -> that could make me change a lot of stuff in some projects :)
[23:47:28] * Dominus ignores Malignant_Manor's comment for now :)
[23:47:48] * Dominus hopes that someone picks up the exult pieces at some point...
[23:48:30] --> Kirben has joined #nuvie
[23:49:00] <Dominus> wjp, if you have some free time, can you look at the end of configure.ac? there is some linux only stuff that should probably ifed
[23:49:51] <Dominus> and do you think I can commit these configure changes? I think it looks good now
[23:50:17] <wjp> the AC_CONFIG_FILES ones? Those shouldn't hurt
[23:50:23] <Dominus> (only changed the withval check)
[23:50:55] <Dominus> yes, these don't hurt, only give a slight warning
[23:51:04] <wjp> don't forget to remove the CODESIGN= block from Makefile.am
[23:51:05] <wjp> warning?
[23:51:32] <Dominus> WARNING: 'builds/linux/nuvie.cfg.in' seems to ignore the --datarootdir setting
[23:51:41] <Dominus> probably on linux too, though
[23:52:03] <Dominus> thanks for reminding me on the codesign block - copied and forgot it...
[23:53:50] <wjp> hm
[23:54:04] <wjp> right
[23:54:49] <wjp> apparently something since autoconf 2.60
[23:55:10] <wjp> those will have to be fixed
[23:58:24] <-- Dominus has left IRC (Read error: Connection reset by peer)
[23:58:24] --> Dominus1 has joined #nuvie
[23:59:45] <CIA-75> nuvie: 03dominus * r1527 10/nuvie/trunk/ (ChangeLog Makefile.am configure.ac): * added --with-macosx-code-signature=identity to configure to allow code signing on Mac OS X. By default it uses "Developer ID Application" with which an identity generated by Apple should begin with.