01:30:54 for some reason i read the .merges as the voice of mario saying "versus" in N64 mario kart. 01:32:30 is there a --no-trezor flag for make ... 01:37:07 Not that I know of 01:37:14 What are you trying to do? 01:38:50 libwallet won't compile without Trezor AFAIK 02:37:04 trying to compile master with those recent mergers 02:38:41 On what platform? 02:38:47 i commented out all the trezor lines in CMakeLists.txt .. 02:38:57 i think this is ubuntu 18 02:39:15 yeah 02:43:55 huh. doesn't work on ubuntu 22, and i didn't fiddle with the cmakelist there. 02:44:00 ModuleNotFoundError: No module named 'psutil' 02:50:14 `pip install psutil` 02:52:03 i guess the same goes for monotonic, requests, zmq and deepdiff? 02:52:13 yup! 02:52:24 i guess the ubuntu 1 liner needs updating 02:52:49 Yeah probably 02:53:30 It's in the tests README tho 02:54:28 welp those might not have been my problem after all. https://paste.debian.net/1304582/ 02:54:50 CMP0148 not known to this version... perhaps sudo update and all that 02:58:12 nope. me can't compile :( 03:13:20 Same here: I can't compile :/ 03:15:38 https://cmake.org/cmake/help/latest/policy/CMP0148.html 03:16:55 The CMake version was bumped in this PR https://github.com/monero-project/monero/pull/9076 03:19:06 Even on Ubuntu 23, my CMake version is only 3.25.1, so this policy isn't available to me. 03:36:49 Writing a patch 04:01:33 gingeropulous: does this help: https://github.com/monero-project/monero/pull/9126? 07:50:59 It's a warning. The python modules are optional. If not found, the relevant tests are just disabled. 12:53:37 that did it. thanks jeffro256 ! 13:28:11 jeffro256: can you just change these 3 lines in your PR without reverting the patch first? https://github.com/monero-project/monero/pull/9126/files#diff-1e7de1ae2d059d21e1dd75d5812d5a34b0222cef273b7c3a2af62eb747f9d20aR43-R45 13:28:31 feels currently the PR does more changes than necessary 16:06:17 I still want to rollback the minimum version required changes 16:06:44 But I could squash the commits if that's what ur asking 16:24:50 Does anyone have idea how tail emission reward is choosen, why 0.60 XMR? How is it calculated? 16:31:53 IIRC it was what made inflation < 1% the first year. In turn, I believe this was chosen to not made inflation higher than gold's. I do not have very high confidence that I am correct, however. 16:32:57 jeffro256: our mininum required cmake version is 3.5 https://github.com/monero-project/monero#dependencies 16:33:15 You may want to lookup posts by ArticMine on bitcointalk, for keywords "gold" "tail", it's likely to find something authoritative. 16:33:23 it being 3.1 somewhere was likely an oversight and it should be charged to 3.5 16:33:28 changed 16:52:11 jeffro256: quick point. We don’t need to revert. If we just check the policy with this syntax. The cmake we have will work for older versions and newer versions without any issues: https://github.com/monero-project/monero/pull/9126#discussion_r1459160012 17:11:12 Okay good point , didn't know it was 3.5, will change 17:17:22 luigi1112: please also merge 9126, some people have build issues 17:17:26 .merge+ 9126 17:17:26 Added 18:46:41 ok 18:48:48 ty, tobtoht can you force push 9128 to restart CI?