Config
Log for #openttdcoop.devzone on 26th September 2011:
Times are UTC Toggle Colours
06:15:16  <Brot6> NewGRF Meta Language - Feature #3095 (Assigned): Dependency check (output of image files in makef... (planetmaker) @ http://dev.openttdcoop.org/issues/3095
06:19:06  <Brot6> NewGRF Meta Language - Feature #3095: Dependency check (output of image files in makefile format) (planetmaker) @ http://dev.openttdcoop.org/issues/3095#change-8008
06:25:54  <Brot6> NewGRF Meta Language - Feature #3095: Dependency check (output of image files in makefile format) (planetmaker) @ http://dev.openttdcoop.org/issues/3095#change-8009
06:28:12  <planetmaker> ^ one step more to proper gimp support, Terkhen ;-)
06:50:46  *** ODM has joined #openttdcoop.devzone
06:59:25  *** ODM has quit IRC
07:03:45  <Ammler> planetmaker: silly question, but does it work, if a dep is missing?
07:04:52  <planetmaker> hm... the question is not silly :-O
07:06:58  *** ODM has joined #openttdcoop.devzone
07:09:35  <Ammler> oh, and Guten Morgen
07:09:43  <planetmaker> :-) same to you!
07:13:33  <planetmaker> new diff attached ;-)
07:13:43  <planetmaker> thanks
07:13:53  <Brot6> NewGRF Meta Language - Feature #3095: Dependency check (output of image files in makefile format) (planetmaker) @ http://dev.openttdcoop.org/issues/3095#change-8010
07:55:31  <Brot6> NewGRF Meta Language - Feature #3095: Dependency check (output of image files in makefile format) (planetmaker) @ http://dev.openttdcoop.org/issues/3095#change-8011
07:56:39  *** hanf has joined #openttdcoop.devzone
08:39:37  <Terkhen> planetmaker: great :)
09:39:59  *** hanf has quit IRC
10:01:06  <V453000> all of the sounds must be in midi, including vehicle sounds etc?
10:18:55  <Ammler> no
10:19:08  <Ammler> those are waves
10:19:25  <Ammler> check opensfx
10:19:33  <Ammler> or ukrs
10:19:54  <Ammler> dunno, if there are newgrfs with sounds on devzone
10:20:19  <Ammler> there is a nml example with sound, at least a test
10:21:25  <V453000> oh
10:21:26  <V453000> ok, thanks
10:26:37  <Ammler> sounds as midi could be hard to create :-)
10:26:50  <V453000> thats what I wondered :D
10:27:18  <Ammler> also I guess, openttd supports more bits for sounds
10:27:54  <Ammler> but that you need to check in the docs yourself
10:28:03  <Ammler> maybe the source is the only doc though :-P
10:31:50  <V453000> Great :D
10:32:17  <V453000> well if wav is going to work, I think it is all fine for me :)
10:37:25  <Brot6> NewGRF Meta Language - Feature #3095: Dependency check (output of image files in makefile format) (planetmaker) @ http://dev.openttdcoop.org/issues/3095#change-8014
10:42:27  <Ammler> planetmaker: you really should learn to use mercurial/git style instead svn stlye for patches :-P
10:42:39  <planetmaker> hm
10:42:56  <planetmaker> yes. I should check my global .hgrc
10:42:59  <Ammler> is that really how you prefer the review?
10:43:33  <planetmaker> hm... that quotes already git = True
10:43:39  <planetmaker> so I'm not sure why it doesn't work
10:43:54  <Ammler> I meant that you should make repo branches instead diffs
10:44:22  <Ammler> you guys could e.g. make a 2nd repo nml-dev or nml-crew or so
10:44:38  <Ammler> if you don't like to screw the main repo too much
10:44:38  <planetmaker> uhm... no
10:44:49  <planetmaker> it's a single patch
10:44:54  <Ammler> ugly
10:45:04  <Ammler> that is how people on svn time developed :-P
10:45:09  <planetmaker> ugly would be a separate commit for each of the unfinished things
10:45:14  <planetmaker> this is a simple thing
10:45:28  <planetmaker> it's not like it's a sub-repo
10:45:34  <Ammler> the merge to "default" is one commit
10:45:37  <planetmaker> the grf2nml would be worth a branch
10:46:02  <planetmaker> i.e. bigger things. But not a few-line patch which one wants feedback on.
10:46:04  <Ammler> every single patch you guys create for review is worth a (unnamed) branch
10:46:29  <Ammler> then a simple merge and it is is
10:46:39  <planetmaker> branches are not for review. But for development of a larger thing
10:46:47  <planetmaker> now all I need is a simple commit...
10:47:01  <planetmaker> or qfinish, if I'd have considered it worth for this
10:47:14  <Ammler> yeah, I see, you still think svnish :-P
10:47:36  <planetmaker> no :-) mq are our friend
10:48:01  <Ammler> mq is just a workaround for people like you
10:48:22  <Ammler> check modern software project
10:48:33  <Ammler> the master branch has mainly just merges
11:19:44  <planetmaker> Terkhen: maybe you want to check: http://devs.openttd.org/~planetmaker/patches/ogfx-rv-gimp.diff for OpenGFX+ RV and http://devs.openttd.org/~planetmaker/patches/dep_check5.diff for NML
11:20:03  <planetmaker> does that suit your needs wrt gimp?
11:22:35  <planetmaker> hm... I think I know where it can become better :-)
11:23:23  <Hirundo> If dep_checkN.diff has a single digit N, it's not finished yet :-)
11:23:47  <planetmaker> :-P
11:24:16  <planetmaker> I think the NML part could be considered finished. The makefile part not
11:26:16  * Hirundo found a typo
11:26:26  <Hirundo> "+        Return wether image files need parsing."
11:26:39  <Hirundo> That makes N+1 :P
11:26:42  <planetmaker> whether?
11:27:10  <Hirundo> my spellchecker suggests either whether or weather
11:27:22  <Hirundo> The latter might not be quite appropriate here
11:28:17  <planetmaker> :-)
11:28:21  * planetmaker checks weather
11:28:33  * planetmaker finds the weather (here) quite appropriate :-)
11:30:04  <Terkhen> why does it need a patch for nml?
11:30:27  <Hirundo> So NML can output a list of files it depends on
11:30:30  <planetmaker> ^
11:30:43  <Hirundo> Might not be trivial in the case of templated gfx paths etc
11:30:48  <Terkhen> and what does it do? it avoids recompiling after a commit/modification of the source file?
11:31:00  <Terkhen> oh, labels? :P
11:31:06  <planetmaker> first it changes to access layers by labels.
11:31:08  <Hirundo> bbl
11:31:17  <planetmaker> I'll probably have to work still a bit on the re-compilation thing
11:31:44  <planetmaker> I haven't tested that too thoroughly yet
11:34:23  <planetmaker> hm, yes :S
11:35:03  <Terkhen> why does it need changes to xcf files?
11:36:26  <planetmaker> Layers with spaces in the names
11:36:49  <planetmaker> Maybe it works with quoting them, but... usually that's a real PITA
11:44:11  <planetmaker> hm, the dep check seems to be totally borked, sorry, Terkhen :-)
11:44:45  <Terkhen> http://paste.openttdcoop.org/show/598/ <-- so that's to be expected?
11:45:08  <planetmaker> no. But you might need to clean first
11:45:30  <planetmaker> that is unexpected :-)
11:45:53  <Terkhen> that's after make clean / hg purge
11:46:13  <planetmaker> hm :S
11:46:20  <Terkhen> right, if I purge I lose files :P
11:49:04  <Terkhen> http://paste.openttdcoop.org/show/599/
11:51:00  <planetmaker> err.... very strange
11:52:25  <Terkhen> indeed
11:52:32  <Terkhen> maybe nml is checking files that it shouldn't be checking for dep?
11:53:14  <planetmaker> hm, ok. A missing custom_tags.txt triggers that
11:53:38  <planetmaker> it's a missing dep actually :-)
11:54:24  <Terkhen> :)
11:54:40  <planetmaker> it will be gone, if you add  $(TAG_FILENAME) to %.nml: ... in scripts/Makefile.nml:39
11:55:10  <planetmaker> The check of the lang files is unexpected at stages which don't compile the actual grf. I didn't expect that
11:58:17  <Terkhen> http://paste.openttdcoop.org/show/600/
12:00:05  <planetmaker> hm... can you run it with _V= ?
12:00:20  <planetmaker> I don't understand why it loops
12:01:57  * planetmaker now defined *.png and *.scm as precious. Re-building it every time sucks ;-)
12:02:49  <Terkhen> where was that var defined?
12:03:05  <planetmaker> which var?
12:03:16  <planetmaker> make _V=
12:03:33  <Terkhen> oh, ok :)
12:05:22  <Terkhen> http://paste.openttdcoop.org/show/601/
12:14:29  <planetmaker> peculiar...
12:17:08  <planetmaker> I don't have yet much hope that it is fixed, but can you try the new version of the patch (same link)?
12:18:22  <Terkhen> sure
12:21:15  <Terkhen> did you mean ogfx-rv patch or nml patch? I updated the ogfx-rv patch and I got the same results
12:24:12  <planetmaker> sorry, the ogfx-patch
12:24:38  <planetmaker> hm. I guess I'll have to try a few more things first. Thank for testing
12:25:04  <Terkhen> np :)
12:27:09  <planetmaker> what system / setup was is it you use, Terkhen?
12:27:28  <Terkhen> debian stable
12:27:33  <Terkhen> virtual machine
12:35:25  <planetmaker> hm, ok. I could test that myself meanwhile, I guess. I just need to install the stuff within that VM
12:35:38  <planetmaker> thanks again
12:35:57  <planetmaker> it's still a nearly virgin VM I have there ;-)
12:36:39  <planetmaker> except some odcctools checkout which didn't want to compile so far
12:52:32  <planetmaker> http://paste.openttdcoop.org/show/602/ <-- this output now has to become reproducable...
12:55:30  <Terkhen> maybe there is something wrong with my checkout
12:55:37  <Terkhen> hg status says it's fine, though
13:10:56  <Brot6> Example NewGRF Project - Feature #1441 (Rejected): Use 'custom_tags' for sed replacement in NFO p... (planetmaker) @ http://dev.openttdcoop.org/issues/1441#change-8015
13:14:15  <Brot6> Example NewGRF Project - Bug #1708 (Rejected): versions for action14 and branches (planetmaker) @ http://dev.openttdcoop.org/issues/1708#change-8016
13:15:04  <Brot6> Example NewGRF Project - Bug #2067 (Rejected): treat $(CLEAN_ADD) more gracefully (planetmaker) @ http://dev.openttdcoop.org/issues/2067#change-8017
13:16:33  <Brot6> Example NewGRF Project - Bug #2071 (Closed): File missing from new projects (planetmaker) @ http://dev.openttdcoop.org/issues/2071#change-8018
13:21:05  <Brot6> Example NewGRF Project - Feature #1173 (Closed): Update release for existing projects (planetmaker) @ http://dev.openttdcoop.org/issues/1173#change-8020
13:27:34  <Brot6> Example NewGRF Project - Feature #1277: Modularize the Makefile even more (planetmaker) @ http://dev.openttdcoop.org/issues/1277#change-8021
14:16:15  *** ODM has quit IRC
14:16:28  *** ODM has joined #openttdcoop.devzone
14:25:17  *** ODM has quit IRC
14:36:30  <planetmaker> Terkhen: http://devs.openttd.org/~planetmaker/patches/ogfx-rv-gimp.diff <-- this does the trick for me
14:43:09  <Terkhen> http://paste.openttdcoop.org/show/603/
14:43:57  <planetmaker> hmpf... but still not time to bash the messenger ;-)
14:44:19  <planetmaker> you deleted all *.dep before, did you?
14:46:52  <Terkhen> hg purge --all && patch -p1 -i ... && make
14:47:51  <planetmaker> hm. the patch seems not updated on the server...
14:50:38  *** hanf has joined #openttdcoop.devzone
14:51:15  <planetmaker> which is strange
14:57:00  <planetmaker> arg... there is it again, the infinite loop
17:03:53  <planetmaker> http://devs.openttd.org/~planetmaker/patches/index.php?source=ogfx-rv-gimp.diff <-- new try, Terkhen ;-)
17:04:02  <planetmaker> sorry for bothering you all the time with it ;-)
17:04:14  <Terkhen> thank you for expending time on this :)
17:04:16  <Terkhen> let's see
17:04:20  <planetmaker> was a really nasty thing...
17:04:34  <planetmaker> argument given twice - which obviously with some makes causes infinite loops
17:05:27  <planetmaker> there's also a number of png in the repo which we might want to delete from it
17:06:43  <Terkhen> the pngs that are not being generated?
17:07:54  <planetmaker> they should be...
17:08:13  <planetmaker> but there's a number of pngs in the repo which are there but are also generated
17:08:23  <planetmaker> at least in mine... hm
17:08:25  <Terkhen> I meant the pngs that don't have a xcf file
17:08:32  <Terkhen> those should be kept
17:08:37  <planetmaker> of course
17:08:39  <Terkhen> but it shouldn0t have any others... hmm
17:09:10  <Terkhen> http://paste.openttdcoop.org/show/604/ <--- sorry, but...
17:09:31  <planetmaker> ... hmpf
17:10:16  <planetmaker> what happens, if you add %.nml to .PRECIOUS in scripts/Makefile.common?
17:11:44  <planetmaker> hm... grrr
17:13:39  <Terkhen> maybe I'm doing something wrong? it is quite strange that it isn't working for me at all
17:14:09  <planetmaker> nah, you're right...
17:14:09  <Terkhen> my nml is up to date and patched... ogfx-rv is up to date too and with the latest patch
17:14:23  <planetmaker> try a "touch ogfx-rv.src.dep"
17:14:26  <planetmaker> and then run make
17:14:32  <planetmaker> does it do something else then?
17:14:50  <planetmaker> that seems to make a difference strangely
17:15:17  <Terkhen> I added %.nml and tried make clean -> infinite dep too
17:15:20  <Terkhen> tried touch and make
17:15:22  <Terkhen> now it is compiling
17:15:31  <Terkhen> hack a touch :P
17:15:43  <planetmaker> yeah, I'm considering it :-P
17:15:57  <planetmaker> but it's not proper
17:21:11  <Brot6> vactrainset: compile of r1 still failed (#3044) - http://bundles.openttdcoop.org/vactrainset/nightlies/ERROR/r1
17:21:46  <Terkhen> http://paste.openttdcoop.org/show/605/ <--- it ends with an error anyways
17:25:18  <planetmaker> hm
17:49:40  *** frosch123 has joined #openttdcoop.devzone
18:43:33  *** JVassie has joined #openttdcoop.devzone
19:02:04  <Brot6> clientpatches: compile of r22963 still failed (#2964) - http://bundles.openttdcoop.org/clientpatches/testing/ERROR/r22963
19:07:10  <Brot6> openttd-vehiclevars: update from r22961 to r22963 done - http://bundles.openttdcoop.org/openttd-vehiclevars/testing/r22963
19:08:22  <Brot6> serverpatches: compile of r22963 still failed (#2966) - http://bundles.openttdcoop.org/serverpatches/testing/ERROR/r22963
19:09:55  <Brot6> 32bpp-ez-patches: compile of r22963 still failed (#2446) - http://bundles.openttdcoop.org/32bpp-ez-patches/testing/ERROR/r22963
20:41:10  *** frosch123 has quit IRC
21:19:32  <Brot6> Total Town Replacement Set - Bug #3096 (New): Provide ECS Houses support (George) @ http://dev.openttdcoop.org/issues/3096
22:19:12  *** hanf has quit IRC
22:42:37  *** JVassie has quit IRC
23:39:21  <Brot6> Central European Train Set - Support #2919: Livery/color standards (michi_cc) @ http://dev.openttdcoop.org/issues/2919#change-8022

Powered by YARRSTE version: svn-trunk