#exult@irc.freenode.net logs for 12 Jan 2013 (GMT)

Archive Today Yesterday Tomorrow
Exult homepage

[00:48:34] --> Marzo has joined #exult
[00:51:12] <-- Kirben has left IRC ()
[02:32:00] --> Kirben has joined #exult
[02:32:01] --- ChanServ gives channel operator status to Kirben
[03:03:23] <-- Marzo has left IRC (Ping timeout: 252 seconds)
[03:30:59] <-- Kirben has left IRC ()
[04:50:57] --> Kirben has joined #exult
[04:50:58] --- ChanServ gives channel operator status to Kirben
[12:08:43] --> Marzo has joined #exult
[12:21:24] <-- Dominus has left IRC (Remote host closed the connection)
[12:21:56] --> Dominus has joined #exult
[12:21:56] --- ChanServ gives channel operator status to Dominus
[12:41:10] <-- Kirben has left IRC ()
[14:04:09] <-- Dominus has left IRC (Read error: Connection reset by peer)
[14:04:14] --> DominusExult has joined #exult
[14:04:15] --- ChanServ gives channel operator status to DominusExult
[15:12:54] <-- sh4rm4 has left IRC (Ping timeout: 276 seconds)
[15:26:34] --> sh4rm4 has joined #exult
[15:58:26] <-- Philantrop has left IRC (Remote host closed the connection)
[15:58:50] --> Philantrop has joined #exult
[17:59:51] <DominusExult> bah, automake 1.13 broke our configure now
[17:59:57] --- DominusExult is now known as Dominus
[18:40:18] <Dominus> wjp, are you there? About the autotools. one thing should be fixable right away, replacing AM_CONFIG_HEADERS with AC_CONFIG_HEADERS (configure.ac line 62)
[18:40:37] <Dominus> that really breaks autogen.sh right away
[18:41:06] <Dominus> and the warning Makefile.am:1: warning: 'INCLUDES' is the old name for 'AM_CPPFLAGS' (or '*_CPPFLAGS')
[18:41:26] <Dominus> should be straightforward as well
[18:43:47] <Dominus> what's less straightforward is configure.ac:55: warning: AM_INIT_AUTOMAKE: two- and three-arguments forms are deprecated. For more info, see:
[18:43:47] <Dominus> configure.ac:55: http://www.gnu.org/software/automake/manual/automake.html#Modernize-AM_INIT_AUTOMAKE-invocation
[18:46:08] <Dominus> Marzo, sorry, I forgot to mention that the blacksmiths fill the water from their pockets if there is no well. I noticed that when I looked for a good blacksmithing example in the original.
[18:47:25] <Marzo> NPCs do it with everything they need, it seems
[18:47:43] <Marzo> If they can procure it from nearby, they do; otherwise, they create it on the spot
[18:48:03] <Marzo> For example, blacksmiths usually pick sword blanks from nearby if any are available
[18:48:09] <Marzo> And so on
[18:48:11] <Dominus> otherwise it would have been a hell lot worse to create the map, I imagine
[18:48:17] <Marzo> Yeah
[18:48:34] <Marzo> By the way, have you tried saving (or quick-saving) several times in a row?
[18:48:45] <Marzo> (in Exult, I mean)
[18:48:49] <Marzo> (and recently)
[18:49:58] <Dominus> btw. ever since you took on pathfinding for the ships, I noticed a much better behaviour of the party members in following the Avatar. At the dock of the golden ankh, the party memebers that didn't follow the avatar onto the dock to the ship kept standing next to the dock at the shore. now they follow the avatar eventually
[18:50:07] <Dominus> (before double clicking the sail)
[18:50:30] <Dominus> I read the commit message and need to try the save thing :)
[18:52:00] <Marzo> The improvement is actually due to the code no longer allowing move_aside when a NPC is moving
[18:52:16] <Marzo> This could break paths
[18:52:43] <Marzo> This is more unlikely now
[18:53:25] <Marzo> For he save thing, I am toying with the idea of trying a smarter things than just randomly displaying it, but I am wondering how much effort to put in
[18:53:26] <wjp> Dominus: the start of section 6.4.1 seems to give instructions on how to upgrade that line
[18:54:47] <Dominus> marzo, not sure if it is really worth to put much effort in it. It's a nice touch and certainly something to do if you feel like it
[18:55:20] <Dominus> wjp. I tried the straight forward upgrade thing but that didn't work right away, I think
[19:23:00] <Dominus> wjp, If I try to change AM_INIT_AUTOMAKE($PACKAGE,$VERSION,no-define) to AC_INIT($PACKAGE, $VERSION)
[19:23:01] <Dominus> AC_CONFIG_SRCDIR(exult.cc) AM_INIT_AUTOMAKE
[19:23:27] <Dominus> and place that below the PACKAGE and VERSION defines, I get the warning
[19:23:43] <Dominus> configure.ac:15: warning: AC_INIT: not a literal: $PACKAGE
[19:23:43] <Dominus> configure.ac:15: warning: AC_INIT: not a literal: $VERSION
[19:29:56] <wjp> apparently it wants the literal names instead of variables
[19:31:26] <wjp> it might then in fact define the PACKAGE and VERSION variables based on that, but we'd have to check
[20:16:53] <-- Marzo has left IRC (Ping timeout: 244 seconds)
[21:18:16] <Dominus> wjp, yes, AC_INIT(exult, 1.5.0svn) sets PACKAGE and VERSION
[21:18:39] <Dominus> if I remove our own PACKAGE and VERSION define it works
[21:19:03] <Dominus> I now have to track down why my compile failes with a missing vorbis header, though :)
[21:23:21] <Dominus> missing compiler flags, only... :)
[21:37:40] <Dominus> wjp, this is a diff for this but it still needs advice http://pastebin.com/KT28Qvtj
[21:38:44] <Dominus> the package and version define is meaningless now hence the comment but I guess the AC_DEFINE_UNQUOTED can go as well
[21:39:00] <Dominus> and then there is the question of how far this is backward compatible
[21:39:35] <Dominus> or whether we actually need Ver_Major/MINOR/EXTRA
[21:41:53] <wjp> we don't need these three
[21:42:05] <wjp> and their AC_DEFINE_UNQUOTEDs can go too
[21:42:27] <Dominus> and the unquoted for package and version?
[21:42:27] <wjp> the VERSION one should still be necessary
[21:42:31] <Dominus> :)
[21:42:49] <wjp> PACKAGE too
[21:42:56] <Dominus> k
[21:43:10] <wjp> although I suppose these may be automatic nowadays too
[21:44:46] <wjp> if you want to try removing them, VERSION is displayed in many places in exult
[21:45:10] <Dominus> hmm, I'll give it a try
[21:46:17] <Dominus> then the last item is the makefile.am warning about INCLUDES versus AM_CPPFLAGS
[21:47:36] <wjp> should be safe to just replace INCLUDES = blah by AM_CPPFLAGS = blah
[21:48:25] <wjp> or at least you'll know soon enough if it isn't :-)
[21:48:32] <Dominus> thanks for confirming. I didn't want to blindly change things
[21:48:45] <Dominus> yeah, I'll give it a test :)
[21:54:03] <Dominus> hmm, couldn't find a problem with leaving the unquoted stuff out, but no idea if I got everything.
[21:59:20] --> Marzo has joined #exult
[23:02:38] <-- SugarCube has left IRC (Ping timeout: 255 seconds)
[23:15:17] --> SugarCube has joined #exult
[23:25:11] <Dominus> Marzo: I hope I'm able to play a bit next week to make sure (as if one ever could) there are no more bad regressions and then... I hope then we'll be able to throw out a new version
[23:39:45] <Marzo> Next week I hope I have time to remake the blacksmithing schedule
[23:40:35] <Marzo> (have a work due early monday, which is why I haven't done that yet)
[23:53:09] --> Kirben has joined #exult
[23:53:10] --- ChanServ gives channel operator status to Kirben