03:34:04 Experimental (bcrawl) branch on underhound.eu updated to: 0.23-a0-5199-gb287095a7e 05:07:08 Unstable branch on crawl.akrasiac.org updated to: 0.32-a0-1039-ge526281 (34) 05:38:27 03dolorous02 07https://github.com/crawl/crawl/pull/3791 * 0.32-a0-1040-gc6e4ea2c6f: Add rare fire-themed temple to Ignis and Makhleb. 10(13 hours ago, 1 file, 35+ 0-) 13https://github.com/crawl/crawl/commit/c6e4ea2c6fde 05:40:24 <06d​olorous_84348> After some thought, I've moved the temple to altar.des, since that includes temples to minor gods. 05:49:18 I think "temple" is an unfortunate word in this context given AFAIK it is usually only used of the Temple, and that's what temple.des does 06:29:32 New branch created: pull/3792 (1 commit) 13https://github.com/crawl/crawl/pull/3792 06:29:33 03caryoscelus02 07https://github.com/crawl/crawl/pull/3792 * 0.32-a0-1040-g86c3965d97: Fix formicids able to shaft while in non-mutaion-preserving forms 10(50 minutes ago, 1 file, 7+ 0-) 13https://github.com/crawl/crawl/commit/86c3965d9712 06:39:08 03dolorous02 07* 0.32-a0-1040-g55e95659df: Add goblin sharper lines for renaining Nem cards. 10(2 minutes ago, 1 file, 23+ 0-) 13https://github.com/crawl/crawl/commit/55e95659dfa8 06:43:51 04Build failed for 08master @ 55e95659 06https://github.com/crawl/crawl/actions/runs/8924846870 09:27:20 03dolorous02 07* 0.32-a0-1041-g0624dd83d0: Adjust a goblin sharper line. 10(3 minutes ago, 1 file, 1+ 1-) 13https://github.com/crawl/crawl/commit/0624dd83d0e2 09:32:07 04Build failed for 08master @ 0624dd83 06https://github.com/crawl/crawl/actions/runs/8927210080 09:38:01 03dolorous02 07* 0.32-a0-1042-ge50fcabe14: Fix spelling. 10(3 minutes ago, 2 files, 4+ 4-) 13https://github.com/crawl/crawl/commit/e50fcabe14df 09:41:15 04Build failed for 08master @ e50fcabe 06https://github.com/crawl/crawl/actions/runs/8927350462 10:56:51 03dolorous02 07* 0.32-a0-1043-geb237df2fa: Add other oddities cards to goblin sharper lines. 10(13 minutes ago, 1 file, 15+ 1-) 13https://github.com/crawl/crawl/commit/eb237df2faca 11:01:31 04Build failed for 08master @ eb237df2 06https://github.com/crawl/crawl/actions/runs/8928289726 11:04:19 03dolorous02 07* 0.32-a0-1044-g348a825d6c: Remove low weight from misplaced card message. 10(4 minutes ago, 1 file, 0+ 1-) 13https://github.com/crawl/crawl/commit/348a825d6c3f 11:06:45 04Build failed for 08master @ 348a825d 06https://github.com/crawl/crawl/actions/runs/8928371362 11:26:40 03dolorous02 07* 0.32-a0-1045-g8129f212cf: Move Nem's oddities cards into their own entry. 10(8 minutes ago, 1 file, 6+ 1-) 13https://github.com/crawl/crawl/commit/8129f212cf45 11:31:14 04Build failed for 08master @ 8129f212 06https://github.com/crawl/crawl/actions/runs/8928663690 12:20:58 03mumra02 07https://github.com/crawl/crawl/pull/3746 * 0.32-a0-993-g5a83655869: Let blindness be cured by potion of curing 10(16 hours ago, 2 files, 13+ 8-) 13https://github.com/crawl/crawl/commit/5a836558691f 12:20:58 03mumra02 07https://github.com/crawl/crawl/pull/3746 * 0.32-a0-994-g008d8d38fe: Monster version of Dazzling Flash 10(8 hours ago, 12 files, 119+ 49-) 13https://github.com/crawl/crawl/commit/008d8d38fefb 12:28:36 <04d​racoomega> I think that if you let !curing remove blindness then it effectively will never matter at the times where it could actually matter 12:28:59 <04d​racoomega> You can ignore it when it's not being impactful enough to care, and generally expect to have enough potions to remove it any time this is not the case, I think 12:29:04 A comment in crawl-ref/source/util/levcomp.lpp says that it is "Based loosely on NetHack's lev_comp.l", but NetHack (including lev_comp.l) is under an incompatible license to crawl (NetHack GPL rather than GPLv2-or-later) 12:29:39 <04d​racoomega> (Confusing is a far more crippling status and yet this is already fairly true for it beyond earlier parts of the game) 12:30:03 <04d​racoomega> And there are multiple physical afflictions already that !curing does not affect 12:30:51 <04d​racoomega> Does 'based loosely' count as a direct derivative here, though? (I genuinely have no idea about this code) 12:32:31 I have no idea 12:35:36 there's also the question of whether it's significant, if it's not linked in 12:36:41 (I'm not sure if it is or not; I generally think of stuff in util as being separate from crawl itself) 12:36:58 New branch created: pull/3793 (1 commit) 13https://github.com/crawl/crawl/pull/3793 12:36:58 03mumra02 07https://github.com/crawl/crawl/pull/3793 * 0.32-a0-1046-gbf53647b58: Break system packages 10(12 minutes ago, 1 file, 1+ 1-) 13https://github.com/crawl/crawl/commit/bf53647b5862 12:40:45 no, it's definitely linked in 12:41:36 time to talk to the nethack devs about a license exemption? 12:43:33 <06m​umra> Hmm yeah fair enough, i wasn't sure (easy commit to omit anyway) 12:43:40 <04d​racoomega> I mean, I'd be unsure from that comment (without examining the code itself) whether it's incorporating code or just doing 'the same general idea' 12:49:10 <06m​umra> https://nethackwiki.com/wiki/Source:NetHack_3.4.3/util/lev_comp.l 12:49:20 <06m​umra> I mean it looks entirely different 12:49:29 <06m​umra> The comment didn't even get the filename right :{ 12:51:13 <06m​umra> Adding in that --break-system-packages flag appears to fix the MacOS build https://github.com/crawl/crawl/actions/runs/8929480658/job/24527360894?pr=3793 12:53:09 I had been considering adding it (and enforcing the caveats, just in case) 12:53:40 <12g​e0ff> too late, the GPL police is comin' 🚨 12:56:03 <06m​umra> I think the caveats (specifically about adding the --user flag) don't really matter since it's just a container and you throw it away at the end, it's not like you care about Homebrew being broken after the build has run 12:56:44 <06m​umra> If you're doing it on your own dev machine then sure (oops, I already did it without 😂 ) 12:56:52 they matter if they break some other package we need during the build, but I suppose --user doesn't necessarily protect us from that 12:58:07 I'm just cautious about such things since I know people who've broken their OS installs by upgrading / replacing packages used by their distro's packaging system 12:58:37 (RH/Fedora actually uses a separate Python installation for the installer these days to avoid that) 12:59:23 <06m​umra> I think the --user flag is to avoid breaking permissions on a normal multi user system, but in a docker container you're just running everything as root anyway so it probably doesn't matter 12:59:55 no, it means "install the package for the current user instead of system-wide like e.g. brew does" 13:00:09 <06m​umra> Sure, but in a container the current user is root regardless 13:00:24 so --break-system-packages without --user will overwrite the system packages 13:00:42 yes, but root still has a separate user package database 13:01:16 (/root/.local/lib/python-) 13:01:28 <06m​umra> Ah i see 13:02:38 well, that's linux, it's probably under the root account's Library on OS X 13:03:08 won't try to guess where, I no longer own any Macs 13:09:27 <06m​umra> Trying with --user as well to see what happens 13:09:59 03mumra02 07https://github.com/crawl/crawl/pull/3793 * 0.32-a0-1046-gabb3cd00e0: Break system packages (fix MacOS build) 10(79 seconds ago, 1 file, 10+ 1-) 13https://github.com/crawl/crawl/commit/abb3cd00e002 13:19:11 <06m​umra> Well the build still works 🤷 14:25:33 03dolorous02 07* 0.32-a0-1046-g9e7e224b41: Add deck of punishment lines for goblin sharper. 10(19 minutes ago, 1 file, 8+ 0-) 13https://github.com/crawl/crawl/commit/9e7e224b41b4 14:25:33 03dolorous02 07* 0.32-a0-1047-g293f3812c8: Reorganise goblin sharper's card lines. 10(7 minutes ago, 1 file, 58+ 40-) 13https://github.com/crawl/crawl/commit/293f3812c8d4 14:25:33 03dolorous02 07* 0.32-a0-1048-g1bd1766845: Reorder a few goblin sharper lines. 10(5 minutes ago, 1 file, 4+ 4-) 13https://github.com/crawl/crawl/commit/1bd176684574 14:31:05 04Build failed for 08master @ 1bd17668 06https://github.com/crawl/crawl/actions/runs/8930679714 14:33:00 New branch created: pull/3794 (1 commit) 13https://github.com/crawl/crawl/pull/3794 14:33:01 03mumra02 07https://github.com/crawl/crawl/pull/3794 * 0.32-a0-1046-g51b5d5de19: Balloon Yak: new Zot monster (regret-index) 10(9 minutes ago, 19 files, 115+ 14-) 13https://github.com/crawl/crawl/commit/51b5d5de1928 14:36:40 <06m​umra> Should i merge that CI fix? 14:43:19 hmm a most definite maybe on that ... 14:50:03 I would say so 14:50:36 that said, I'm not a committer (to this project, at least) 15:00:41 03mumra02 {Peter Hurst} 07* 0.32-a0-1049-gd495c7ad2a: Break system packages (fix MacOS build) 10(2 hours ago, 1 file, 10+ 1-) 13https://github.com/crawl/crawl/commit/d495c7ad2afe 15:04:57 <06m​umra> Somebody with a mac would have to confirm the build is actually working. But pipelines not failing is good! 15:37:50 Unstable branch on underhound.eu updated to: 0.32-a0-1048-g1bd1766845 (34) 16:10:43 -!- rozlav7 is now known as rozlav 22:35:25 Unstable branch on crawl.develz.org updated to: 0.32-a0-1049-gd495c7ad2a (34) 22:58:06 Windows builds of master branch on crawl.develz.org updated to: 0.32-a0-1049-gd495c7ad2a 23:13:11 Unstable branch on cbro.berotato.org updated to: 0.32-a0-1049-gd495c7ad2a (34) 23:55:12 Monster database of master branch on crawl.develz.org updated to: 0.32-a0-1049-gd495c7ad2a