-
gingeropolous
for some reason i read the .merges as the voice of mario saying "versus" in N64 mario kart.
-
gingeropolous
is there a --no-trezor flag for make ...
-
jeffro256
Not that I know of
-
jeffro256
What are you trying to do?
-
jeffro256
libwallet won't compile without Trezor AFAIK
-
gingeropolous
trying to compile master with those recent mergers
-
jeffro256
On what platform?
-
gingeropolous
i commented out all the trezor lines in CMakeLists.txt ..
-
gingeropolous
i think this is ubuntu 18
-
gingeropolous
yeah
-
gingeropolous
huh. doesn't work on ubuntu 22, and i didn't fiddle with the cmakelist there.
-
gingeropolous
ModuleNotFoundError: No module named 'psutil'
-
jeffro256
`pip install psutil`
-
gingeropolous
i guess the same goes for monotonic, requests, zmq and deepdiff?
-
jeffro256
yup!
-
gingeropolous
i guess the ubuntu 1 liner needs updating
-
jeffro256
Yeah probably
-
jeffro256
It's in the tests README tho
-
gingeropolous
welp those might not have been my problem after all.
paste.debian.net/1304582
-
gingeropolous
CMP0148 not known to this version... perhaps sudo update and all that
-
gingeropolous
nope. me can't compile :(
-
jeffro256
Same here: I can't compile :/
-
jeffro256
-
jeffro256
The CMake version was bumped in this PR
monero-project/monero #9076
-
jeffro256
Even on Ubuntu 23, my CMake version is only 3.25.1, so this policy isn't available to me.
-
jeffro256
Writing a patch
-
jeffro256
gingeropulous: does this help:
monero-project/monero #9126?
-
moneromooo
It's a warning. The python modules are optional. If not found, the relevant tests are just disabled.
-
gingeropolous
that did it. thanks jeffro256 !
-
selsta
-
selsta
feels currently the PR does more changes than necessary
-
m-relay
<jeffro256:monero.social> I still want to rollback the minimum version required changes
-
m-relay
<jeffro256:monero.social> But I could squash the commits if that's what ur asking
-
m-relay
<detherminal:matrix.org> Does anyone have idea how tail emission reward is choosen, why 0.60 XMR? How is it calculated?
-
moneromooo
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.
-
selsta
jeffro256: our mininum required cmake version is 3.5
github.com/monero-project/monero#dependencies
-
moneromooo
You may want to lookup posts by ArticMine on bitcointalk, for keywords "gold" "tail", it's likely to find something authoritative.
-
selsta
it being 3.1 somewhere was likely an oversight and it should be charged to 3.5
-
selsta
changed
-
xFFFC0000
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:
monero-project/monero #9126#discussion_r1459160012
-
m-relay
<jeffro256:monero.social> Okay good point , didn't know it was 3.5, will change
-
selsta
luigi1112: please also merge 9126, some people have build issues
-
selsta
.merge+ 9126
-
xmr-pr
Added
-
luigi1112
ok
-
selsta
ty, tobtoht can you force push 9128 to restart CI?