Log for #openttdcoop.devzone on 14th April 2012:
Times are UTC Toggle Colours
00:23:54  *** Zuu has quit IRC
04:54:19  *** andythenorth has joined #openttdcoop.devzone
06:08:57  <Brot6> FISH - Revision 711:b0f5f5da25a2: Feature: generation 1 graphics for Large Coaster (andythenorth) @
06:13:07  *** ODM has joined #openttdcoop.devzone
06:50:40  <Brot6> CHIPS Station Set - Revision 181:dc43f81a33be: Add: source file for large pax building (andythenorth) @
06:57:29  <Brot6> FIRS Industry Replacement Set - Bug #3085 (Confirmed): Unwanted closures (andythenorth) @
07:15:25  <Brot6> FIRS Industry Replacement Set - Revision 2728:0de274168e1f: Codechange: remove legacy tiles (closes ... (andythenorth) @
07:15:25  <Brot6> FIRS Industry Replacement Set - Code Review #3430 (Closed): 'legacy' tiles in basetiles.pnml (andythenorth) @
07:21:54  *** Nat_aS has quit IRC
07:37:09  <Brot6> NewGRF Meta Language - Revision 1883:3bf8cd44f9e3: Fix: ind. tile 'autoslope' cb num was set to 0x3B... (andythenorth) @
07:38:48  <Brot6> FIRS Industry Replacement Set - Bug #3503 (Closed): Terraform underneath quarry and claypit is possi... (andythenorth) @
07:45:08  <Brot6> FIRS Industry Replacement Set - Revision 2729:1dba6f4919fd: Docs: update changelog preparatory to 0.... (andythenorth) @
07:46:10  <Brot6> FIRS Industry Replacement Set - Revision 2730:040060e570fc: Added tag 0.7.2 for changeset 1dba6f4919... (andythenorth) @
07:53:36  <Brot6> firs: update from 0.7.1 to 0.7.2 done -
08:13:07  *** andythenorth has quit IRC
08:28:42  *** andythenorth has joined #openttdcoop.devzone
08:31:17  <andythenorth> hmm
08:31:24  * andythenorth places a bet that FIRS md5 sums won't match
08:31:27  <andythenorth> due to new nml
08:31:41  <andythenorth> if I'm right, the bundle on the CF is still broken
08:34:10  *** seberoth has quit IRC
08:34:40  *** seberoth has joined #openttdcoop.devzone
08:37:18  <planetmaker> new FIRS?
08:37:26  <andythenorth> new FIRS
08:37:48  <planetmaker> yes, better upload the one compiled by the CF or first compare md5sums
08:37:58  <andythenorth> md5 doesn't match
08:38:01  <andythenorth> expected
08:38:11  <planetmaker> with different NML: yes
08:38:14  <planetmaker> with same NML: no
08:38:19  <andythenorth> I patched NML
08:38:29  <andythenorth> release can wait until CF NML updates
08:38:53  <planetmaker> uh, did NML change in the last 24 hours?
08:38:59  <andythenorth> I need to retract some of my 'omfg why is the FIRS codebase full of new bugs' comments in recent months :P
08:39:09  <andythenorth> at least one of them so far has been an NML bug
08:39:46  <planetmaker> ah
08:39:50  <planetmaker> I see
08:40:01  <andythenorth> the beauty of NML is that I can read most of the src
08:40:06  <andythenorth> and fix it :o
08:40:15  <andythenorth> grfcodec...not
08:41:13  <planetmaker> grfcodec doesn't know what properties and variables mean. Thus this bug could not even happen there
08:41:20  <planetmaker> only with nforenum maybe ;-)
08:41:58  <andythenorth> anyway, one fewer FIRS bug
08:42:08  <Terkhen> :)
08:42:10  <andythenorth> it first got posted at flyspray, then bounced to FIRS, finally it's NML :)
08:42:30  <andythenorth> and the release has lots of translation updates
08:46:10  <planetmaker> I skip my comment to point out that mb's answer to your smoke thread displays gross ignorance of how newgrfs work :-P
08:46:19  <planetmaker> he doesn't want actionA
08:47:31  <andythenorth> I thought he was somewhat expert on newgrfs?
08:47:37  <andythenorth> he was there when they were invented
08:47:38  <planetmaker> and I find it funky that he replies to your patch "but I didn't want"...
08:47:54  <planetmaker> he wants newgrf smoke. That means not actionA at all
08:48:21  <planetmaker> (you want that, too, I assume)
08:48:49  <planetmaker> and he's no expert in it. He just knows how to write some stuff from long experience. Not from understanding of the foundations
08:49:17  <planetmaker> *some newgrf stuff
08:50:37  <andythenorth> I actually don't want newgrf smoke, not really
08:50:47  <andythenorth> it means I'd have to specify my own effect vehicles
08:50:55  <andythenorth> and handle animating / triggering them
08:51:01  <andythenorth> boring :P
08:51:02  <andythenorth> more code
08:51:10  <andythenorth> current smoke is more than good enough for me :P
08:51:42  <planetmaker> :-) You want offsets :-P
08:53:08  <andythenorth> offsets I can probably hack on
08:54:06  <planetmaker> they would go a long way
08:54:55  *** ODM has quit IRC
09:04:49  *** andythenorth has quit IRC
09:11:17  *** Zuu has joined #openttdcoop.devzone
09:53:11  *** andythenorth has joined #openttdcoop.devzone
10:56:13  <andythenorth> Terkhen planetmaker I am forcing myself to do some FIRS work
10:56:31  <andythenorth> it's not too hard if I just do a few tickets
10:56:40  <planetmaker> :-)
10:56:46  <andythenorth> mostly they are bugs that need replicating and finding though
10:56:52  <andythenorth> and that's the most demotivating :P
10:57:00  <andythenorth> the debt needs paying down
10:57:20  <planetmaker> the more elaborate a project gets the more the amount of that work takes usually ;-)
10:57:37  <andythenorth> and also not doing it...makes adding new features harder
10:57:42  <andythenorth> :)
10:58:02  <planetmaker> it's one of the problems of OpenGFX itself: you can't invent new features
10:58:24  <Terkhen> good :)
10:58:44  <Terkhen> yes, it's a problem when code grows huge
10:58:47  * planetmaker ponders releasing OpenGFX 0.4.4
10:59:13  <andythenorth> releases are good
11:01:42  <andythenorth> I've pulled these tickets for 0.7.3
11:01:44  <andythenorth>
11:01:47  <andythenorth> it's done when it's done :P
11:02:01  <andythenorth> any help appreciated ;)
11:02:53  <Brot6> OpenGFX - Revision 973:8daf216e4243: Update: Changelog for 0.4.4 (planetmaker) @
11:03:50  <Brot6> FISH - Bug #3659 (Closed): DevZone compile failed (andythenorth) @
11:11:54  <Ammler> 
11:11:55  <Ammler> - Add: [Makefile] Introduce target 'clean-gfx' which deletes png files which can be re-built from source and allow 'maintainer-clean' to delete the opengfx.check.md5
11:12:04  <Ammler> very evil ^
11:12:37  <planetmaker> why?
11:12:54  <Ammler> because then my package will again delete the check file
11:13:04  <Ammler> is there a distclean?
11:13:06  <planetmaker> oh :-(
11:13:08  <planetmaker> yes
11:13:18  <planetmaker> run distclean clean-gfx
11:13:24  <planetmaker> if you want to re-build everything
11:13:30  <Ammler> well, add clean-gfx to distclean
11:14:10  <Ammler> distclean is new?
11:14:24  <planetmaker> you might end up then with something you can't build anymore. distclean is also supposed to re-instate the files as supplied
11:14:29  <planetmaker> distclean is not new
11:15:02  <Ammler> I don't like to use a uncommon target for package maintenance
11:15:12  <planetmaker> distclean:: clean
11:15:12  <planetmaker> 	$(_V)-rm -rf $(SRC_DIR)/$(FILENAME_STUB) $(DIR_NAME_SRC) $(DIR_NAME)
11:15:12  <planetmaker> 	$(_V)-rm -rf $(DIR_BASE)*nightly*.zip
11:15:13  <planetmaker> 	$(_V)-rm -rf $(MD5_FILENAME)
11:15:37  <planetmaker> I added this target on special request of blathijs :-)
11:15:53  <planetmaker> basically this change is his recomendation
11:16:07  <Ammler> yes, I am sure, he wouldn't mind to remove the pngs too
11:16:21  <planetmaker> he actually did
11:16:26  <planetmaker> he wanted it separate
11:16:32  <Ammler> I assume, he asked for this target before you introduced gimp
11:16:38  <planetmaker> on grounds that distclean should not delete stuff which you got in the tar
11:16:44  <Rubidium> maintainer-clean is meant for the maintainer of the package, so there it should remove the .md5
11:16:47  <planetmaker> distclean should restore the tar to pristine state
11:17:03  <Ammler> oh, planetmaker you see, maintainer of the package
11:17:07  <Ammler> not for you
11:17:09  <planetmaker> yes. That's me
11:17:12  <Ammler> so please keep the check file
11:17:18  <planetmaker> not a distribution maintainer, Ammler
11:17:26  <Ammler> ah
11:17:38  <Ammler> so there is no target for me? :'-(
11:17:38  <planetmaker> distclean is for distributors
11:17:44  <planetmaker> which is for you
11:17:54  <Ammler> no, does not remove files, which should be built
11:18:06  <planetmaker> distclean should clean the pngs only, if the pngs were not part of the package
11:18:27  <Ammler> so you think, dist should not rebuild png?
11:18:27  <planetmaker> the problem in this is that we do supply the pngs.
11:18:42  <Rubidium> s/clean-gfx/mostlyclean/ ;)
11:18:46  <planetmaker> By default not. Those who want to build them, they can also call clean-gfx now
11:18:57  <Rubidium> "Like ‘clean’, but may refrain from deleting a few files that people normally don't want to recompile."
11:18:58  <Ammler> ok
11:19:07  <planetmaker> Rubidium: that would change it. clean-gfx now *only* deletes the pngs which can be built
11:20:02  <Ammler> so I use "distclean clean-gfx", well why not :-P
11:20:35  <Ammler> you should change the spec in doc in such cases
11:20:55  <planetmaker> I don't really mind there exactly what. I'm happy to implement the clean targets as people need or desire them. Only maintainer-clean is for me to also remove the check.md5 file
11:21:11  <planetmaker> and *everything* which can be generated from versioned source
11:21:52  <planetmaker> maintainer-clean is the ultimate cleaning ;-)
11:22:07  *** andythenorth has quit IRC
11:22:28  <planetmaker> I could change distclean to also clean the pngs, if desired. And then clean and clean-gfx jointly give something like distclean
11:23:03  <planetmaker> maybe that's better as all distributors seem to want build it
11:23:15  <planetmaker> from graphics source
11:23:39  <Ammler> whatever you do, it would be nice to keep up2date
11:24:17  <planetmaker> hm, yes, right
11:24:43  <planetmaker> honestly I don't know what to do. I get diverging input :-)
11:25:26  <Ammler> I am fine with distclean
11:25:44  <Ammler> if that buids, it is ok, also adding clean-gfx is ok
11:25:46  <Rubidium> IMO distclean shouldn't remove things that are in the tar
11:26:17  <Ammler> maybe use a nicer target name
11:26:18  <Rubidium> for what it's worth, OpenTTD's maintainer-clean doesn't remove .ottdrev
11:26:32  <planetmaker> shouldn't it?
11:26:56  <Ammler> clean-binaries
11:27:01  <Ammler> or something
11:27:18  <planetmaker> but... it cleans graphics :-)
11:27:25  <Rubidium> for OpenTTD there's no point in doing so as the compile farm inserts it into the tarballs, so it never ends up in svn
11:27:27  <Ammler> for now
11:27:36  <planetmaker> and fits opposite of 'make gfx'
11:27:53  <Ammler> ok
11:28:24  <planetmaker> that was my reasoning to make it a separate clean target.
11:28:43  <planetmaker> It's optional. Not needed. And needs much work to undo
11:28:49  <Brot6> Dutch Trains 2.0 - Revision 458:86d20de134c5: Fix (r457): don't forget to commit graphics file as we... (foobar) @
11:28:49  <Brot6> Dutch Trains 2.0 - Revision 459:1bd0ddda4219: Change (r451): different colour for Mat '24 mP liverie... (foobar) @
11:28:49  <Brot6> Dutch Trains 2.0 - Revision 460:5d19d9b43ceb: Feature: Additional livery for Mat '24 coach (graphics... (foobar) @
11:28:51  <Brot6> Dutch Trains 2.0 - Feature #3910 (Closed): Mat'24 additional sprites (foobar) @
11:28:58  <planetmaker> much work in time-consuming to build
11:30:46  <Ammler> very sad, you can't merge from stable to default or backwards, I wonder if that really is not possible
11:32:26  <planetmaker> you can merge
11:32:40  <planetmaker> look at all the recent backports.
11:32:45  <planetmaker> Though it's not merge. But graft
11:32:49  <Ammler> yep
11:32:53  <Ammler> that's the point
11:32:55  <planetmaker> hg graft XXX
11:33:00  <planetmaker> and I was done
11:33:20  <Ammler> and graft is a "new transplant"?
11:33:25  <planetmaker> kinda
11:33:54  *** frosch123 has joined #openttdcoop.devzone
11:34:11  <Ammler> the only reason, you weren't able to merge were teh release info, right?
11:34:29  <Ammler> how would those hurt in default?
11:35:49  <planetmaker> Rubidium: I'm puzzled by OpenTTD paths again, it doesn't find my opengfx 0.4.4 (/Users/ingo/Documents/OpenTTD/data/opengfx) but uses OpenGFX 0.4.3 instead (~/Documents/OpenTTD/content_download/data/OpenGFX-0.4.3.tar)
11:37:02  <planetmaker> Ammler: what do you want to merge?
11:37:04  <planetmaker> and why?
11:37:13  <Ammler> what path does the bananans ogfx have?
11:37:26  <planetmaker> The difference is grf v7 vs. grf v8, change of source paths, ...
11:37:37  <planetmaker> the latter is bananas, Ammler
11:37:45  <Ammler> planetmaker: it was just a generic note that you branched opengfx quite ugly
11:38:06  <planetmaker> Really, I don't understand your issue with the branching
11:38:15  <Ammler> planetmaker: I meant, what path does the bananas ogfx have
11:38:16  <planetmaker> Do you know *why* I branched?
11:38:40  <planetmaker> The reason was that I wanted to change some things quite drastically. And I did.
11:38:54  <planetmaker> OpenGFX trunk won't work with OpenTTD 1.1 and earlier
11:39:03  <planetmaker> While OpenGFX 0.4 will
11:39:06  <Ammler> but you still have almost everything copied from 0.4 to default
11:39:16  <planetmaker> But not vice versa
11:39:28  <Ammler> merge is not both ways, is it?
11:39:47  <planetmaker> It's not. Please test locally what you get from a merge
11:40:11  <Ammler> so if you would have merged stable to default it would have been nice
11:40:11  <planetmaker> And enjoy finding of the small fraction you actually do want to merge
11:40:57  <planetmaker> I don't think so. And I've had hated the time spending on fixing zillions of merge issues due to diverging code base
11:41:25  *** jmodule has joined #openttdcoop.devzone
11:41:35  <planetmaker> I much prefer a 2-seconde graft command there
11:42:23  *** jmodule has quit IRC
11:43:15  <Ammler> well, I guess mostly because you tried to merge default to stable
11:43:50  <Rubidium> planetmaker: no idea why that would happen
11:46:58  <Brot6> OpenGFX - Revision 974:e49b3799321b: Doc: rpm spec (Ammler) @
11:51:09  <Ammler> oh, distclean is useless to run after unpacking tar :-)
11:51:21  <Ammler> make clean-gfx would suiffice
11:51:38  <planetmaker> it would, yes
11:51:48  <Ammler> what does blathjis need it for?
11:52:45  <planetmaker> dunno really :-)
11:58:01  <Ammler> is this WE the big day of openttd release or will we get another fool?
12:02:19  <planetmaker> probably
12:02:51  <planetmaker> so... should I keep the clean-gfx (for now)?
12:02:59  <Ammler> fine with me
12:03:11  <Ammler> if you change it (again), please change the spec
12:04:28  <planetmaker> I will try to remember
12:05:39  <Ammler> more important (IMO) would be a check if gimp is needed and force 1 thread only then
12:06:51  <planetmaker> yes... blathijs had some nice ways to work with gimp successfully on linux multithreaded
12:07:05  <planetmaker> but it doesn't work for me. But maybe it should just be done his way, if on linux
12:07:18  <planetmaker> that's for 0.5.0 though
12:07:29  <planetmaker> or 0.4.5, if there should be one
12:14:56  <Brot6> OpenGFX - Bug #3439: build does not work with -j > 1 (planetmaker) @
12:22:34  <Brot6> Dutch Trains 2.0 - Revision 461:b978704cc829: Feature: New graphics for Plan L coach (graphics by Vo... (foobar) @
12:22:35  <Brot6> Dutch Trains 2.0 - Revision 462:e216a348f578: Codechange: rename Mat '24 mP graphics file (foobar) @
12:22:35  <Brot6> Dutch Trains 2.0 - Revision 463:46be0ddba9f9: Codechange: Add GPL notice to source code files (foobar) @
12:23:39  <Brot6> Dutch Trains 2.0 - Revision 464:3cdc5f1f6a52: Docs: prepare for release (foobar) @
12:23:39  <Brot6> Dutch Trains 2.0 - Revision 465:e922b98c0eb0: Release: 2.0.0-alpha2 (foobar) @
12:31:35  <planetmaker> hm...
12:31:48  <planetmaker> does that shed some light, Rubidium?
12:33:12  <frosch123> it should only pick the newest version
12:33:22  <Brot6> dutchtrains: update from 2.0.0-alpha1 to 2.0.0-alpha2 done -
12:33:29  *** FooBar has joined #openttdcoop.devzone
12:33:35  <planetmaker> it doesn't. I expect r973, but it uses 0.4.3 which is r928 or so
12:35:37  <planetmaker> look at the 1st line, line 31 and 86 ff
12:36:12  <frosch123> it prefers older versions if the md5sum do not match
12:36:30  <planetmaker> hm. *That* might explain it
12:36:35  <planetmaker> I need to check
12:36:48  <planetmaker> It's the old OpenGFX and I possibly built with too new nml
12:36:55  <planetmaker> which has not the new md5sum check
12:50:44  <planetmaker> frosch123: thanks for the valuable hint. That was the reason
12:51:05  <frosch123> :)
13:57:40  <Brot6> Dutch Trains 2.0 - Bug #3911 (New): Purchase sprite wagon DDM/NID (Mahoo76) @
14:02:06  *** andythenorth has joined #openttdcoop.devzone
14:11:52  <Brot6> Japanese Trees - Revision 2:3a8697056702: Add support for snow in temperate in TTDPatch (PaulC) (dandan) @
14:11:52  <Brot6> Japanese Trees - Revision 3:2eb7e956ece3: Add support for DOS palette. (dandan) @
14:12:35  <Brot6> Japanese Trees - Feature #3776 (Closed): [TTDP] Snowy trees in temperate (dandan) @
14:13:46  <Brot6> Dutch Trains 2.0 - Support #3911 (New): Purchase sprite wagon DDM/NID (Mahoo76) @
14:13:46  <Brot6> Dutch Trains 2.0 - Support #3911: Purchase sprite wagon DDM/NID (foobar) @
15:17:05  *** andythenorth has quit IRC
15:36:35  *** andythenorth has joined #openttdcoop.devzone
15:37:22  <andythenorth> is it still going on?
15:37:54  <V453000> nah
15:38:16  <andythenorth> k
15:41:29  <andythenorth> planetmaker: does Radmir's patch here look plausible?
15:45:12  <andythenorth> hmm
15:45:38  <andythenorth> on game start, turning on debug text reveals that 'days since last delivery' is about 748,000
15:47:04  <andythenorth> ho
15:47:11  <andythenorth> yes that patch looks plausible :)
15:47:24  <andythenorth> currently the register numbers are used instead of the register values :)
15:49:09  <Brot6> FIRS Industry Replacement Set - Bug #3085: Unwanted closures (andythenorth) @
16:00:34  *** FooBar has quit IRC
16:12:21  <Brot6> FIRS Industry Replacement Set - Revision 2731:779079aaabf3: Fix (possibly): prevent unwanted closure... (andythenorth) @
16:12:21  <Brot6> FIRS Industry Replacement Set - Bug #3085 (Closed): Unwanted closures (andythenorth) @
16:33:25  *** andythenorth is now known as Guest306
16:33:25  *** andythenorth has joined #openttdcoop.devzone
16:37:09  <Brot6> Dutch Trains 2.0 - Feature #3899: Postrijtuigen and similar (Voyager1) @
16:53:29  *** andythenorth is now known as Guest309
16:53:29  *** andythenorth has joined #openttdcoop.devzone
16:55:44  *** andythenorth has quit IRC
16:55:58  *** andythenorth has joined #openttdcoop.devzone
17:18:33  <Brot6> nml: update from r1882 to r1883 done -
17:25:17  *** andythenorth has quit IRC
17:32:33  *** andythenorth has joined #openttdcoop.devzone
17:33:06  <andythenorth> :)
17:37:41  <andythenorth> so how do I get FIRS 0.7.2 to rebuild with new NML?
17:37:43  <andythenorth> patience?
17:41:34  <Brot6> Japanese Landscape - Revision 20:d7c3f47bb768: Update foundations (Toni's sprites) (dandan) @
17:42:17  <Rubidium> andythenorth: change the .devzone spec?
17:42:27  <Rubidium> I reckon Ammler can tell you
17:51:19  <andythenorth> it's weird to have a software tag that is dependent on an otherwise unrelated compile farm :)
17:57:30  <Brot6> Dutch Track Set - Revision 65:46eec9d806bf: Feature: Betuweroute Tracks (By Yoshi) (Transportman) @
18:33:50  <Brot6> Japanese Stations - Revision 10:074a10cf85af: Update foundations and station foundations (Toni's spr... (dandan) @
18:33:50  <Brot6> Japanese Stations - Revision 11:4332374d0303: Add support for DOS palette. (dandan) @
18:50:55  <Brot6> GRFCodec - Bug #3912 (New): Bogus white pixel warning followed by "error 252" (oberhuemer) @
19:10:03  <Brot6> GRFCodec - Bug #3912: Bogus white pixel warning followed by "error 252" (Rubidium) @
19:21:01  <Brot6> GRFCodec - Bug #3912 (Closed): Bogus white pixel warning followed by "error 252" (oberhuemer) @
19:21:01  <Brot6> GRFCodec - Revision 929:702c5db9ca6a: - Fix #3912 reading incorrect graphics when multiple graphics ... (Rubidium) @
19:21:01  <Brot6> GRFCodec - Bug #3912 (Closed): Bogus white pixel warning followed by "error 252" (Rubidium) @
20:18:10  *** jmodule has joined #openttdcoop.devzone
20:22:00  *** jmodule has quit IRC
20:58:26  *** frosch123 has quit IRC
21:04:35  *** andythenorth has quit IRC
21:08:37  *** Zuu has quit IRC
22:23:50  *** Zuu has joined #openttdcoop.devzone

Powered by YARRSTE version: svn-trunk