00:04:23 -!- Tiobot is now known as Guest47509 00:08:01 His Crawl website has some...equally...interesting remarks 00:08:05 "DEWz Kiku -> ELair -> Fedmid -> JSlab -> Zin Sif 00:08:06 Drow sorceress, barely dressed, slinging spells without stress. Briefly plays a nun, I guess? Decides it's more fun wearing less." 00:09:27 ebering: done (I think) 00:22:41 -!- amalloy_ is now known as amalloy 00:25:49 dude's stuff is fucking weird 00:26:15 especially when you get into the aforementioned non-Crawl-related stuff 00:27:54 well we like to keep gossip about such stuff out of ##crawl-dev 00:28:02 it can end up feed the trolls, in the end 00:28:17 *feeding 00:33:10 understood 00:39:19 03Alex Jurkiewicz02 07https://github.com/crawl/crawl/pull/1281 * 0.25-a0-505-g7896f04: fixup! fixup! fixup! fixup! fixup! Exclude catch2-tests/ from coverage data 10(25 seconds ago, 1 file, 0+ 1-) 13https://github.com/crawl/crawl/commit/7896f04af3ae 00:44:42 03Alex Jurkiewicz02 07https://github.com/crawl/crawl/pull/1281 * 0.25-a0-506-g6cf4f61: fixup! fixup! fixup! fixup! fixup! fixup! Exclude catch2-tests/ from coverage data 10(4 minutes ago, 1 file, 4+ 4-) 13https://github.com/crawl/crawl/commit/6cf4f61be59d 01:14:31 New branch created: camo (1 commit) 13https://github.com/crawl/crawl/tree/camo 01:14:31 03PleasingFungus02 07[camo] * 0.25-a0-503-g2ee4643: Potion of agility -> camouflage 10(88 minutes ago, 28 files, 89+ 57-) 13https://github.com/crawl/crawl/commit/2ee4643df0c4 02:10:09 Windows builds of master branch on crawl.develz.org updated to: 0.25-a0-502-g6516661 02:22:06 🙈 02:47:46 The build passed. (camo - 2ee4643 #12666 : Nicholas Feinberg): https://travis-ci.org/crawl/crawl/builds/649303708 02:48:20 Monster database of master branch on crawl.develz.org updated to: 0.24-a0-443-g80245de 03:13:17 03Alex Jurkiewicz02 07https://github.com/crawl/crawl/pull/1281 * 0.25-a0-507-g7f526eb: fixup! fixup! fixup! Display ccache config per-build 10(3 minutes ago, 1 file, 4+ 4-) 13https://github.com/crawl/crawl/commit/7f526eb61630 03:23:19 Unstable branch on crawl.beRotato.org updated to: 0.25-a0-502-g6516661 (34) 03:26:43 -!- amalloy is now known as amalloy_ 03:31:37 Fork (bcadrencrawl) on crawl.kelbi.org updated to: 0.22.1-2003-gc51002e74b 03:47:25 hey devs, is there a secret CODECOV_TOKEN in here? https://github.com/crawl/crawl/settings/secrets 03:51:03 03Alex Jurkiewicz02 07https://github.com/crawl/crawl/pull/1281 * 0.25-a0-501-g0adb6cb: Always upload coverage data to Codecov 10(2 days ago, 1 file, 4+ 0-) 13https://github.com/crawl/crawl/commit/0adb6cbce32e 03:51:03 03Alex Jurkiewicz02 07https://github.com/crawl/crawl/pull/1281 * 0.25-a0-502-g36a2805: Disable Codecov pull request comments 10(13 hours ago, 1 file, 1+ 0-) 13https://github.com/crawl/crawl/commit/36a2805746a2 03:51:03 03Alex Jurkiewicz02 07https://github.com/crawl/crawl/pull/1281 * 0.25-a0-503-g3350396: Exclude catch2-tests/ from coverage data 10(13 hours ago, 1 file, 9+ 0-) 13https://github.com/crawl/crawl/commit/335039619637 03:51:03 03Alex Jurkiewicz02 07https://github.com/crawl/crawl/pull/1281 * 0.25-a0-504-g8a0328b: s/coveralls/codecov/ 10(12 hours ago, 1 file, 2+ 2-) 13https://github.com/crawl/crawl/commit/8a0328bd8e58 03:51:03 03Alex Jurkiewicz02 07https://github.com/crawl/crawl/pull/1281 * 0.25-a0-505-g08c0204: Display ccache config per-build 10(22 hours ago, 1 file, 34+ 0-) 13https://github.com/crawl/crawl/commit/08c0204d2232 03:51:03 03Alex Jurkiewicz02 07https://github.com/crawl/crawl/pull/1281 * 0.25-a0-506-g36e3582: Add size limits to ccache 10(6 hours ago, 1 file, 41+ 10-) 13https://github.com/crawl/crawl/commit/36e3582beda8 03:56:26 03Alex Jurkiewicz02 07https://github.com/crawl/crawl/pull/1281 * 0.25-a0-506-ge0ca8af: Add size limits to ccache 10(6 hours ago, 1 file, 41+ 10-) 13https://github.com/crawl/crawl/commit/e0ca8af26dba 07:54:01 Did anyone else get an irc pm from our security bug friend 07:54:14 I am less than pleased with this characters approach 07:55:38 !tell dpmdpm if you don't have an irc bouncer to stay online, try e-maill (the crawl-ref-discuss mailing list, while not used much these days, still exists) 07:55:38 ebering: OK, I'll let dpmdpm know. 10:05:37 I am reworking Elyvilon. Because serving this god is powerful, but boring. I would like to know your opinion on this proposal. 10:05:43 https://github.com/crawl/crawl/pull/1279 10:36:06 ebering: I think they don't want to discuss it in a public forum 10:37:08 and yes, they seem to be going by some security process that is not really oriented towards all-volunteer low resource open source projects 10:37:32 I get that, but sending an irc pm and not leaving any method of reply and then signing off is not a helpful way to communicate 10:37:55 agreed 10:38:10 he did provide an email address over reddit modmail if anyone wants to email with him (I don't) 10:40:37 heh it's the same guy as this: https://nethack.org/security/CVE-2020-5209.html 10:41:28 actually all of these https://www.nethack.org/security/index.html 10:41:48 apparently the nethack devs were more excited about the process 10:43:26 03PleasingFungus02 07* 0.25-a0-503-ge281fa6: Value Tremorstones lower 10(8 minutes ago, 2 files, 8+ 2-) 13https://github.com/crawl/crawl/commit/e281fa673e8b 10:48:02 Unstable branch on crawl.kelbi.org updated to: 0.25-a0-503-ge281fa673e (34) 10:58:55 iirc, if someone pushes patches to each of the stone_soup-0.foo branches, that should be enough, right? or is the fetcher tag based? 11:01:12 I got a pm as well; i'd be more open towards communication if I had a clear idea of what this cve process actually is and what it entails 11:02:03 but I guess the cat is out of the bag to some extent, now that it's been discussed on crawl-dev, so I will take a look at patching, probably tomorrow 11:03:51 alexjurkiewicz: really depends on your tests, but unless you group all assertions at the end of each test, an assertion failure will cause reduced coverage 11:04:19 alexjurkiewicz: and it's not predictable/obvious, making it an unreliable metric, so usually I prefer not to bother; your tests should be passing anyway :) 11:05:22 advil: re namespacing, I expect you'd have a better grasp on the current state of name clashing and the best way to handle this 11:06:27 even with your federation scheme, I'd lean towards enforcing that accounts auto-created on not-yet-registered servers would use a shared name 11:21:01 !tell PleasingFungus I like the direction of !camo and was thinking it would be good to merge !might and !brill into !augmentation for reasons I'd be happy to elaborate on when I have a lil more time 11:21:02 ebering: OK, I'll let pleasingfungus know. 11:22:27 advil: also, any status update on py2019? I idly fiddled with tornado and independently discovered that removing all the io loop cruft made it work, before remembering your branch and realising you'd already done all that :) 11:23:23 well, it was "done" but not not tested in the wild 11:23:38 Stable (0.23) branch on underhound.eu updated to: 0.23.1-91-gf373564dc4 11:24:04 the intent was that it has no impact on existing servers, everything in there is py27 compatible for the versions that the servers are running 11:24:24 and the tornado versions 11:24:29 sure 11:24:59 this is where it's too bad that webtiles server is in the same repository 11:25:13 because it's kind of a pain to test on any server without just merging the branch 11:26:50 hmm 11:26:53 right, the way it's currently deployed, yeah 11:27:36 I was also thinking of adding a requirements.txt along with a pinned version file 11:27:54 so that new servers will auto-update dependencies as well 11:28:58 probably with pip-compile and pip-sync, since I'm most familiar with those, but other suggestions are ok too 11:33:44 woah, ok, alex's webtiles-changes already has one 11:34:43 I'm also not sure how py2019 interacts with alex's branch (that mostly developed while I was away) 11:36:05 probably straight-forward to solve except for the signal handling stuff, which you both changed iirc 11:36:17 s/solve/resolve/ 11:37:03 my changes were because some stuff was broken by changes in tornado versions, I think he added config reload? 11:37:37 yeah 11:37:37 so they probably impact the same code but should be orthogonal, I would merge mine first 11:38:22 otoh, his has a few weeks of testing; you could merge his first, and he might be willing to give py2019 a few weeks after that 11:38:44 that was in terms of the signal handling specifically 11:39:08 iirc you did the signal handling last, but if they reorder cleanly, why not 11:39:14 unless he made certain specific changes his will presumably still not be compatible with other tornado versions 11:39:27 doesn't look like it, just handled sigusr1 11:43:59 btw, I recall mentions of moving scoring off cao to a dedicated server? i'd be happy to contribute to that, e.g. infra costs 11:45:17 preferably in a way where any new infra setup is automated, so as not to add yet another server with a busy operator 11:46:29 and also to avoid the growing number of legacy compilers we support 11:47:44 what was the need for moving scoring off CAO? 11:50:51 i can't remember the reasons mentioned, but it'll ease the maintenance burden on cao, and make it easier to work on scoring, which aiui is in need of some serious sprucing up 12:35:16 I'm not sure there's a big need to move it off now that multiple people have access 12:35:44 for working on scoring, you want to do that locally anyways 12:35:53 think of cao as a production server in terms of pushing changes 12:36:05 it's not very hard to get going locally 12:38:40 -!- amalloy_ is now known as amalloy 12:38:42 the main issue was just that only noping people had the ability to restart it, and if that involved a full db reload it isn't entirely trivial to do 12:39:07 "noping" = "no-ping" 12:41:11 alexjurkiewicz: yes, there is such a secret CODECOV_TOKEN 13:17:47 with tremorstones being added it definitely does seem like a good time to remove a few other evokers - probably at least one summoning one (sack of spiders?) and at least one direct damage one (lamp of fire?), maybe fan of gales too? 13:19:29 does fan even do direct damage anymore? 13:20:03 i thought it just pushed dudes now, and if they collide with something they take damage 13:20:20 kate-: we should remove the water elementals from phial of floods imo 13:20:58 right yeah, i was meaning it's not in the direct damage or summoning categories re: fan but i don't think it's a super worthwhile effect to reserve an item slot for in every game 13:21:17 (agree w/removing fan) 13:22:35 actually; its probably (if you remove the summons) reasonable to turn phial into a one-time that stacks instead of an xp evoker 13:22:37 i feel like the phial is pretty boring without the elementals 13:23:24 i guess it knocks back a bit? 13:23:40 yeah, it knocks back and changes terrain 13:25:27 i guess currently it's just notable for being very strong since it deals a lot of damage and the elementals can silence things 13:26:19 I think if it applied a temporary engulf status to targets in place of the elementals it would be in a better place to become a one-shot 13:26:29 I'm not thinking super carefully at the moment though 13:26:43 that does sound better maybe, yeah 13:27:38 a premise that's motivating me is: evocations would be in a better place if the non-body-slot evocables were all consumable 13:27:54 instead of permanent inventory slot taxes 13:32:34 yeah, the way xp evokers work out does kinda mean eventually every game just trades a few inventory slots for reliable access to all those same effects, at least having them consumable means there can be more variety in how much access you have to the effects 14:20:21 -!- aidanh_ is now known as aidanh 16:11:04 advil: I'm happy to be a guinea pig for test branches of webtiles server now. My checkout for webserver is seperate from anything else 16:11:53 amalloy: thanks for checking that secret 16:14:22 I want it on the record that I suggested making all evocables consumable to PF yesterday in PM. 16:17:00 if fan is consumable, it's a cool "get away from me so I can go upstairs" item 16:43:34 well the xp evokers are sort of like that already, consumables that you find somewhat frequently; I think the problem is the item itself is something you always carry, regardless of rarity, and the effect is rarely useful 16:45:00 if you make it a true consumable and relatively rare, you sort of haven't solved anything fundamently in terms of inventory and mental burden; crawl just has a lot of items still 17:01:01 03Alex Jurkiewicz02 07https://github.com/crawl/crawl/pull/1281 * 0.25-a0-501-gcea0e51: Disable Codecov pull request comments 10(26 hours ago, 1 file, 1+ 0-) 13https://github.com/crawl/crawl/commit/cea0e51723e2 17:01:01 03Alex Jurkiewicz02 07https://github.com/crawl/crawl/pull/1281 * 0.25-a0-502-g62a7ee5: Exclude catch2-tests/ from coverage data 10(26 hours ago, 1 file, 8+ 0-) 13https://github.com/crawl/crawl/commit/62a7ee5bff1e 17:01:01 03Alex Jurkiewicz02 07https://github.com/crawl/crawl/pull/1281 * 0.25-a0-503-g4ffcabf: s/coveralls/codecov/ 10(25 hours ago, 1 file, 2+ 2-) 13https://github.com/crawl/crawl/commit/4ffcabf6e16a 17:01:01 03Alex Jurkiewicz02 07https://github.com/crawl/crawl/pull/1281 * 0.25-a0-504-gea567fe: Display ccache config per-build 10(35 hours ago, 1 file, 34+ 0-) 13https://github.com/crawl/crawl/commit/ea567fe2162e 17:01:01 03Alex Jurkiewicz02 07https://github.com/crawl/crawl/pull/1281 * 0.25-a0-505-ge44d735: Add size limits to ccache 10(19 hours ago, 1 file, 41+ 10-) 13https://github.com/crawl/crawl/commit/e44d7356aab6 18:22:50 Unstable branch on underhound.eu updated to: 0.25-a0-503-ge281fa673e (34) 19:58:35 -!- amalloy is now known as amalloy_