00:02:56 bash script to auto transfer savegames that are too old, any critique? https://pastebin.com/PQnrAT7p 00:03:16 alexjurkiewicz: that's because yermak didn't clear his more for 6 minutes and I wasn't payingn attention to the ttyrec player to move things along 00:03:19 oops 00:04:29 ebering, also PF didn't actually die in your video. 00:07:14 he did, but in the opposite direction of you cleared his more immediately 00:07:22 the ttyrec doesn't have the frame with 0 hp 00:09:26 good awk floraline, can't you use `awk '{print $7}'`? 00:15:31 thanks, i googled it 00:15:54 remove-trunks -v returns a huge list of players for each version after the 6th field on each line 00:16:32 so i want fields 7 thru the end of the line 00:17:00 that monstrosity is what stack overflow told me to do 00:20:41 ah, every field, yours is good then 00:21:17 cool :) thanks for taking a look 00:22:16 i'm going to put this in my cron job, it already cleans up old trunk versions that have no players but now it can make them have no players 00:34:29 03PleasingFungus02 07* 0.25-a0-505-gae45692: Split tremorstone explosions in half (Wise Wolf) 10(33 minutes ago, 2 files, 7+ 3-) 13https://github.com/crawl/crawl/commit/ae4569222e44 00:48:00 Unstable branch on crawl.kelbi.org updated to: 0.25-a0-505-gae4569222e (34) 00:48:43 !lg * thisweek trunk kpath~~tremor|kaux~~tremor 00:48:44 No games for * (thisweek trunk kpath~~tremor|kaux~~tremor). 00:48:46 hmm 01:39:19 New branch created: pull/1286 (1 commit) 13https://github.com/crawl/crawl/pull/1286 01:39:19 Branch pull/1286 updated to be equal with launching: 13https://github.com/crawl/crawl/pull/1286 01:55:40 Gadrel (L17 DsHu) ASSERT(!invalid_monster(&mons)) in 'mon-death.cc' at line 1716 failed. (Snake:3) 02:10:28 Windows builds of master branch on crawl.develz.org updated to: 0.25-a0-505-gae45692 02:48:45 Monster database of master branch on crawl.develz.org updated to: 0.24-a0-443-g80245de 02:55:33 03gammafunk02 07* 0.25-a0-506-g3442025: Fix acquirement for rare cases (amalloy) 10(28 hours ago, 1 file, 47+ 29-) 13https://github.com/crawl/crawl/commit/344202577d11 02:55:33 03gammafunk02 07* 0.25-a0-507-gf66f01b: Fix unrand uniqueness for acquirement (AliasTheSpectator) 10(22 minutes ago, 1 file, 4+ 1-) 13https://github.com/crawl/crawl/commit/f66f01b915e3 03:23:16 Stable (0.23) branch on crawl.beRotato.org updated to: 0.23.1-92-g5aa5405 03:36:10 Stable (0.22) branch on crawl.beRotato.org updated to: 0.22.1-60-gb8697bf 03:55:11 Stable (0.18) branch on crawl.kelbi.org updated to: 0.18.1-51-g5e05c4d488 03:58:56 Stable (0.19) branch on crawl.beRotato.org updated to: 0.19.5-2-g88d7a0c 04:10:57 Stable (0.18) branch on crawl.beRotato.org updated to: 0.18.1-51-g5e05c4d 04:17:24 Stable (0.21) branch on crawl.kelbi.org updated to: 0.21.2-2-g132acf7e8d 04:24:30 Unstable branch on crawl.beRotato.org updated to: 0.25-a0-507-gf66f01b (34) 04:40:55 Stable (0.22) branch on crawl.kelbi.org updated to: 0.22.1-60-gb8697bfa38 04:47:40 -!- amalloy is now known as amalloy_ 05:05:21 Stable (0.23) branch on crawl.kelbi.org updated to: 0.23.1-92-g5aa5405057 05:29:18 Stable (0.24) branch on crawl.kelbi.org updated to: 0.24.0-124-g2791043dd8 05:37:42 Unstable branch on crawl.kelbi.org updated to: 0.25-a0-507-gf66f01b915 (34) 06:01:22 Fork (bcrawl) on crawl.kelbi.org updated to: 0.23-a0-2959-g901f94f422 06:23:53 Fork (gooncrawl) on crawl.kelbi.org updated to: 0.22-a0-974-g70ddf4710b 06:28:36 ... oh, Kelbell hasn't announced us yet because it's got to compile all the things 06:48:41 Fork (stoatsoup) on crawl.kelbi.org updated to: 0.22-s0-42-g5133f79d94 07:15:21 Fork (bloatcrawl2) on crawl.kelbi.org updated to: 0.25-a0-830-g81abc0230d 09:08:36 03kate-02 07* 0.25-a0-508-g3095992: Fix the skin of Zhor's inscription 10(86 minutes ago, 1 file, 1+ 1-) 13https://github.com/crawl/crawl/commit/3095992e705c 09:08:36 03kate-02 07* 0.25-a0-509-g1cd534d: Remove Bloodbane and merge its properties into Leech 10(7 days ago, 13 files, 12+ 48-) 13https://github.com/crawl/crawl/commit/1cd534d472d2 09:08:36 03kate-02 07* 0.25-a0-510-ge065f6d: Turn the boots of the Assassin into a hat 10(6 days ago, 12 files, 22+ 45-) 13https://github.com/crawl/crawl/commit/e065f6de7db8 09:12:29 the above does mean that there's no unrand boots left now, and a lot of unrand hats (6 hats plus etheric cage which is a helmet), so ideas for new boots unrands might be good 09:13:17 (and etheric cage seems like it could go, since MP regen is a standard amulet now and the contam multiplier isn't very notable with haste spell gone) 09:15:06 Uh it strikes me to ask if any of you dev types know Sequell is working but missing from ##crawl 09:33:40 Unstable branch on crawl.kelbi.org updated to: 0.25-a0-510-ge065f6de7d (34) 09:39:07 i did not - i think that's something greensnark would be able to fix? (not sure if anyone else also has access, apologies for the ping if so) 09:43:13 yeah I think that's probably snark, unless gammafunk can do something 09:43:57 alexjurkiewicz I added you to the test "serveradmins" team, once you accept can you poke around and see if it gave you any unexpected permissions? 09:45:03 should also make it possible for you to view the draft security advisory: https://github.com/crawl/crawl/security/advisories/GHSA-vjrj-g7gf-rgjv 10:35:41 advil: if you've got a chance could you update the csdc pin please? 11:22:41 Stable (0.23) branch on underhound.eu updated to: 0.23.1-92-g5aa5405057 14:45:14 -!- amalloy_ is now known as amalloy 15:58:50 advil: i can't edit the main repo and I can see the sec advisory 15:59:38 no edit permissions on any of the 5 most recently edited repos 16:10:56 ok, sounds as intended 16:11:22 I realized that the one team that did exist is "secret" so it probably looked like there were no teams at all before 16:11:57 floraline: do you want to be in the serveradmins team on github? Will use it for some light messaging and it will enable you to see things like this security advisory before it is made public 16:12:05 also johnstein ^ 16:12:18 (and I just added gammafunk without asking, sorry) 16:17:11 thx 17:08:58 advil: sure, thanks. 17:10:33 now we just have to remember that this team exists next time something like this security advisory comes up :-) 17:30:33 are the communications private? could be a good way to get communication with other server admins if we want to tell them something? I suppose the issues stopping that in the past aren't solved by this platform 18:22:00 Unstable branch on underhound.eu updated to: 0.25-a0-510-ge065f6de7d (34) 18:27:07 I noticed a minor mainline bug while cleaning up some of my own bugs. If Erica spawns with an artefact flaming scimitar it won't draw her normal sprite (drawing the version as if you'd cast Tukima's on her instead); this is because it checks weapon->brand instead of get_weapon_brand(*weapon). 18:27:33 Do you have Mantis access? 18:28:21 The same -was- true if an artefact dancing sword existed (Dith wouldn't know it was flaming back when he hated that). [And only as a reporter.] 18:28:55 Reporter's fine, it's just given the difficulties with Mantis access I try to offer to report bugs for people without. 18:29:56 gammafunk not sure; the security communications are private though. To test, I guess the question is whether someone logged into github but not part of the crawl org at all can see this page: https://github.com/orgs/crawl/teams/serveradmins 18:30:15 yes, a bug report in this channel goes into the void unless the right person looks at the right time 18:30:32 advil: 404. 18:30:32 which can sometimes happen, for sure 18:30:47 ah interesting 18:31:11 same for https://github.com/orgs/crawl/teams/developers ? 18:31:21 I think 404 is normal for pages one can't see on gh - seen it before - to avoid leaking information about URLs that exist 18:31:37 That's also a 404. 18:35:15 not that I anticipate a lot of sekret server admin discussions, I think the normal thing to do would be to @ mention the team in a public issue 18:40:16 If Erica spawns with an artefact scimitar it doesn't draw her normal unique weapon sprite. 13https://crawl.develz.org/mantis/view.php?id=12199 by bcadren 18:46:23 I had a lot of issues with artefacts due to using .brand and ->brand instead of get_weapon_brand(); in rooting them out I noticed a couple mainline ones too...that one and one with Dith's old fiery conduct (removed via removing that conduct; not via it actually being fixed). 18:47:12 Bcadren: Do you happen to remember where the Dith one was? We kept that conduct. 18:47:33 (actually I'll ask on Discord to keep it out of -dev) 18:54:57 advil: maybe it could be good to invite server admins to that group; I'm not sure if all of them are on github, though 18:55:27 I know dplusplus is, I assume TZer0 is, johnstein is, not really sure about the rest 19:59:40 ??sigmund 19:59:40 sigmund[1/4]: A yellow @ with a shiny scythe, a magic wand, and a necklace of player skulls. A notoriously lethal early-game unique human wizard. Spells: throw flame, confuse, invisibility, magic dart. 20:00:54 !learn edit Borgnjor's_Vile_Clutch[1] s/smite targeted constriction in a 3x3 area/constriction all along a 6-cell-long penetrating beam/ 20:00:54 Borgnjor's Vile Clutch[1/4]: Not actually a nasty purse used by necromancers but the actual and good name for the zombie hands player spell - L5 earth/necromancy, constriction all along a 6-cell-long penetrating beam. New in 0.21. 20:01:34 -!- TAS-2012v is now known as TAS_2012v 20:02:58 !learn set Borgnjor's_Vile_Clutch[4] Locks enemies in place and does ongoing damage. 20:02:59 Borgnjor's Vile Clutch[4/4]: Locks enemies in place and does ongoing damage. 20:11:08 why is melee_attack::calc_your_to_hit_unarmed() it's own thing? I've long since merged it with the general to-hit function and just noticed it was a thing in mainline while cherrypicking 20:15:05 Psymania: we want learndb to document both stable and trunk, so more like this 20:16:55 this might be too long, we'll see 20:16:56 !learn set borgnjor's_vile_clutch[1] Not actually a nasty purse used by necromancers but the actual and good name for the zombie hands player spell - L5 earth/necromancy that gives smite-targeted constriction in a 3x3 area. In 0.25, targets along a 6-cell-long penetrating beam. 20:16:56 Borgnjor's Vile Clutch[1/4]: Not actually a nasty purse used by necromancers but the actual and good name for the zombie hands player spell - L5 earth/necromancy that gives smite-targeted constriction in a 3x3 area. In 0.25, targets along a 6-cell-long penetrating beam. 20:17:19 the first bit is just a bit of learndb flavor, added by myself; as the spell's creator, I'm given some license 20:19:00 okay :) 20:19:03 Psymania: ... another learndb project is to find entries like that which made sense at the time but now say "Fred's Mighty Zap[1/1] Mights everything in LOS, friend or foe, and blah blah several sentences. (In 0.8, changed to do something entirely.)" 20:20:06 bvc[4] is a little awkward, so I'd trim some of [1] and emphasize that info; you could argue that constriction is clear enough about "locks enemies in place and does ongoing damage", but it'd be ok to make it explicit 20:20:10 maybe more like this 20:21:29 this might still be too long, here goes 20:21:59 !learn set borgnjor's_vile_clutch[1] Not a nasty purse used by necromancers but the real name for zombie hands - L5 earth/necromancy spell that gives smite-targeted constriction in a 3x3 area, locking monsters in place and damaging over time. In 0.25, targets along a 6-cell-long penetrating beam. 20:21:59 Borgnjor's Vile Clutch[1/4]: Not a nasty purse used by necromancers but the real name for zombie hands - L5 earth/necromancy spell that gives smite-targeted constriction in a 3x3 area, locking monsters in place and damaging over time. In 0.25, targets along a 6-cell-long penetrating beam. 20:22:08 !learn del bvc[4] 20:22:08 bvc has only 1 entry. 20:22:17 !learn del borgnjor's_vile_clutch[4 20:22:17 Deleted Borgnjor's Vile Clutch[4/4]: Locks enemies in place and does ongoing damage. 21:17:04 Aidanh you're usually my favorite dev; but this commit (Remove/mark unused parameters) is being a pain to merge for my fork...it's been like an hour of it now; cases where I'd added additional parameters; calls made bad in other areas now covered by the commit, cases where I actually used the parameter that was unused by mainline, et. all. (Trying 21:17:04 to get forward to present to get that vulnerability patch since cherrypicking just that one directly doesn't work). 21:18:17 advil: the other option btw is to create a pinned issue called "subsscribe to this issue for security updates" 21:46:38 Bcadren and I have managed to get the fix to apply to bcadrencrawl 23:01:27 aidanh I reviewed your commit 768f60da. You need to block load too, not just loadstring. I will put a link to a new pastebin exploit in the dev-s channel. I don't think you need to worry about loadfile because I don't know of a way to get a file on to a remove server filesystem. But you would know better than me on that one. 23:29:07 -!- Euph0ria is now known as OICU812 23:57:34 found another mainline bug. There's a function in item-name.cc that doesn't short-circuit properly due to a missing parentheses...it was causing an assert failure for me because I was getting more aggressive with the is_artefact() asserts than mainline.