Back to Home / #openttd / 2018 / 10 / Prev Day | Next Day
#openttd IRC Logs for 2018-10-15

---Logopened Mon Oct 15 00:00:26 2018
00:31-!-triolus [] has quit [Remote host closed the connection]
00:57-!-Flygon_ [] has joined #openttd
00:57-!-Flygon_ is "Flygon" on #openttd
01:03-!-snail_UES_ [] has quit [Quit: snail_UES_]
01:05-!-Flygon [] has quit [Ping timeout: 480 seconds]
02:36-!-Wacko1976 [] has quit [Ping timeout: 480 seconds]
03:00-!-Supercheese [] has quit [Quit: Valete omnes]
03:29-!-Wacko1976 [] has joined #openttd
03:29-!-Wacko1976 is "The Chat Cool People Use" on #openttd #/r/openttd
03:33-!-Samu [] has joined #openttd
03:33-!-Samu is "OFTC WebIRC Client" on #openttd
04:32-!-matt21347 [~matt21347@] has joined #openttd
04:32-!-matt21347 is "matt" on #openttd #debian-next #debian-welcome #debian
05:43-!-Wacko1976 [] has quit [Ping timeout: 480 seconds]
08:27-!-dvim [] has joined #openttd
08:27-!-dvim is "martynas" on #qemu #openttd #openjdk
08:54-!-snail_UES_ [] has joined #openttd
08:54-!-snail_UES_ is "Jacopo Coletto" on #openttd
08:55-!-matt21347 [~matt21347@] has quit [Remote host closed the connection]
08:55-!-matt21347 [~matt21347@] has joined #openttd
08:55-!-matt21347 is "matt" on #debian #debian-welcome #debian-next #openttd
09:04-!-dvim [] has quit [Read error: Connection reset by peer]
09:07-!-dvim [] has joined #openttd
09:07-!-dvim is "martynas" on #openttd #openjdk
09:09-!-snail_UES_ [] has quit [Quit: snail_UES_]
09:29-!-GroovyNoodle [] has joined #openttd
09:29-!-GroovyNoodle is "" on #openttd
09:41-!-Flygon_ [] has quit [Read error: Connection reset by peer]
09:42<dihedral>hey ho
10:20-!-nielsm [] has joined #openttd
10:20-!-nielsm is "Niels Martin Hansen" on #openttd #tycoon
10:43-!-Alberth [] has joined #openttd
10:43-!-mode/#openttd [+o Alberth] by ChanServ
10:43-!-Alberth is "purple" on @#openttd
10:47<@planetmaker>aloha :)
10:48<@planetmaker> <-- anyone can answer my question? :D
10:48<@planetmaker>nml version detection is somewhat broken and I want it more pretty than calling git n times
10:49<Eddi|zuHause>i'm fairly sure there's a git module in python so you don't have to mess with command line
10:50-!-TheMask96 [] has quit [Ping timeout: 480 seconds]
10:50<@planetmaker>I know there is. Not sure I want to include that there, though
10:50<@planetmaker>(it's described in the link I provided there)
10:52<Eddi|zuHause>so then you have two ways to go, neither one of which you like?
10:52<Eddi|zuHause>not sure i can help with that :p
10:53-!-TheMask96 [] has joined #openttd
10:53-!-TheMask96 is "Martijn Zweistra" on #openttd @#altcontrol #openttd.notice
10:53<@planetmaker>:D maybe.
10:59<@Alberth>" One common usage of git rev-parse is to print the SHA1 hashes given a revision specifier. In addition, it has various options to format this output such as --short for printing a shorter unique SHA1 " ?
10:59<@Alberth>ie the second post in your link
11:02<@planetmaker>yeah, maybe that can be utilized to shorten the stuff a lot
11:02<@Alberth>except revision is less than useless to describe the content of the repo
11:02<@planetmaker>it's just a meaningless name.
11:03<@planetmaker>git show HEAD --pretty='%h8 "%at"' or so gives me hash and commit time
11:03<@Alberth>arguably, head^1 or so isn't much better
11:20<@planetmaker>perhaps sth like "git describe HEAD" is a good choice. In connection with the date or so
11:21<@planetmaker>"git describe --tags" I mean.
11:22-!-WWacko1976-work [] has quit [Ping timeout: 480 seconds]
11:24<@planetmaker>0.4.5-1-gc95db24a is actually a decent version for me. It contains the hash for identification. And the distance to the last tag
11:25<@planetmaker>And the last tag itself
11:27<@Alberth>yep, seems useful
11:35<Eddi|zuHause>planetmaker: while you're here, do you have any idea how devzone compiles work? because last time i tried it seemed to have problems "building" eddi-nml, and it couldn't use that to build CETS
11:36<Eddi|zuHause>(everything works locally)
11:37<Eddi|zuHause>(i never tried "building" nml, i just run it)
11:37<Eddi|zuHause>(and i never changed anything build-related)
11:38<@planetmaker>well, yes... did it have problems with eddi-nml itself? Or with using it on a project?
11:39-!-HerzogDeXtEr [] has joined #openttd
11:39-!-HerzogDeXtEr is "purple" on #openttd
11:39<Eddi|zuHause>both, i think
11:39<Eddi|zuHause>but i don't know whether one depended on the other or they are unrelated problems
11:40<@Alberth>the former probably defeats the latter :)
11:42<Eddi|zuHause>in CETS: 13:12:49 nmlc: error: no such option: --verbosity ( seems to suggest it runs an outdated version of nmlc
11:43<Eddi|zuHause>and i can't make any sense of ... it tries to run some pythonw.exe stuff to make an archive or something
11:49<@planetmaker> <-- @Alberth, what do you think?
11:50<@planetmaker>Eddi|zuHause, hm, yes, it tries to build a windoze binary... that's where the pythonw comes from
11:50<@planetmaker>all nicely using a python3.2 or so started in a somewhat ancient wine
11:50<Eddi|zuHause>planetmaker: but there should be no difference to regular nml in that regard
11:51<@Alberth>shouldn't there a abspath in line 51 ?
11:52<@Alberth>line 62, do line counting in python
11:53<@planetmaker>not as far as I know. It's verbatim copy from the existing version detection in line 51...55
11:53<Eddi|zuHause>"# we want to return to where we were. So save the old path" <-- that comment doesn't match any surrounding code
11:53<@Alberth>version.split('-')) == 1 "-" not in version right?
11:54<@planetmaker>Eddi|zuHause, also correct. Bitrot also in the hg version it seems
11:55-!-sla_ro|master [] has joined #openttd
11:55-!-sla_ro|master is "slamaster" on #sla #openttd
11:56-!-dvim [] has quit [Ping timeout: 480 seconds]
11:56<@Alberth>"-" is pretty fragile, probably, for branch names like "branch-2" or "new-feature"
11:57<Eddi|zuHause>but that would just add more split segments?
11:57-!-Oroburos [] has joined #openttd
11:57-!-Oroburos is "realname" on #biz #redditprivacy #openttd
11:59<Eddi|zuHause>so effectively that means tags containing "-" don't count as "release"?
12:00<@planetmaker>hm, yes
12:04<@Alberth>you don't assign version there?
12:04<@Alberth>except for ''
12:05<@planetmaker>line 78 and 80
12:05<Eddi|zuHause>you only assign version_list before line 68
12:05<Eddi|zuHause>but try to use version there
12:06<@Alberth>test is at 68
12:06<@planetmaker>oh, yes
12:06<Eddi|zuHause>version_list in line 56
12:07<Eddi|zuHause>how does "git describe" work? i can't get useful output on openttd trunk
12:07<Eddi|zuHause>> git describe
12:07<Eddi|zuHause>fatal: Keine Tags können 'fbfa4eb5920b10f57b9fc72730206f4a05c3638d' beschreiben.
12:07<Eddi|zuHause>Versuchen Sie --always oder erstellen Sie einige Tags.
12:07<@Alberth>you didn't check out a branch?
12:08<Eddi|zuHause>says i'm on branch master
12:09<@planetmaker>checking for a release probably can also be done by checking for "-g" in position -10:-9 in the version string. That's more reliable and present when it's not a tag as it's part of the hash
12:10<@planetmaker>Eddi|zuHause, git describe --tags
12:10<Eddi|zuHause>planetmaker: same
12:10<@planetmaker>might not work w/o tags
12:10<Eddi|zuHause>does openttd trunk not contain any tags
12:11<@planetmaker>bad :)
12:11<@planetmaker>thus needs different version detection
12:11<Eddi|zuHause>well, probably only in some branches
12:12<@Alberth>it's a converted svn repo, eh :)
12:12<Eddi|zuHause>well "git tag" is definitely not empty :p
12:12<@Alberth>git tag should tell you
12:13<@Alberth>no tag at or above fbfa4eb5920b10f57b9fc72730206f4a05c3638d probably
12:13<@planetmaker>no tag in its history
12:13<@planetmaker>all tags are in branches which are not ancestors
12:14<@planetmaker>so it's a good question... do we want such versioning in NML :)
12:15<@planetmaker>would mean to tag a new release for a branch. and to make a new commit on master which then is called "0.6.0-dev" or so (if you just made the 0.5.0 tag on master which then branches off)
12:15<Eddi|zuHause>seems prone to break once you have diverging and re-merging through pull requests and stuff
12:16-!-Supercheese [~Superchee@] has joined #openttd
12:16-!-Supercheese is "Supercheese" on #openttd
12:39<@planetmaker>that needs research :)
12:44<Eddi|zuHause>planetmaker: think your version patch could benefit from setting "LC_ALL=C"?
12:55-!-Thedarkb-T60 [] has joined #openttd
12:55-!-Thedarkb-T60 is "realname" on #openttd #oolite
12:58<@planetmaker>Eddi|zuHause, that as well, yes!
13:00<@planetmaker>thank you you two for your valuable inputs. I'll need to rewrite and re-think some parts... and learn more git :)
13:11-!-Wacko1976 [] has joined #openttd
13:11-!-Wacko1976 is "The Chat Cool People Use" on #openttd #/r/openttd
13:12-!-Thedarkb-T60 [] has quit [Ping timeout: 480 seconds]
13:13-!-Pikka [~Albert@] has quit [Ping timeout: 480 seconds]
13:14-!-Progman [] has joined #openttd
13:14-!-Progman is "Peter Henschel" on #openttdcoop #openttd
13:20-!-Wolf01 [] has joined #openttd
13:20-!-Wolf01 is "Wolf01" on #openttd
13:32-!-andythenorth [] has joined #openttd
13:32-!-andythenorth is "andythenorth" on #openttd
13:33-!-dvim [] has joined #openttd
13:33-!-dvim is "martynas" on #openttd #openjdk
13:36-!-Thedarkb-T60 [] has joined #openttd
13:36-!-Thedarkb-T60 is "realname" on #openttd #oolite
14:00-!-frosch123 [] has joined #openttd
14:00-!-frosch123 is "frosch" on #openttdcoop.devzone #openttd
14:03<frosch123>planetmaker: you could also copy ottd's
14:04<frosch123>it does branches, tags, and stuff
14:05-!-HerzogDeXtEr1 [] has joined #openttd
14:05-!-HerzogDeXtEr1 is "purple" on #openttd
14:11-!-HerzogDeXtEr [] has quit [Ping timeout: 480 seconds]
14:12<@planetmaker>frosch123, hm, that actually might be the simpliest idea :)
14:14<andythenorth>I stopped tanks
14:14<andythenorth>for a bit
14:15<nielsm>speaking of, isn't there a make target to just produce the "static" version data file?
14:15<frosch123>planetmaker: example output: 20181011-pr6939-g2e16bfddM 20181011 2 2e16bfdd936a5d6c73b8ced18c0f8d8d30751ea5
14:15<andythenorth>Wolf01: I am working on this a bit
14:15<andythenorth>it's a lot more done
14:15<nielsm>I don't have any git tools available in my windows build env, but I have a wsl bash open constantly
14:15<nielsm>and basically need to run manually
14:17<frosch123>nielsm: i think that magic is baked into the compile farm
14:17<frosch123>possibly the old compile farm only :p
14:17<@planetmaker>apropos... old CF. Old code: is there anywhere the stuff which is not OpenTTD, but the extras like master server, osie and alike?
14:18<frosch123>it's still in svn
14:18-!-HerzogDeXtEr1 [] has quit [Ping timeout: 480 seconds]
14:18<@planetmaker>which is... not exactly accessible :)
14:18<frosch123>and on some trello :)
14:18<frosch123>planetmaker: svn://
14:18<frosch123>or similar
14:19<@planetmaker>oho. I didn't know that URL
14:19<andythenorth>there is also the list
14:20<andythenorth>and I linked the trello list
14:21<andythenorth>Wolf01: more done
14:24<Wolf01>Nice, it's big
14:28<andythenorth>can anyone repro that this errors?
14:28<andythenorth>I'm logged in on the coop account
14:37-!-Thedarkb-T60 [] has quit [Read error: No route to host]
14:37-!-Thedarkb-T60 [] has joined #openttd
14:37-!-Thedarkb-T60 is "realname" on #openttd #oolite
14:40-!-gelignite [] has joined #openttd
14:40-!-gelignite is "gelignite" on #openttd
14:53-!-glx [] has joined #openttd
14:53-!-mode/#openttd [+v glx] by ChanServ
14:53-!-glx is "Loïc GUILLOUX" on @#opendune @#openttd.noai #openttd.notice +#openttd
14:58<@planetmaker>hmpf... :|
14:59<@planetmaker>$ git -C /home/planetmaker/ottd/grfdev/nml update-index --refresh -q
14:59<@planetmaker>nml/ needs update
14:59<@planetmaker>(py36) planetmaker@aeolusmaximus-vortex:~$ echo $?
15:00<@planetmaker>so why is that command there in, when it actually may fail and not do a thing (in OpenTTD it misses the -q flag which lets it continue anyway)
15:00<@planetmaker>but even then... it returns a non-0 exit status
15:14-!-Wolf01 is now known as Guest593
15:14-!-Wolf01 [] has joined #openttd
15:14-!-Wolf01 is "Wolf01" on #openttd
15:20-!-Guest593 [] has quit [Ping timeout: 480 seconds]
15:25-!-Alberth [] has left #openttd []
15:38-!-Stimrol [] has quit [Read error: Connection reset by peer]
15:41-!-Stimrol [] has joined #openttd
15:41-!-Stimrol is "Stimrol" on #openttd
15:55-!-dvim [] has quit [Ping timeout: 480 seconds]
16:13-!-gelignite [] has quit [Quit: Good fight, good night!]
16:14-!-sla_ro|master [] has quit []
16:15<@planetmaker> should be a version which now does the same thing as OpenTTD's
16:16-!-iSoSyS [~iSoSyS@2001:8a0:e946:8400:25a1:fc1:4137:d943] has joined #openttd
16:16-!-iSoSyS is "Miguel" on #/r/openttd #openttd
16:19<frosch123>get_child_output("git -C {} update-index --refresh".format(path).split(), env=env) <- ["git", "-C", path, "update-index", "--refresh"] is easier imo
16:19<frosch123>it also works with spaces in paths
16:22<frosch123>i think instead of the try-except you can also pass "check=False" to subprocess.
16:22<frosch123>but not sure
16:24<frosch123>hmm, i did not finish the nml docker thing, i guess
16:27<@planetmaker>hm, right, spaces in paths :|
16:27<@planetmaker>I didn't like to write every word quoted :D
16:32<@planetmaker>try...except is ok for python, I think
16:40<@planetmaker> has the check=... argument. But that's python 3.5+. In my understanding subprocess.check_output is python 3.1+
16:40<@planetmaker>But that lacks the check argument
16:41<frosch123>oh, new stuff, never mnid then :)
16:42<@planetmaker>actually, my use of the subprocess.DEVNULL constant raises the minimum requirement to 3.3 (instead of 3.2 which is is currently, IIRC)
16:43<@planetmaker>I might want to circumvent using that constant, too
16:51<@planetmaker>gotta play with pull requests, thus created one:
16:57<frosch123>planetmaker: you did the default mistake :)
16:58<frosch123>never create a pr from your "master"
16:58-!-dvim [] has joined #openttd
16:58-!-dvim is "martynas" on #openttd #openjdk
16:58<frosch123>after merge openttd's master will differ from your master, so you have to do some git magic to repoint your master to the real one
16:59<frosch123>with github: do everything in a custom branch, never commit to master
16:59<@planetmaker>ah, bah
16:59<@planetmaker>time to find out how to rectify this default mistake :)
17:00<frosch123>i guess it's easier after merge
17:00<frosch123>otherwise you have a pr to a no longer existing branch, which makes more trouble :p
17:01<@planetmaker>I'd create a new PR for that sake
17:03-!-nielsm [] has quit [Ping timeout: 480 seconds]
17:04<andythenorth>branches for all :)
17:04<andythenorth>no charge
17:04<frosch123>did your fix your hg checkout? :p
17:06-!-blathijs [] has quit [Ping timeout: 480 seconds]
17:06<@planetmaker>my hg checkout?
17:06<frosch123>nah, i was just mocking andy
17:10<andythenorth>I deleted it and cloned again
17:11<andythenorth>never use hg branches
17:11<andythenorth>never use hg up
17:12<@planetmaker>nothing wrong with either. But don't use *named* hg branches liberally
17:13<andythenorth>just for info (I did check the docs)
17:13<andythenorth>the correct way to checkout a tag is 'hg up [tag name]'?
17:14<@planetmaker>but it doesn't mean you end-up with an unmodified version, if you had a modified one
17:14<@planetmaker>add a hg revert --all before or after you do it
17:15<@planetmaker>(updating doesn't undo any changes you made to the previous version - it simply merges them to the new one)
17:15<andythenorth>in my experience, if there are uncommitted local changes, hg up [tag] fails with an unrecoverable merge
17:15<andythenorth>which I did fix the first time by editing things in .hg
17:15<andythenorth>but now I just delete the repo
17:15-!-matt21347 [~matt21347@] has quit [Quit: Leaving]
17:16-!-Wormnest [~Wormnest@] has joined #openttd
17:16-!-Wormnest is "Wormnest" on #openttd
17:16<andythenorth>yeah, that's what catches me out
17:16<andythenorth>I expect to move to a place on the tree, not merge
17:17<andythenorth>one day, git for newgrfs :P
17:18<andythenorth>but no editing history, no interactive rebase, and no force push :P
17:18*andythenorth has simple needs ?P
17:21<andythenorth>also bed
17:21-!-andythenorth [] has left #openttd []
17:21<@planetmaker>never ever edit "things" in .hg...
17:22-!-Progman [] has quit [Remote host closed the connection]
17:31-!-blathijs [] has joined #openttd
17:31-!-blathijs is "Matthijs Kooijman" on #openttd
17:39-!-frosch123 [] has quit [Quit: be yourself, except: if you have the opportunity to be a unicorn, then be a unicorn]
18:03-!-Wolf01 [] has quit [Quit: Once again the world is quick to bury me.]
18:13-!-Thedarkb-T60 [] has quit [Ping timeout: 480 seconds]
18:14-!-GroovyNoodle [] has quit [Quit: Leaving]
18:16-!-Lejving [] has quit [Ping timeout: 480 seconds]
18:35<Eddi|zuHause><andythenorth> in my experience, if there are uncommitted local changes, hg up [tag] fails with an unrecoverable merge <-- i would have assumed one could simply abort a merge... but why would it start a merge in the first place? merge with what?
18:35<@planetmaker>merge might be an unconvenient term here. hg simply keeps changes in the working dir, no matter what revision you changeout
18:36<Eddi|zuHause>so a revert --all would have done that...
18:36<@planetmaker>yet with active changes, when updating, the algorithm employed is that of a merge. Yet... there is no merge performed in any terms of VCS
18:36<@planetmaker>a hg revert --all will always rescue you
18:36<Eddi|zuHause>i just don't understand andy, then...
18:36<@planetmaker>if necessary update after that anew. And done
18:37<Eddi|zuHause>i have successfully employed hg branches an never had any problems...
18:37-!-Thedarkb-T60 [] has joined #openttd
18:37-!-Thedarkb-T60 is "realname" on #openttd #oolite
18:37-!-Wacko1976 [] has quit [Ping timeout: 480 seconds]
18:38<@planetmaker>I've done A LOT of things in hg repos... and never had to throw away my repo just for the sake of updating accross branches or alike
18:46-!-Wormnest [~Wormnest@] has quit [Quit: Leaving]
18:52-!-iSoSyS [~iSoSyS@2001:8a0:e946:8400:25a1:fc1:4137:d943] has quit [Remote host closed the connection]
19:11-!-Oroburos [] has quit [Quit: Leaving]
19:14-!-peter1138 [] has quit [Remote host closed the connection]
19:14-!-peter1138 [] has joined #openttd
19:14-!-mode/#openttd [+o peter1138] by ChanServ
19:14-!-peter1138 is "Peter Nelson" on #debian #tycoon @#openttd
19:24-!-Flygon [] has joined #openttd
19:24-!-Flygon is "Flygon" on #openttd
19:45-!-Lejving [] has joined #openttd
19:45-!-Lejving is "realname" on #openttd #/r/openttd #factoriocoop #mashinky #openttdcoop #openttdcoop.stable @#openttdcoop.pz
19:51-!-sim-al2 [] has quit [Ping timeout: 480 seconds]
19:54-!-Samu [] has quit [Quit: Page closed]
20:09-!-Pikka [~Albert@] has joined #openttd
20:09-!-Pikka is "realname" on #openttd
20:31-!-Thedarkb-T60 [] has quit [Ping timeout: 480 seconds]
20:58-!-mindlesstux [] has joined #openttd
20:58-!-mindlesstux is "Tux" on #openttd #ipv6
20:58-!-Supercheese [~Superchee@] has quit [Quit: Valete omnes]
21:07-!-over_mind is "over_mind" on #debian
21:07-!-over_mind [b9dc6515@] has joined #openttd
21:21-!-Supercheese [] has joined #openttd
21:21-!-Supercheese is "Supercheese" on #openttd
21:22-!-WWacko1976-work [] has joined #openttd
21:22-!-WWacko1976-work is "YO!" on #openttd #/r/openttd
21:29-!-wodencafe [] has quit [Quit: Konversation terminated!]
21:30-!-tokai [] has joined #openttd
21:30-!-mode/#openttd [+v tokai] by ChanServ
21:30-!-tokai is "Christian Rosentreter" on +#openttd
21:33-!-dvim [] has quit [Ping timeout: 480 seconds]
21:35-!-Supercheese [] has quit [Read error: Connection reset by peer]
21:35-!-Supercheese [] has joined #openttd
21:35-!-Supercheese is "Supercheese" on #openttd
21:37-!-tokai|noir [] has quit [Ping timeout: 480 seconds]
21:52-!-over__mind [6dc98564@] has joined #openttd
21:52-!-over__mind is "over_mind" on #Qubes_OS #publiclab #ovirt #openttd #oftc #debian-next #debian ##politics
21:53-!-over_mind [b9dc6515@] has quit []
21:58-!-snail_UES_ [] has joined #openttd
21:58-!-snail_UES_ is "Jacopo Coletto" on #openttd
22:26-!-dvim [] has joined #openttd
22:26-!-dvim is "martynas" on #openttd #openjdk
22:30-!-over__mind is now known as over_mind
22:49-!-glx [] has quit [Quit: Bye]
23:07-!-haudrauf [] has quit [Ping timeout: 480 seconds]
23:08-!-haudrauf [] has joined #openttd
23:08-!-haudrauf is "Haudrauf" on #openttd #frickelplatz @#ffod @#ffnord @#ffki @#ffhl @#ffhh @#fffl #cryptoparty @#ccchh
23:14-!-KouDy [] has quit [Read error: Connection reset by peer]
23:16-!-KouDy [] has joined #openttd
23:16-!-KouDy is "KouDy" on #openttd
23:25-!-KouDy [] has quit [Remote host closed the connection]
23:27-!-KouDy [] has joined #openttd
23:27-!-KouDy is "KouDy" on #openttd
23:54-!-Pikka [~Albert@] has quit [Ping timeout: 480 seconds]
---Logclosed Tue Oct 16 00:00:27 2018