-
knaccc
in the GUI, if you send a tx and for the amount choose "send all", does the change output go back to your wallet, or are both outputs sent to the destination?
-
knaccc
or is the change output a zero output sent to a dummy address
-
selsta
should be same as CLI
-
sqozz
hey all. I'm currently trying to compile monero-gui on my gentoo box. Apparently my cmake is to new so I can't build the latest release (0.16.3) but have to switch to master. I was playing around and realized that librandomx can't be found because it gets installed into monero/lib64. After copying it to monero/lib the build continues but now throws errors about a missing reference to
-
sqozz
monero_crypto_amd64_64_24k_generate_key_derivation and I started to wonder if there is any supported way to build master and not a release
-
selsta
did you follow the readme?
-
selsta
what command did you enter?
-
sqozz
selsta: yeah, following the readme was my first attempt :)
-
selsta
also which cmake version are you using? :D
-
sqozz
selsta: `LANG=C QT_SELECT=5 ./build.sh` most of the time. While playing around I was building single components with the according cmake+make commands just to speed debugging up but for the "final test" I always used build.sh
-
sqozz
cmake 3.16.5
-
selsta
I doubt that 3.16.5 is too new, I use 3.18.2
-
selsta
can you do a fresh git clone, enter ./build.sh and post logs?
-
sqozz
selsta: ah, I might have mixed something up. It was complaining about library_version_type which comes from boost (not cmake - my fault). Given
github.com/monero-project/monero/bl…fb3fa73c90/src/wallet/wallet2.h#L38 I figured I need to build from master
-
selsta
master should build fine
-
selsta
-
selsta
builds on all OS with cmake and qmake
-
selsta
hmm, I might know the problem
-
sqozz
selsta: slap me if I totally misunderstand something but the linked job also builds the latest tag/release, no? "You are building a tagged release"
-
selsta
-
selsta
change this to `git -C $MONERO_DIR checkout origin/master`
-
selsta
then it should work, the PR to change submodule to master is not merged yet
-
selsta
and the boost bugfix is only in master
-
sqozz
selsta: I commented the line in my attempt but then I end up with the issue I talked about in the beginning about a missing/wrongly installed librandomx.a (in monero/lib64 instead of monero/lib) :)
-
selsta
you "commented" the line?
-
selsta
can you try changing it to master? or did you manually change the submodule to master?
-
selsta
as a first step it might be a good idea to try to build monero-cli first
github.com/monero-project/monero
-
sqozz
as in "put a # in front of the line". And yes, afterwards I did a manual "git checkout master" for the monero submodule
-
selsta
did you do git fetch, git checkout origin/master ?
-
selsta
I had issues in the past when only doing `git checkout master`
-
sqozz
no, just a "checkout master". Let me try this exercise again :)
-
selsta
also make sure to delete your build directory before retrying to avoid any weird problems
-
sqozz
sure
-
sqozz
alright so with the checkout removed from get_libwallet_api.sh (
paste.opensuse.org/view/raw/4fb529f4) I'm back to the randomx issue. Here is a log of ./build.sh:
paste.opensuse.org/view/raw/6d90cb38
-
selsta
why `LANG=C` ?
-
selsta
I don’t really see an error in the logs (apart from the linking issue), weird.
-
sqozz
because I was stupid in the past and installed my system with a german locale
-
selsta
okay, different idea
-
selsta
remove the build folder and do `USE_SINGLE_BUILDDIR=ON DEV_MODE=ON make release -j3`
-
selsta
with master monero submodule checked out
-
sqozz
and the linking error is kind of expected given how librandomx got installed:
paste.opensuse.org/view/raw/2ba17912
-
sqozz
the build folder inside monero-gui you mean, right? And I guess I need "qmake ../monero-wallet-gui.pro CONFIG+=release" before "USE_SINGLE_BUILDDIR=ON DEV_MODE=ON make", no?
-
selsta
no
-
selsta
remove build folder inside monero-gui
-
selsta
and then just `USE_SINGLE_BUILDDIR=ON DEV_MODE=ON make`, qmake is not involved
-
selsta
we have two build systems currently because we migrate from qmake to cmake
-
sqozz
understood, running now
-
iDunk
FWIW, you are getting that error because libwallet-crypto.a is missing somewehere. It's a supercop submodule issue, not randomx.
-
sqozz
iDunk: yeah the missing reference seems to be provided by supercop. But since I had to tinker around so much to reach that point I figured I might miss something before which is causing this missing reference ;)
-
sqozz
selsta: unfortunately also no luck with make release, it now complains about QT_LINKED_OPENSSL:
paste.opensuse.org/view/raw/a5e18be8
-
sqozz
ah, but in /usr/include/qt5/Gentoo/gentoo-qconfig.h so most likely not related to monero-gui at all
-
selsta
sqozz: remove -Werror from CMakeLists.txt
-
selsta
-
selsta
but removing -Werror might work
-
sqozz
yeah, also found qt5-build.eclass
-
sqozz
-
sqozz
with the define removed in /usr/include/qt5/Gentoo/gentoo-qconfig.h it was able to build successfully \o/ thanks a lot for your help
-
selsta
nice, though not sure why it failed with qmake, does not matter I guess
-
sqozz
if you're in the progress of switching away it really doesn't matter
-
grydz
Hi everyone, what's the release date of the Monero client 0.17.0.0?
-
hyc
Soon.
-
sethsimmons
<grydz "Hi everyone, what's the release "> Hey grydz , the goal is by 9/17
-
sethsimmons
Not sure if there have been changes, but that's been the goal for a while now.
-
sarang
Weekly meeting in #monero-research-lab is today at 17:00 UTC (about 90 minutes from now)
-
rbrunner
Is the testnet fork height already fixed? And where in the source are the fork heights defined?
-
hyc
not yet
-
hyc
and src/hardforks/hardforks.cpp
-
hyc
probably worth setting the testnet height soon tho
-
sarang
yes
-
rbrunner
Thanks, hyc. I was searching everywhere except for its own folder ... I also think forking soon is probably a good idea. After all, we had September 17 as tentative date for the release :)
-
sarang
I'm sure the Ledger and Trezor teams need to set this in their code somehow?
-
rbrunner
Sounds logical. I don't think you can query the network for hardfork heights, or can you?
-
hyc
don't recall offhand
-
hyc
I would assume there's a daemon rpc for fork info
-
grydz
Thanks sethsimmons. Would be great if the hardfork height for v13 on testnet could be set to approximately the release date of 0.17.0.0!
-
moneromooo
Waiting till mainnet switch would defeat most of the point, would it not.
-
grydz
Sure ;)
-
rbrunner
Yeah, and whatever happened to beta-testing ...
-
hyc
yeah, testnet ought to have forked ... right around now
-
moneromooo
I wanted to fork it shortly after it got merged, but forgot. I'll PR now.
-
moneromooo
Forking in 24 hours is fine ?
-
hyc
sounds ok to me
-
hyc
anyone who is likely to squawk is probably in this channel
-
moneromooo
PRed, but still building.
-
hyc
ok I'm building it now too
-
hyc
hmm, haven't fired up testnet in ages
-
hyc
my monerod on ubuntu still has undef'd references to libgss_krb5
-
hyc
I thought someone patched this in cmake already?
-
xmr-pr
moneromooo-monero opened pull request #6794: hardforks: add v13/v14 for testnet
-
xmr-pr
-
moneromooo
Rings a bell.
-
asymptotically
I think I scared them away by saying they should use zmq's pkgconfig file instead of just adding -lgss_krb5
-
hyc
hm, that's a good point
-
rbrunner
That's in about 24 hours then
-
rbrunner
Ah, saw you wrote it
-
rbrunner
Nothing to squawk from me :)
-
sarang
Quick update: the Trezor team plans to make a Monero codebase PR for Trezor support tomorrow
-
sarang
but otherwise it sounds like they will be ready to go
-
vtnerd_
selsta : I didn't use ASM speedupds for node synching at luigi's suggestion. the concern was over multiple ed25519 implemenations and possible chain forking
-
selsta
knaccc: ^
-
vtnerd_
it would suggest a bug in either implementation, which is pretty serious, but I then decided to do the "safe" approach and only use it for wallets
-
vtnerd_
this of course can be changed, but thats why its only used for wallets atm
-
knaccc
vtnerd_ oh cool, so there is a speedup for wallet scanning but not for node tx verification
-
selsta
one idea would be to use ASM for initial sync and then switch to normal EC lib
-
selsta
as speed is only really relevant for the initial sync
-
vtnerd_
there's also the possibility of extending stuff with SIMD ops - the ASM assumes base x86-64 only. I haven't done much investigation into that though; donna64 has some SSE2 speedups that bump it above the ASM we use
-
selsta
so even more speeds up theoretically possible, nice :)
-
selsta
I think that makes more sense to investigate for now than GPU scanning
-
moneromooo
Interesting idea selsta.
-
vtnerd_
ok, just took a quick looks. I think all of the accelleration is in choose_t SIMD ops, which is the obvious thing I was going to try
-
selsta
moneromooo: would there be any major risks left?
-
vtnerd_
also arm64 has the advantage of guaranteeing more SIMD instructions than base x86-64 iirc
-
selsta
if a different result gets calculated it would simply stop syncing?
-
moneromooo
Well, a different result for those blocks :)
-
moneromooo
But at least it's when people are likely to pay attention, not after htey leave their node unattended for a long while.
-
moneromooo
Yes.
-
vtnerd_
also, with apple moving to arm, arm64 speedups could be useful