00:36:17 New branch created: pull/2871 (1 commit) 13https://github.com/crawl/crawl/pull/2871 00:36:17 03Zhang Kai02 {GitHub} 07https://github.com/crawl/crawl/pull/2871 * 0.30-a0-440-g7e4365dc10: Create mutations.txt 10(82 seconds ago, 1 file, 831+ 0-) 13https://github.com/crawl/crawl/commit/7e4365dc102a 00:46:48 New branch created: pull/2872 (1 commit) 13https://github.com/crawl/crawl/pull/2872 00:46:48 03Zhang Kai02 {GitHub} 07https://github.com/crawl/crawl/pull/2872 * 0.30-a0-440-gc1ae0b232d: Update spells.txt 10(2 minutes ago, 1 file, 179+ 155-) 13https://github.com/crawl/crawl/commit/c1ae0b232dd2 00:47:55 Unstable branch on cbro.berotato.org updated to: 0.30-a0-439-g9e77e4d916 (34) 00:54:21 Monster database of master branch on crawl.develz.org updated to: 0.30-a0-439-g9e77e4d916 01:35:38 Fork (bcrawl) on crawl.kelbi.org updated to: 0.23-a0-4850-gd9e8576752 04:22:12 Experimental (bcrawl) branch on underhound.eu updated to: 0.23-a0-4850-gd9e8576752 05:27:55 Unstable branch on crawl.akrasiac.org updated to: 0.30-a0-439-g9e77e4d (34) 07:24:33 <06a​dvil> can someone on linux (ideally with dejavu fonts installed) tell me what happens if you run from the repo: util/find_font "DejaVu Sans Mono" 07:26:01 <06a​dvil> (I think that shouldn't work, if I'm parsing this script correctly) 07:27:55 https://paste.tomsmeding.com/0cOcwKRS 07:28:10 <06a​dvil> thanks 07:29:38 <06a​dvil> I wonder if we really need hardcoded font dir checks any more 07:45:18 <08w​ormsofcan> xmas sigmund tiles 07:45:18 <08w​ormsofcan> https://cdn.discordapp.com/attachments/747522859361894521/1051147601367548014/xmas-sigmund.png 07:45:19 <08w​ormsofcan> https://cdn.discordapp.com/attachments/747522859361894521/1051147601862459392/xmas-scythe.png 07:45:19 <08w​ormsofcan> https://cdn.discordapp.com/attachments/747522859361894521/1051147602357407876/xmas-scythedoll.png 07:46:01 <08w​ormsofcan> don't know if you can use special weapon definitions to give him a candy cane 08:00:50 03advil02 07* 0.30-a0-440-gd6ef553a8b: build: comment and improve util/find_font 10(14 minutes ago, 1 file, 22+ 2-) 13https://github.com/crawl/crawl/commit/d6ef553a8bdd 08:08:27 <09h​ellmonk> Turn ballisto spores into presents 08:14:30 03Aidan Holm02 {advil} 07[wide-minor-tag] * 0.30-a0-441-gece1214217: Allow phasing out support for old minor versions 10(2 years, 7 months ago, 3 files, 39+ 3-) 13https://github.com/crawl/crawl/commit/ece12142178a 08:14:30 Branch pull/1375 updated to be equal with wide-minor-tag: 13https://github.com/crawl/crawl/pull/1375 08:16:49 <06a​dvil> a present box icon for acquire scrolls might be cute actuall 08:17:01 <06a​dvil> so, we appear to be at tag minor 249 08:17:25 <06a​dvil> I'm pretty scared of 1375 (maybe everyone is) but possibly we should get over that and consider merging it 08:17:40 <06a​dvil> or actually break save compat 08:18:08 Unstable branch on crawl.kelbi.org updated to: 0.30-a0-440-gd6ef553a8b (34) 08:18:54 <06a​dvil> one thing is that we should add some of those hardcoded tag # checks as compile checks 08:19:20 <06a​dvil> since servers don't actually check CI status before installing (maybe they should) 08:23:17 <10P​leasingFungus> oh, belated thought re steam 08:23:52 <10P​leasingFungus> there’s pretty good odds that we get nearly no exposure from it 08:26:05 <10P​leasingFungus> so if the goal of a port is more exposure, we should keep that in mind 08:26:18 <10P​leasingFungus> ‘being on steam’ isn’t worth much these days 08:27:01 <09g​ammafunk> you'd need to run ads for dcss on my channel if you really wanted to break out into the mainstream 08:27:06 <10P​leasingFungus> mm 08:28:46 <09g​ammafunk> do you think merging that PR would cause much havoc in terms of having a stream of untransferable games for admins to deal with? 08:30:34 <09h​ellmonk> Turn shard shrikes into a crudely drawn delibird 08:31:34 <06a​dvil> my understanding is that it shouldn't change anything? 08:31:39 <06a​dvil> but I haven't really tested that 08:33:16 <06a​dvil> hmmm, maybe I still don't understand how save compat works when that invalid enum value changes though 08:36:39 <06a​dvil> re steam, would we need to code sign the mac version? 08:37:46 <10P​leasingFungus> what is the significance of tag minor 249? 08:37:55 <06a​dvil> v close to 256 08:38:03 <10P​leasingFungus> no, we wouldn’t have to do code signing, thank god 08:38:19 <06a​dvil> does steam have its own signing or something? 08:38:29 <06a​dvil> just wondering how it interacts with modern gatekeeper 08:38:49 <10P​leasingFungus> basically anything run via steam is ok by gatekeeper, iirc. will find a blog post in a sec 08:38:58 <10P​leasingFungus> apple claimed back in like uh. 2019. that they were going to end that arrangement 08:39:02 <10P​leasingFungus> then they didn’t 08:39:14 <10P​leasingFungus> probably because it would break almost everything on steam 08:39:20 <06a​dvil> heh 08:39:32 <06a​dvil> though they weren't so afraid of that with the 64bit thing 08:39:56 <06a​dvil> (I still have my laptop on mojave) 08:40:25 <10P​leasingFungus> mm 08:40:45 <10P​leasingFungus> yeah, idk. all i know is i wouldn’t ship on macos at all if it weren’t for steam 08:40:51 <10P​leasingFungus> despite developing on macos! 08:41:15 <10P​leasingFungus> what’s the significance of 256? 08:41:27 <06a​dvil> tag minor is serialized in a byte 08:43:29 04Build failed for 08wide-minor-tag @ ece12142 06https://github.com/crawl/crawl/actions/runs/3664763356 08:45:45 <10P​leasingFungus> that is not particularly true 08:46:01 <10P​leasingFungus> save_version get_save_version(reader &file) { int major, minor; try { major = unmarshallUByte(file); minor = unmarshallUByte(file); if (minor == UINT8_MAX) minor = unmarshallInt(file); } 08:46:21 <10P​leasingFungus> looks like we'll automatically fall over to a 32-bit int once we hit minor version 256? 08:46:37 <06a​dvil> oh, maybe an alternative to this PR did get merged 08:46:39 <10P​leasingFungus> void write_save_version(writer &outf, save_version version) { marshallUByte(outf, version.major); if (version.minor < UINT8_MAX) marshallUByte(outf, version.minor); else { marshallUByte(outf, UINT8_MAX); marshallInt(outf, version.minor); } } I guess this is the more relevant function 08:46:46 <10P​leasingFungus> we have this conversation like once a year 08:47:05 <10P​leasingFungus> there's a deep sense of fear and panic engraved in the bones of the dev team about the minor version 08:47:30 <10P​leasingFungus> april 2020, 43576957056 08:47:45 <06a​dvil> %git 435769570560c5b0 08:47:46 <04C​erebot> Aidan Holm * 0.25-a0-898-g4357695705: Add support for minor tags > 255 (2 years, 8 months ago, 9 files, 147+ 43-) https://github.com/crawl/crawl/commit/435769570560 08:48:35 <10P​leasingFungus> tyty 08:48:39 <10P​leasingFungus> on mobile now 08:49:04 <10P​leasingFungus> anyway we could still consider merging that pr 08:49:07 <06a​dvil> so I guess that PR would just simply allow removing old save compat 08:49:08 <10P​leasingFungus> if we wanted to drop 08:49:09 <10P​leasingFungus> yes 08:49:19 <10P​leasingFungus> but there’s no rush 08:49:42 <10P​leasingFungus> a nice rainy day project, maybe 08:49:47 <10P​leasingFungus> like much of crawl 🙂 08:49:58 <06a​dvil> well, it's soon going to be the oldest PR 08:53:26 <10P​leasingFungus> inspiring 08:53:35 <10P​leasingFungus> and vaguely thematically appropriate 08:55:35 <09g​ammafunk> I'm not panicking about save compat, you're panicking about save compat!!! 08:56:03 <09g​ammafunk> (I too had totally forgotten about that fix being merged re: minor versions) 09:01:33 <06a​dvil> with that context, merging this seems easier; it would immediately drop support for tag 33 games tho 09:01:44 03advil02 07[wide-minor-tag] * 0.30-a0-442-g1ada5ead49: fix: update tag test for catch2 v3 10(63 seconds ago, 1 file, 1+ 1-) 13https://github.com/crawl/crawl/commit/1ada5ead4935 09:01:44 Branch pull/1375 updated to be equal with wide-minor-tag: 13https://github.com/crawl/crawl/pull/1375 09:08:29 <10P​leasingFungus> much though i adore back compat, i can’t say i care about tag major 33 09:08:48 <06a​dvil> my feeling is that probably anything before about 150 is not really very playable on upgrade 09:09:06 <10P​leasingFungus> iirc kate has had said something similar 09:09:12 <10P​leasingFungus> it seems plausible 09:10:18 <05k​ate> i have a bunch of existing old games that are pretty unplayable yeah 09:11:37 <09g​ammafunk> I think all servers with trunk versions that can date back to 0.11 are dgl-based ones, do they give the prompt upon failed transfer attempt giving the player the option to either delete the save or continue with the current version? 09:11:53 <09g​ammafunk> in fact, is this literally only cao? 09:11:55 <10P​leasingFungus> can imagine a world in which we’d tried to mitigate that, by replacing removed monsters instead of ghosting them, replacing removed items, etc. but probably a poor use of hypothetical effort 09:12:15 03advil02 07[wide-minor-tag] * 0.30-a0-443-gee5ae1b7c3: fix: tweak minor tag tests 10(2 minutes ago, 2 files, 5+ 0-) 13https://github.com/crawl/crawl/commit/ee5ae1b7c33b 09:12:15 Branch pull/1375 updated to be equal with wide-minor-tag: 13https://github.com/crawl/crawl/pull/1375 09:12:16 <09g​ammafunk> not sure if cbr2 can have trunks from 0.11 09:12:18 <06a​dvil> I don't think any servers are likely to have trunk saves that are still this old kicking around 09:12:29 <06a​dvil> cbr2 definitely doesn't 09:12:42 <09g​ammafunk> I mean, mikee had one like a year or two ago 09:12:49 <06a​dvil> on what server? 09:13:19 <09g​ammafunk> well let's see, we can find it because his mummy was LITERALLY KILLED BY PLEASINGFUNGUS' PAN ORC END 09:13:21 <06a​dvil> ah, right, there is a thing I always forget which is that just loading up a save does not upgrade version in all chunks 09:13:33 <09g​ammafunk> ah, right 09:13:54 <09g​ammafunk> !lg mikee mu orc:2 09:13:56 <04C​erebot> No games for mikee (mu orc:2). 09:14:03 <09g​ammafunk> oh right one of his damn alts 09:14:12 <06a​dvil> so to correct myself: I doubt any servers have saves where character chunks are anywhere near that old. But old servers like cao may still have saves that have unupgraded map chunks 09:15:38 <09g​ammafunk> well I do think servers have even character chunks that old, but I think they are all parked and likely never to be loaded again. Have you checked cao for this, is that why you're so certain? Maybe you've done recentish purges of old trunks 09:16:01 <06a​dvil> I've done a lot of force upgrading on cao while disk space was tight 09:16:04 <09g​ammafunk> gotcha 09:16:29 <06a​dvil> I haven't been bothering lately but nothing there is on a game version more than a year or two old, which will have upgraded the char chunk 09:16:37 <09g​ammafunk> right 09:16:50 <06a​dvil> and the current map chunk (plus other stuff I believe) 09:17:00 <09g​ammafunk> in that case, yeah, guess it would just be the issue of unupgraded chunks, which I'm not sure we can do anything about 09:17:34 <06a​dvil> well, prob could come up with a way of upgrading them 09:17:40 <06a​dvil> not sure if that's worth the time 09:18:24 <06a​dvil> it's maybe sort of a misfeature that crawl upgrades map chunks lazily 09:18:40 <09g​ammafunk> wow, extremely rude of discord search to allow "mike" to match "mikee", now I have to sift through messages from octopode mike as well 09:18:40 <06a​dvil> but it's just a bit of a pain to try to change that 09:18:53 <06a​dvil> and by "a bit" I probably mean hideously awful 09:20:06 <06a​dvil> fr save upgrading in libcrawl feature 09:21:05 <09g​ammafunk> fr: libcrawl 09:21:59 <09g​ammafunk> (both because it would be cool and because the FAR LEFT LIBS on the DEV TEAM will have finally RUINED THE GAME) 09:22:17 <10P​leasingFungus> about time 09:22:19 <09g​ammafunk> found that game, it was 3 years old 09:22:23 <09g​ammafunk> !lg qwyjybo mu orc -log 09:22:24 <04C​erebot> 1. hypermikee, XL13 MuEE, T:32834: http://crawl.develz.org/morgues/trunk/hypermikee/morgue-hypermikee-20180210-065929.txt 09:22:34 <09g​ammafunk> ah, cdo ,nice 09:22:39 <10P​leasingFungus> i’ve been trying to ruin the game for years, but someone reverted shadow traps! 09:22:46 <10P​leasingFungus> what’s a guy gotta do 09:22:55 <09g​ammafunk> false alarm: > 0 | D:1 | hypermikee, the Mummy Earth Elementalist, began the quest for the Orb. > 0 | D:1 | Reached XP level 1. HP: 13/13 MP: 3/3 > 0 | D:1 | Upgraded the game from 0.12-a0-2413-g03c53c6 to 0.22-a0-48-g8770677 > 0 | D:1 | Upgraded the game from 0.22-a0-48-g8770677 to 0.22-a0-159-g7ce444f 09:22:57 <09g​ammafunk> not even 0.11 09:23:08 <09g​ammafunk> gloom traps? 09:23:23 <06a​dvil> I think that one should be fine (based on how I understand things to work) because it was upgraded on turn 0 09:23:50 <09g​ammafunk> in terms of save chunks? yeah 09:23:56 <09g​ammafunk> or level chunks rather 09:24:06 <10P​leasingFungus> shadow traps just shouldn’t have been triggered by summons 09:24:11 <09g​ammafunk> that game definitely behaved very oddly 09:24:25 <09g​ammafunk> iirc he had a depths entrance on like d:15 but D was still 27 levels? 09:24:31 <10P​leasingFungus> lol 09:24:42 <06a​dvil> yeah, I guess the dungeon structure may be set on game creation 09:24:53 <06a​dvil> but fixups will try to add depths entrances 09:25:01 <06a​dvil> I think there may still be some lingering bugs with those fixups 09:25:45 <09g​ammafunk> I want to say there were crashes associated with his trying to enter depths, not sure if the crash history would show this though 09:26:00 <09g​ammafunk> in any case, not relevant to the PR or current problems I guess 09:27:07 <09g​ammafunk> oh also it was over four years old,not three 09:27:21 <10P​leasingFungus> time flies 09:27:24 <06a​dvil> oh, it fixed itself 09:27:25 <06a​dvil> https://github.com/crawl/crawl/issues/1128 09:27:33 <06a​dvil> so I closed the issue 09:27:43 <10P​leasingFungus> nice 09:27:57 <09g​ammafunk> lol 09:28:11 <09g​ammafunk> > Entering D:27 in any version after 0.13 creates a set of stairs to the Depths. In version 0.14 and 0.15, these stairs are a mimic monster. In 0.16, these stairs are a mimic that vanishes in smoke. In 0.17 and later, these stairs are actual stairs that take the player to the Depths, but doing so causes a crash. 09:30:08 03advil02 07[wide-minor-tag] * 0.30-a0-443-g539b42731e: fix: tweak minor tag tests 10(20 minutes ago, 2 files, 7+ 0-) 13https://github.com/crawl/crawl/commit/539b42731e65 09:30:08 Branch pull/1375 updated to be equal with wide-minor-tag: 13https://github.com/crawl/crawl/pull/1375 09:39:37 <06a​dvil> hm https://github.com/crawl/crawl/pull/2398/ tries to weight acquirement base type by skill, which seems potentially good (not sure), but the PR is not finished and never got dev comments (and it looks a bit messy to me) 10:29:59 commit 744735845 seems to cause species who can swim (e.g. octopode) to get the "You sink to the bottom." message whenever they step into deep water, I assume this wasn't intended (it looks like a you.can_swim() check was removed in a refactor) 10:31:01 (I could mute the message in rc obviously but presumably it's a bug) 10:43:17 <06a​dvil> huh, why do both delay.cc:_monster_warning and view.cc:_handle_comes_into_view both exist? they appear to do the same thing in slightly different ways (at different times?) 10:43:46 also, commit 77163bc4 means that you can no longer tell whether ?identify is permanently useless by looking at its color 10:44:00 i.e. it's more annoying to know when it is safe to drop it 10:44:00 I was puzzled by that when doing friendly ghosts but never got to the bottom of it 10:44:38 in general I'm not convinced it's great to use temporary uselessness to determine inventory color, maybe these items should only be greyed out when in an action menu but not in inventory? 10:45:10 <06a​dvil> I could see an argument for three colors 10:45:31 <10P​leasingFungus> elliptic: oops, not intended 10:45:36 e.g. it isn't actually useless to have a scroll of identify in your inventory when you have nothing left to identify on D:2, since you will likely pick something up soon 10:45:48 <10P​leasingFungus> but the commit message did say there was one unintended bug in that commit! 10:45:54 it did 10:46:03 you should thank me for proving the commit message accurate 10:46:28 <10P​leasingFungus> tyty 10:46:46 <06a​dvil> I think the idea is that primary function of the color is to indicate that the item won't work, not that it should be dropped 10:46:56 <10P​leasingFungus> assuming there isn’t a second bug 🙂 10:47:33 <08n​icolae> there's only ever one bug 10:47:47 is it important to know that items won't work when you can just try them out and there's no cost when they don't work? 10:49:21 <06a​dvil> it does seem helpful to me (esp to newer players), yes 10:49:22 at least for "strategic" consumables this seems not particularly useful, maybe it's nice to grey out stuff like ?tele while in treeform or whatever to help people learn those interactions 10:50:09 but I'm not really understanding how it is useful for new players to have ?identify and ?enchant scrolls sometimes be a different color (a color mostly used for things that are permanently useless) 10:51:54 <06a​dvil> er, how is it not helpful to cue whether things will or won't work? 10:52:12 can easily imagine a relatively new player who has seen e.g. ring of poison resistance being marked as useless as a naga then play an octopode, identify a scroll of enchant armour, and assume that octopodes can never wear any armour 10:52:55 what I'm saying is that for strategic consumables the relevant thing isn't whether they can be used imminently, but whether they can have some future use 10:54:50 <06a​dvil> I guess my response is that both things are useful, which is why I said I can see the argument for 3 levels of indication 10:55:12 <06a​dvil> at some point I'd like to add the specific uselessness reason to the description as well, but that's only implemented for scrolls right now 10:55:43 <06a​dvil> that = code to generate that reason 10:56:10 3 levels would be fine, I'm just saying that if there are only 2 levels then I think it would make sense to treat strategic consumables differently 10:58:03 like my point is basically that the vast majority of interactions with useless items are things that are permanently useless, and people get used to dropping such items and forgetting about them 10:59:55 and then the first time they play an octopode or get an early artefact weapon and aren't carrying around other weapons, they get these temporarily useless scrolls and it's weird 11:00:45 mainly though I liked the old feature of the game telling you when identify scrolls had become permanently useless - maybe instead they could just vanish in a puff of smoke or something :P 11:05:41 <06a​dvil> well, id and enchant (and brand) probably can be easily changed in inventory 11:10:29 <09g​ammafunk> as a long-time and avid supporter of mimics in all their delightful forms, +1 for useless scrolls of identification turning into mimics 11:13:47 <10P​leasingFungus> “same but unironically” 11:13:58 <10P​leasingFungus> (i don’t feel strongly about it.) 11:45:56 <08n​icolae> i disagree strongly 11:46:05 <08n​icolae> i believe all scrolls of identification should be mimics 11:48:58 03nicolae02 07[svaults-sfor-sbranches] * 0.29-a0-943-g7669466f39: Cut the reclaimed swamp decor vaults for now 10(2 minutes ago, 1 file, 0+ 169-) 13https://github.com/crawl/crawl/commit/7669466f39e0 11:48:58 Branch pull/2632 updated to be equal with svaults-sfor-sbranches: 13https://github.com/crawl/crawl/pull/2632 12:06:34 04Build failed for 08svaults-sfor-sbranches @ 7669466f 06https://github.com/crawl/crawl/actions/runs/3665493179 13:02:43 03PleasingFungus02 07* 0.30-a0-441-g4e368e6683: Allow applying Haste and Might to statues 10(3 days ago, 1 file, 0+ 2-) 13https://github.com/crawl/crawl/commit/4e368e6683f8 13:02:43 03PleasingFungus02 07* 0.30-a0-442-gbaa0261540: Don't allow Mighting monsters with no attacks 10(2 days ago, 1 file, 1+ 0-) 13https://github.com/crawl/crawl/commit/baa026154056 13:02:43 03PleasingFungus02 07* 0.30-a0-443-gcbdd7f1727: Rewrite monster net immunity code 10(2 days ago, 4 files, 28+ 17-) 13https://github.com/crawl/crawl/commit/cbdd7f17275e 13:02:43 03PleasingFungus02 07* 0.30-a0-444-g87402fac92: Fix 'sink to the bottom' messages (elliptic) 10(86 seconds ago, 1 file, 10+ 3-) 13https://github.com/crawl/crawl/commit/87402fac92c6 13:09:53 <09g​ammafunk> when zappin' 13:10:03 03PleasingFungus02 07* 0.30-a0-445-g92f4fb2d71: Don't warn when zappin your spectral (Aoae, acrobat) 10(5 minutes ago, 1 file, 5+ 0-) 13https://github.com/crawl/crawl/commit/92f4fb2d710c 13:22:32 <10P​leasingFungus> was trying to reduce commit message length 13:22:37 <10P​leasingFungus> oughta used emoj 13:38:04 <09h​ellmonk> 🚫⚠️⚡👻⚔️ 13:40:47 Unstable branch on crawl.kelbi.org updated to: 0.30-a0-445-g92f4fb2d71 (34) 13:44:55 re: temporarily useless vs. permanently useless items 13:45:04 Previously, grey ?identify meant you identified all potions and scrolls. Now, it could also mean that you just don't have unid'ed items in your inventory. I agree, that it feels like a regression 13:45:13 I have a suggestion for making ?id always useful and would be glad to hear your feedback 13:45:27 The current ?id's description is "A useful magic scroll which identifies the properties of any unknown object" It won't have to change :) 13:45:39 FR: Some/all randarts generate with a hidden property (???): e.g., the +3 dagger "Plog" {vorpal, rElec rF+ ???} 13:45:43 1) ??? can be any valid artefact property 2) ??? does nothing until identified 3) ??? can be identified only via a scroll (or by Ash) 13:45:49 It makes randarts more interesting, as you can gamble on improving them. On the other hand, it adds more fiddling and doesn't allow a free inventory slot in the 2nd half of the game. 14:21:12 the gamble would be that the property might be negative? 14:22:58 the gamble itself sounds sort of neat but also potentially confusing to players, especially since mechanically this is pretty different from how ?identify is used elsewhere 14:23:45 (the confusing part is that the property does nothing until identified, while other items do not have this feature) 14:25:05 I would maybe rather use ?enchant weapon/armour for this purpose 14:25:41 (would mean you couldn't have ??? on jewellery randarts) 14:27:21 <10P​leasingFungus> agreed with concerns about this being completely different from how id works otherwise 14:27:49 <10P​leasingFungus> reminds me of how you could id wands to reduce charge consumption 14:28:14 <10P​leasingFungus> which was removed at least in part because it felt different from the normal purpose of id 14:28:50 <10P​leasingFungus> feels like maybe a good gimmick for a god ability 14:29:03 <10P​leasingFungus> rather than something available to all players every game 14:42:42 <10P​leasingFungus> specifically reminds me a lil of igni ipthes 14:51:51 The game no longer has identification though use, so maybe dormant ??? properties won't be too confusing for players 14:52:17 One more thing on the topic of useless items, there's a PR about autoexplore visiting piles with useless items: https://github.com/crawl/crawl/pull/2650 14:52:23 For example, Felids travel to and stop at all piles with glowing weapons and armour, which are completely useless for them 14:52:28 The PR was submitted before 0.29 release, but apparently fell through the cracks. Also, huge thanks for merging a bunch of PRs recently! 14:58:00 s/no longer has identification though use/no longer has identification of non-consumables though use/ 14:58:56 New branch created: pull/2873 (9 commits) 13https://github.com/crawl/crawl/pull/2873 14:58:58 03Aliscans02 07https://github.com/crawl/crawl/pull/2873 * 0.30-a0-425-g52b3326106: Create a GUI to set GameOption options. 10(2 days ago, 11 files, 324+ 95-) 13https://github.com/crawl/crawl/commit/52b3326106fd 14:58:58 03Aliscans02 07https://github.com/crawl/crawl/pull/2873 * 0.30-a0-426-g5800b0ba76: Only set a few special menu colours when a keyword is a complete word. 10(2 days ago, 1 file, 22+ 22-) 13https://github.com/crawl/crawl/commit/5800b0ba768f 14:58:58 03Aliscans02 07https://github.com/crawl/crawl/pull/2873 * 0.30-a0-427-g9c2da81735: Display the first name in the GameOption::names list. 10(5 hours ago, 2 files, 15+ 19-) 13https://github.com/crawl/crawl/commit/9c2da81735cf 14:58:58 03Aliscans02 07https://github.com/crawl/crawl/pull/2873 * 0.30-a0-428-g3d7d21661e: Set Options.prefs_dirty when relevant options are set in the GUI. 10(5 hours ago, 3 files, 57+ 33-) 13https://github.com/crawl/crawl/commit/3d7d21661e1a 14:58:58 03Aliscans02 07https://github.com/crawl/crawl/pull/2873 * 0.30-a0-429-gcb345704c3: Add a search command, which filters the displayed list of options. 10(5 hours ago, 1 file, 108+ 6-) 13https://github.com/crawl/crawl/commit/cb345704c346 14:58:58 03Aliscans02 07https://github.com/crawl/crawl/pull/2873 * 0.30-a0-430-g58d7d4530f: Add help feature to the options menu and some submenus. 10(5 hours ago, 4 files, 135+ 35-) 13https://github.com/crawl/crawl/commit/58d7d4530fcd 14:58:58 03Aliscans02 07https://github.com/crawl/crawl/pull/2873 * 0.30-a0-431-gc8c5719cc8: Arrange the options according to the help files. 10(2 hours ago, 4 files, 107+ 27-) 13https://github.com/crawl/crawl/commit/c8c5719cc8bb 14:58:58 03Aliscans02 07https://github.com/crawl/crawl/pull/2873 * 0.30-a0-432-g11ed36cdc7: Allow GameOption options to be saved to disk. 10(2 hours ago, 1 file, 37+ 3-) 13https://github.com/crawl/crawl/commit/11ed36cdc793 14:58:58 03Aliscans02 07https://github.com/crawl/crawl/pull/2873 * 0.30-a0-433-g65a310dcb5: Clear the "undocumented option" list. 10(2 hours ago, 3 files, 29+ 11-) 13https://github.com/crawl/crawl/commit/65a310dcb52e 15:00:09 ^ Wow, now the game is ready for Steam! 15:10:28 03Aliscans02 07https://github.com/crawl/crawl/pull/2873 * 0.30-a0-434-g0440a2c956: unbrace (for last change but 1) 10(2 minutes ago, 1 file, 0+ 2-) 13https://github.com/crawl/crawl/commit/0440a2c956ad 15:14:15 <10P​leasingFungus> sick 15:45:00 03Aliscans02 07https://github.com/crawl/crawl/pull/2873 * 0.30-a0-435-g9b57931ea2: Prevent a tile_use_small_layout crash. 10(52 seconds ago, 1 file, 1+ 1-) 13https://github.com/crawl/crawl/commit/9b57931ea2b7 16:12:22 <06a​dvil> wow 16:23:56 Unstable branch on underhound.eu updated to: 0.30-a0-445-g92f4fb2d71 (34) 16:28:04 03Aliscans02 07https://github.com/crawl/crawl/pull/2873 * 0.30-a0-436-g9a1c2ba287: Give some other MultipleChoiceGameOptions an option whenever defined. 10(65 seconds ago, 1 file, 2+ 2-) 13https://github.com/crawl/crawl/commit/9a1c2ba287cd 16:33:06 It should all work now. I don't think any of the other types of GameOption make assumptions about the default value. 16:35:02 <06a​dvil> great! 16:36:20 <06a​dvil> I will try to take a look soon 16:37:20 <06a​dvil> re that last commit, I'm guessing somewhere there should be a hardcoded list of options not to show 16:38:10 <06a​dvil> since both of those options affect game start only, they'd probably be kind of confusing to show at all in this kind of ui (though giving them a default for that case is still good) 16:38:58 I don't know. Either that, or they shouldn't be GameOptions in that situation. 16:43:20 I like the way Nethack shows options which can't be set during the game (they appear, but there's no way to change them). There isn't a list of options which have no effect during the game, so I haven't tried to do anything like that. 16:46:27 The problem I'm having is that it takes me quite a while to compile Crawl, so I've only really tested one set of compile options. 16:47:11 <06a​dvil> ah are you using msys2? 16:48:02 <06a​dvil> if say at some point you may want to try wsl, I think it would compile a lot faster, and you could install ccache 16:48:10 <06a​dvil> (is ccache available on msys2?) 16:49:06 <09g​ammafunk> should be, but of course I've not tested it 16:49:25 <09g​ammafunk> https://packages.msys2.org/package/mingw-w64-x86_64-ccache?repo=mingw64 16:50:41 No, I'm on Linux. I used ccache some time ago, but I got a few weird error messages and gave up on it. 16:51:56 <09g​ammafunk> once every contributor gets a PR successfully merged for the first time, bh should send them a new state of the art dev laptop imho 16:52:09 <06a​dvil> ah, ccache is definitely worth it 16:52:17 <06a​dvil> was thinking msys2 because it has excruciatingly slow linking 16:53:21 <09g​ammafunk> ccache definitely makes development a lot smoother, but sadly some of us do development on older machines without that many cores etc 16:53:42 No. I'm having some computer issues at the moment, so I'm only compiling one file at the time. 16:54:10 <06a​dvil> ouch 16:55:09 <09g​ammafunk> still need to set up my ccache to work better with multiple checkouts 16:55:36 <09g​ammafunk> as it stands I can't get cache hits for compilations between different dirs, and I recently learned it's possible to make that work 16:56:16 <09g​ammafunk> and generally need to have at least active checkouts, including an extra one that's a non-debug build to run qw a bit faster 16:56:40 <09g​ammafunk> that one can't really benefit from ccache from other dirs, since the others are full debug 16:57:28 <09g​ammafunk> *generally need to have at least two active checkouts 17:03:41 03advil02 07* 0.30-a0-446-g43e804b486: feat: tweak when temp uselessness is shown 10(3 minutes ago, 7 files, 24+ 9-) 13https://github.com/crawl/crawl/commit/43e804b48643 17:29:27 Unstable branch on crawl.kelbi.org updated to: 0.30-a0-446-g43e804b486 (34) 17:47:41 03Samantha Tobias02 {advil} 07* 0.30-a0-447-g9651f06ebd: fix wizlab wizards "come into view" message (Flugkiller) 10(3 months ago, 1 file, 3+ 3-) 13https://github.com/crawl/crawl/commit/9651f06ebd38 18:04:13 Unstable branch on crawl.kelbi.org updated to: 0.30-a0-447-g9651f06ebd (34) 18:32:44 03PleasingFungus02 07* 0.30-a0-448-g7f7f6b1b85: Replace Cigotuvi's summoning scrolls with poison 10(2 hours ago, 1 file, 2+ 2-) 13https://github.com/crawl/crawl/commit/7f7f6b1b850c 18:32:44 03PleasingFungus02 07* 0.30-a0-449-gb646966af9: Make ?butt alternate with ?summ instead of ?fog 10(2 hours ago, 19 files, 49+ 38-) 13https://github.com/crawl/crawl/commit/b646966af9b1 18:33:47 03advil02 07* 0.30-a0-450-gcec5606451: fix: show classy draconians' flight 10(6 minutes ago, 1 file, 2+ 0-) 13https://github.com/crawl/crawl/commit/cec56064513e 18:36:56 03PleasingFungus02 07* 0.30-a0-451-ge042605de7: Make ?butterfly weight match ?summoning (Prakerore) 10(59 seconds ago, 1 file, 1+ 1-) 13https://github.com/crawl/crawl/commit/e042605de741 19:16:33 Unstable branch on crawl.kelbi.org updated to: 0.30-a0-451-ge042605de7 (34) 23:34:55 Unstable branch on crawl.develz.org updated to: 0.30-a0-451-ge042605de7 (34)