00:11:44 Unstable branch on CRAWL.XTAHUA.COM updated to: 0.24-a0-252-g5076262 (34) 01:38:10 Unstable branch on crawl.develz.org updated to: 0.24-a0-252-g5076262 (34) 02:08:28 Windows builds of master branch on crawl.develz.org updated to: 0.24-a0-252-g5076262 02:59:44 Monster database of master branch on crawl.develz.org updated to: 0.24-a0-252-g5076262 04:28:45 Engulf status for characters that can swim or can't breath is grayed out. But this eliminates both inability to cast spells and incoming damage. So what's the point in this grayed out 'engulf' status? 04:29:37 It'd be logical to stop incoming damage for those species, but leave them unable to cast spells. 04:34:42 At a guess, because the state of being able to swim or unbreathing can be temporary (eg via Necromutation, but there may have been more ways in the past) so you want to know if it's a bad time for it to run out. 05:17:25 Hi, I was wondering about using a high res app icon on non-MacOS systems. This is how it currently works: 05:17:26 #ifdef TARGET_OS_MACOSX 05:17:26 "dat/tiles/stone_soup_icon-512x512.png"; 05:17:26 #else 05:17:26 "dat/tiles/stone_soup_icon-32x32.png"; 05:17:27 #endif 05:17:47 I'm running on Linux, and the 32x32 icon is a bit rough looking in the task switcher etc 05:18:56 I've done a test build with the 512x512 icon and it works fine, but I only have one platform to test on. 05:19:29 I'm happy to submit a PR, but I wanted to first check that nobody knows of a good reason for not using the 512x512 icon. 06:30:52 New branch created: pull/1039 (1 commit) 13https://github.com/crawl/crawl/pull/1039 06:30:52 03Chris Rendle-Short02 07https://github.com/crawl/crawl/pull/1039 * 0.24-a0-253-gfe20157: Use high-res app icon on all platforms 10(13 minutes ago, 1 file, 0+ 4-) 13https://github.com/crawl/crawl/commit/fe201579af90 07:20:21 -!- baco1 is now known as baco 08:11:23 !tell lightbulbjim it probably would work these days but someone will need to test it on windows 08:11:24 advil: OK, I'll let lightbulbjim know. 09:22:29 Stable (0.22) branch on underhound.eu updated to: 0.22.1-59-ge965b3aba3 09:57:28 xeno23 (L21 OpAs) ASSERT(!invalid_monster(&mons)) in 'mon-death.cc' at line 1842 failed. (Depths:1) 10:14:57 I have a question about how to use github. Can't crawl/crawl make a new branch? 10:25:54 sdynet: you can't alter the crawl/crawl repo as you are not a member of the development team 10:26:03 you need to make a github fork of crawl/crawl, and alter that 10:26:31 See: https://help.github.com/en/articles/working-with-forks for lots of details 10:27:10 Within your fork (which, by default, will have the name sydent/crawl) you can create a branch; if you're preparing something to submit to the devteam, be sure to check out the information in the contribution guide 10:27:14 ??patches 10:27:14 patches ~ patch[1/2]: Too get details about how to discuss and submit a patch or pull request, see: https://github.com/crawl/crawl/blob/master/crawl-ref/docs/develop/contribution-process.md 10:27:41 !learn e patch[1 s/Too get/For/ 10:27:42 patch[1/2]: For details about how to discuss and submit a patch or pull request, see: https://github.com/crawl/crawl/blob/master/crawl-ref/docs/develop/contribution-process.md 10:28:53 This is the information I've been looking for. Thank you. :) 10:53:24 I want to remove the exploding and poison from throwables brand. And the steel brand, if possible. Can this be allowed? 10:54:29 Of course the poison needle is maintained. 10:58:53 sdynet: I think (I am not a Crawl dev) the difficulty there is political. There has been any amount of discussion of what to do about Throwing ammo. Just writing a patch to remove those isn't the hard bit, it's agreeing what to do. 11:00:20 But if you do want to write a patch, you will want to read crawl-ref/docs/develop/save_compatibility.txt 11:02:58 Pinkbeast: Thank you. I'll read it. 11:16:43 I looked at the Brainstorming Pages and the GDD Forum. And I've used throwables myself a lot. Based on discussion and direct experience, I came to think that unnecessary brands should be remove. The launcher system and the throwables system are old and require a lot of remuneration, but I try to correct a small part first because changing everything at once is difficult and dangerous. 11:22:43 Stable (0.23) branch on underhound.eu updated to: 0.23.1-50-ga50397a6bc 11:26:41 no idea what other devs curent thoughts on ranged are, but I'm not sure a piecemeal solution is necessarily what people who have worked on this have thought would be best 11:27:15 though I'm not sure I see the harm in removing those specific brands from throwables, either 11:41:24 Well, I think the brand of poison will be removed from both big and small plans. It's because I think poison brand are enough for poison needles. And I'd also like to get rid of steel brands because I think they're nothing more than copying the top items. 11:46:31 New branch created: pull/1040 (2 commits) 13https://github.com/crawl/crawl/pull/1040 11:46:31 03argonaut02 07https://github.com/crawl/crawl/pull/1040 * 0.24-a0-252-g8e06a14: The unrand staff of Battle 10(18 hours ago, 5 files, 56+ 0-) 13https://github.com/crawl/crawl/commit/8e06a1478abf 11:46:31 03argonaut02 07https://github.com/crawl/crawl/pull/1040 * 0.24-a0-253-g2dbfed2: Improve the staff of Battle's lore 10(2 hours ago, 1 file, 6+ 6-) 13https://github.com/crawl/crawl/commit/2dbfed264f3f 11:50:39 I think there are probably the fewest differences of opinion about two brand. If I were allowed to remove both of these, I would like to remove exploding brand after that. 13:05:53 re: throwables, i'd been thinking that removing steel, silver and poison from javelins/tomahawks would be a good start 13:08:45 huh. i thought poison javelins/tomahawks were already removed at the very least, but i appear to be quite wrong 13:09:30 I'd thought steel had been removed. 13:09:45 !source _determine_missile_brand 13:09:47 1/1. https://github.com/crawl/crawl/blob/master/crawl-ref/source/makeitem.cc#L507 13:10:37 geekosaur: maybe we're thinking of those brands that used to be on bolts and arrows 13:13:05 Unstable branch on crawl.akrasiac.org updated to: 0.24-a0-252-g5076262 (34) 13:13:16 still got simplified 13:13:18 er 13:13:19 steel 13:13:24 it used to have a different mulch rate 13:16:12 sdynet: I wrote some rough notes about my thinking in this direction 13:16:20 they're on the dev wiki: https://github.com/crawl/crawl/wiki/Throwing-brands 13:17:42 I haven't had a detailed discussion with the team on the stuff in here, so don't take it as the final word 13:23:26 the needle/dart stuff there looks reasonable to me at least 13:24:30 i was going to suggest just removing confusion/paralysis and keeping sleep for needles but making it distinct from hexes might be better 13:26:06 darts are relevant still? 13:28:37 geekosaur: the linked wiki page suggests replacing needles with darts 17:07:11 03advil02 07* 0.24-a0-253-g26cb771: Don't use div_rand_round for noise values 10(4 minutes ago, 1 file, 1+ 1-) 13https://github.com/crawl/crawl/commit/26cb7716be52 17:17:32 Unstable branch on crawl.kelbi.org updated to: 0.24-a0-253-g26cb7716be (34) 17:20:09 03advil02 07* 0.24-a0-254-gf493760: Prevent messages from impacting non-UI generators 10(2 minutes ago, 1 file, 6+ 0-) 13https://github.com/crawl/crawl/commit/f493760c106e 17:32:31 Unstable branch on crawl.kelbi.org updated to: 0.24-a0-254-gf493760c10 (34) 17:44:06 -!- jfcaron_ is now known as jfcaron 18:18:32 The build was fixed. (master - 26cb771 #11341 : advil): https://travis-ci.org/crawl/crawl/builds/521445876 18:20:47 Unstable branch on underhound.eu updated to: 0.24-a0-254-gf493760c10 (34) 18:29:56 -!- amalloy is now known as amalloy_ 19:16:00 The build was fixed. (master - f493760 #11342 : advil): https://travis-ci.org/crawl/crawl/builds/521450505 19:25:17 Thanks advil, that's fair. I'll see if I can rustle up some other systems to test on. 19:25:17 lightbulbjim: You have 1 message. Use !messages to read it. 19:25:22 !messages 19:25:23 (1/1) advil said (11h 13m 58s ago): it probably would work these days but someone will need to test it on windows 21:02:44 -!- johnsteinVPS is now known as johnstein 21:22:23 advil: these RNG bugs are fantastic 22:41:11 New branch created: pull/1041 (2 commits) 13https://github.com/crawl/crawl/pull/1041 22:41:11 03sdynet02 07https://github.com/crawl/crawl/pull/1041 * 0.24-a0-255-gee69749: Merge remote-tracking branch 'refs/remotes/crawl/master' into throwing 10(8 minutes ago, 0 files, 0+ 0-) 13https://github.com/crawl/crawl/commit/ee697493eb79 22:41:11 03sdynet02 07https://github.com/crawl/crawl/pull/1041 * 0.24-a0-256-g83b2a9c: Remove throwables ammo brands(poison, steel) 10(4 minutes ago, 6 files, 9+ 11-) 13https://github.com/crawl/crawl/commit/83b2a9c3ea3c 22:42:51 nice 22:42:55 I did pull requests: https://github.com/crawl/crawl/pull/1041 22:43:25 Please go over it. :) 23:30:00 -!- amalloy_ is now known as amalloy