01:48:42 <04d​racoomega> I think that one is on github's end (stuff like that seems to happen periodically) 03:35:26 Experimental (bcrawl) branch on underhound.eu updated to: 0.23-a0-5261-gd9800d219b 06:48:05 03CrawlOdds02 07* 0.35-a0-481-gca5e20b62d: Fix displaying messages when destroying melded equipment 10(7 days ago, 2 files, 104+ 63-) 13https://github.com/crawl/crawl/commit/ca5e20b62de8 06:48:05 03CrawlOdds02 07* 0.35-a0-482-g018e5526d6: Fix a few artefact properties to message on meld/unmeld 10(7 days ago, 1 file, 7+ 13-) 13https://github.com/crawl/crawl/commit/018e5526d6db 06:48:05 03CrawlOdds02 07* 0.35-a0-483-g8171d6967a: Show useless armour in wear menu 10(7 days ago, 1 file, 2+ 5-) 13https://github.com/crawl/crawl/commit/8171d6967a2a 06:48:05 03CrawlOdds02 07* 0.35-a0-484-gd7bf041781: Make god_hates_monster a boolean 10(5 days ago, 2 files, 13+ 18-) 13https://github.com/crawl/crawl/commit/d7bf0417813f 06:48:05 03CrawlOdds02 07* 0.35-a0-485-g00d325f07c: Forbid equipment hated by your god. 10(7 days ago, 17 files, 223+ 180-) 13https://github.com/crawl/crawl/commit/00d325f07cf2 06:48:05 03CrawlOdds02 07* 0.35-a0-486-g93b67a4c5e: Forbid scrolls and potions hated by your god 10(6 days ago, 4 files, 43+ 30-) 13https://github.com/crawl/crawl/commit/93b67a4c5e03 06:48:05 03CrawlOdds02 07* 0.35-a0-487-g2b460e5045: Forbid evoking items hated by your god 10(6 days ago, 5 files, 25+ 6-) 13https://github.com/crawl/crawl/commit/2b460e5045ef 06:48:05 03CrawlOdds02 07* 0.35-a0-488-g014f3fe60c: Forbid hated spells rather than applying penance 10(5 days ago, 14 files, 63+ 134-) 13https://github.com/crawl/crawl/commit/014f3fe60cb0 06:48:05 03CrawlOdds02 07* 0.35-a0-489-gf4b7c1965e: Forbid throwing items hated by your god 10(3 days ago, 3 files, 18+ 12-) 13https://github.com/crawl/crawl/commit/f4b7c1965e7d 06:48:05 03CrawlOdds02 07* 0.35-a0-490-g50d0f80f3c: Forbid animating weapons hated by your god 10(3 days ago, 2 files, 23+ 32-) 13https://github.com/crawl/crawl/commit/50d0f80f3c3e 06:48:05 ... and 10 more commits 07:26:21 03CrawlOdds02 07* 0.35-a0-501-g8a40e45818: Fix wizmode transforming into the null form 10(2 minutes ago, 1 file, 3+ 3-) 13https://github.com/crawl/crawl/commit/8a40e458183e 08:04:04 <11O​dds> !versions 08:04:08 <04C​erebot> Latest versions seen in milestones this week: [LLD] => 0.34.1-3-ga2c7840, [CAO] => 0.35-a0-472-gb61ad44, [CDO] => 0.35-a0-478-g324fca1, [CBR2 CUE CXC CRG CBRG] => 0.35-a0-479-gf4db44d, [CDI CNC] => 0.35-a0-480-g9205027, [CPO] => 0.35-a0-500-g1e03fe1 08:04:39 <11O​dds> Looks like CAO hasn't got a new version since this one I triggered manually after the python UTF-8 thing was hopefully fixed 08:29:59 bwhre (L20 OpSh) Crash caused by signal #11: Segmentation fault (Abyss:3) 08:31:59 !crashlog bwhre xl=20 opsh place=abyss:3 08:32:02 1. bwhre, XL20 OpSh, T:35235 (milestone): https://cbro.berotato.org/morgue/bwhre/crash-bwhre-20260611-152952.txt 08:34:17 <11O​dds> Another instance of https://github.com/crawl/crawl/issues/5280 perhaps? 08:36:40 looks suspiciously similar, yes 09:07:48 <11O​dds> Ah right, I've replicated it - pretty sure the root cause is that moving a marker (on abyss shift) is duplicating it in dynamic_markers. I'm not 100% sure what the right fix is because I'm not very familiar with markers; possibly we should just skip adding it to dynamic_markers if it is already there. 09:46:22 <09g​ammafunk> I think if you run a rebuild like you did before, it should show you any rebuild issue 09:47:19 <09g​ammafunk> running one now to check 09:48:09 <09g​ammafunk> tons of these warnings: > In file included from externs.h:41:0, > from AppHdr.h:306, > from ability.cc:6: > rltiles/tiledef_defines.h: In constructor 'tile_with_flags_t::tile_with_flags_t(tileidx_t, tile_flag_t)': > rltiles/tiledef_defines.h:17:67: warning: declaration of 'flags' shadows a member of 'this' [-Wshadow] > rltiles/tiledef_defines.h:17:67: warning: declaration of 'tile' shadows a member of 09:48:09 'this' [-Wshadow] > rltiles/tiledef_defines.h: In member function 'void tile_with_flags_t::set_tile(tileidx_t)': > rltiles/tiledef_defines.h:63:5: warning: declaration of 'tile' shadows a member of 'this' [-Wshadow] > ability.cc: In constructor 'scaling_cost::scaling_cost(int, int)': > ability.cc:135:9: warning: declaration of 'fixed' shadows a member of 'this' [-Wshadow] > At global scope: 10:01:02 <09g​ammafunk> unfortuantely I can't actually monitor this rebuild, which takes a good while, so I can check later if Odds doesn't hae time 10:03:34 <11O​dds> I can kick off a rebuild in a little while if yours isn't locking things 11:14:04 Unstable branch on crawl.akrasiac.org updated to: 0.35-a0-501-g8a40e45818 (34) 11:27:10 <11O​dds> Hmmmm... so it updates when we prod it but didn't for a few days in between 12:31:04 03CrawlOdds02 07* 0.35-a0-502-g3aa999ad01: Fix a crash with dynamic map markers 10(3 hours ago, 1 file, 7+ 1-) 13https://github.com/crawl/crawl/commit/3aa999ad01e3 14:09:00 151 | if (keyin < 'a' || keyin > 'Z') 14:09:02 | ~~~~~~~~~~~~^~~~~~~~~~~~~~ 14:09:11 so, uh, who's the EBCDIC programmer? 14:19:12 could be [, could be ^, could even be _ 14:22:09 but it's actually * 14:33:50 actually that one's always true because the cases are backwards and overlap 14:34:45 (new local install (thanks for trashing my old one, ubuntu), newer clang, new warning) 14:42:06 * being everything 14:42:58 <[> hi 14:43:51 the culprit! 14:46:02 New branch created: pull/5293 (1 commit) 13https://github.com/crawl/crawl/pull/5293 14:46:02 03geekosaur02 07https://github.com/crawl/crawl/pull/5293 * 0.35-a0-503-g495cbbe3cd: fix reversed letter test 10(3 minutes ago, 1 file, 1+ 1-) 13https://github.com/crawl/crawl/commit/495cbbe3cd1a 15:43:38 Unstable branch on underhound.eu updated to: 0.35-a0-502-g3aa999ad01 (34) 16:56:49 <04d​racoomega> Thanks for looking into this (this one was on my immediate to-do list since it was a consequence of recent changes I made to map markers, but I've been so buried under other things that I've not been very timely about bugfixes lately >.>) I do wonder, though; doesn't map_markers::move() imply 100% that the marker must already be in dynamic_markers? So instead we could use a bool argument for the add() call for 'skip adding to 16:56:49 dynamic_markers' instead of scanning the whole list for matches every time we ever add a marker? Am I missing something here? 19:25:31 <09g​ammafunk> weird, it's possible that there was a stale lock or something. If you happen to notice it not rebuilding again, let me know and I'll investigate further 22:36:02 Unstable branch on crawl.develz.org updated to: 0.35-a0-502-g3aa999ad01 (34) 23:00:10 Windows builds of master branch on crawl.develz.org updated to: 0.35-a0-502-g3aa999ad01 23:03:18 <11O​dds> Ah I hadn't clocked that this was recent. Yes, I think a boolean argument would work fine - we'd also need to set it in move_marker I think. And yeah that's probably cleaner than working it out (and cheaper, but I think minutely so unless there are way more dynamic markers and marker moves than I'd realised) 23:32:54 Unstable branch on cbro.berotato.org updated to: 0.35-a0-502-g3aa999ad01 (34) 23:56:19 Monster database of master branch on crawl.develz.org updated to: 0.35-a0-502-g3aa999ad01