00:09:43 -!- amalloy_ is now known as amalloy 01:50:33 -!- Euphoria is now known as TabMasher 02:09:02 Windows builds of master branch on crawl.develz.org updated to: 0.25-a0-399-gcf1075a 02:48:17 Monster database of master branch on crawl.develz.org updated to: 0.24-a0-443-g80245de 03:22:45 Unstable branch on crawl.beRotato.org updated to: 0.25-a0-399-gcf1075a (34) 03:55:15 Fork (bcadrencrawl) on crawl.kelbi.org updated to: 0.22.1-1943-g8311b7e2db 04:19:14 Fork (gooncrawl) on crawl.kelbi.org updated to: 0.22-a0-899-ga83ac8c4e5 07:17:00 -!- amalloy is now known as amalloy_ 11:23:15 Stable (0.23) branch on underhound.eu updated to: 0.23.1-91-gf373564dc4 11:45:04 -!- amalloy_ is now known as amalloy 12:26:11 Sinbi (L27 GrFi) ERROR: range check error (27000 / 2000) (Zig:12) 13:01:29 03ebering02 07* 0.25-a0-400-gd3051d0: Display drawn cards immediately when re-entering Stack Five 10(2 hours ago, 1 file, 14+ 3-) 13https://github.com/crawl/crawl/commit/d3051d076b68 13:01:29 03ebering02 07* 0.25-a0-401-g35c6a73: Re-enter Stack Five ordering after a HUP 10(57 minutes ago, 1 file, 7+ 3-) 13https://github.com/crawl/crawl/commit/35c6a73c01f7 13:01:29 03ebering02 07* 0.25-a0-402-ga0cd47f: Display appropriate capitals for easy_confirm = none 10(77 seconds ago, 2 files, 14+ 6-) 13https://github.com/crawl/crawl/commit/a0cd47fd9c55 13:03:55 03ebering02 07* 0.25-a0-403-g49d089a: Re-enter acquirement selection after a HUP 10(2 minutes ago, 0 files, 0+ 0-) 13https://github.com/crawl/crawl/commit/49d089a364fd 13:17:09 -!- jfcaron_ is now known as jfcaron 13:19:52 Unstable branch on crawl.kelbi.org updated to: 0.25-a0-403-g49d089a364 (34) 17:14:49 wow, how do you do a no-diff commit? 17:19:47 alexjurkiewicz, git commit --allow-empty, or from certain commit rewrites 17:20:10 not as useful in CVS/SVN though, since they aren't tied to particular file(s) 17:20:17 as useful *as 17:20:19 wow 17:20:27 always more to learn with git 17:22:15 why would you make a commit that does nothing? 17:26:47 floraline: to help a future person using git log --grep 17:27:14 commit 402 had two changes in it but the message only mentioned one because I goofed with git add --patch 17:27:39 ah, cool 17:28:44 i'm awful with commit messages so that didn't even occur to me 17:40:49 in my personal projects I'm pretty garbo 17:41:00 but I've done enough crawl archaeology that I try to keep my crawl ones neat 17:47:33 git commit -a --amend -m "Version 6 (final) of PR. Hopefully works this time." 18:23:01 Unstable branch on underhound.eu updated to: 0.25-a0-403-g49d089a364 (34) 18:29:53 I have an alias set up for git commit -a --amend --no-edit 18:30:41 in undergrad a classmate had the alias yolo for "git commit -a -m 'DEAL WITH IT' && git push --force" 18:31:08 true yolo needs a git add -A at the start?? 18:33:49 alexjurkiewicz: have you tried git add -p? i find it's a really nice way to mentally review what i've done before i commit it 18:36:59 yes, I usually use git add -A -p. Crawl has made me pretty anal about not mixing two things in one commit (but I'm still bad at mixing unrelated cleanup commits into PRs) 18:37:52 I also learnt git commit --fixup off ebering's positional magic PR and use it constnatly now 18:40:33 you know about rebase --autosquash --interactive also right 18:42:56 rebase.autoSquash is true by default in my git, but yes also a big fan of rebase -i 19:00:17 gammafunk: about webtiles changing the crawl binary path live. You suggested instead just updating a symlink. For some reason I found this impossible, so I instead had to update the binary path each build and reload webtiles config 19:01:02 gammafunk: I tried to replicate it today and everything seemed to go fine though.... so you may be right. But I would still like to reload games config dynamically at least to support adding new games 19:02:13 I'll add some caveats to the function which say changing existing games is a "here be dragons" sort of untested thing, even if it's theoretically possible. I could cause some errors if you do things like update the crawl binary to a broken build and then try and spectate someone currently playing. So there are issues 22:16:12 -!- amalloy is now known as amalloy_ 23:36:58 alexjurkiewicz: yeah, adding new game configs dynamically is good, and it's fine if we do have a warning about existing games, but broken builds were not actually my concern 23:37:29 alexjurkiewicz: my concern was the other paramaters like various other paths that are passed to crawl upon startup 23:38:07 if an admin think they can update those paths in the config and then send that USR1 signal to reload the config in webtiles yet *not* restart those old crawl processes 23:38:23 those currently running games would crash or just do weird things 23:38:51 if you gave a bad binary path in an update that shouldn't cause too much harm other than people not being able to start a new game, I guess 23:39:16 if you update the config path of eg ttyrec path, or rcfiles path, it won't affect currently running games 23:39:32 yes, but if those old paths are now invalid, I'm saying 23:39:47 ah right, so you update ttyrec path from /old to /new and then rm -rf /old 23:40:07 right, stuff that should be common sense, but we should maybe add some simple warnings about that kind of thing 23:40:22 good idea