libera #monero-dev

5 Sep 2022

Official repo: github.com/monero-project/monero || Don’t ask to ask, just ask:) || Monero GUI dev is at #monero-gui || Monero website dev is at #monero-site || Non-dev questions -> #monero, please || No fork help, you’re on your own

• 00:00
  MeowingCat
  is it possible to get TXs with wallet2 API?
• 00:17
  THECOCO
  DeanGuss, did I hear you say Monero was merging with wownero?
• 00:17
  THECOCO
  this can't be so
• 00:26
  UkoeHB
  THECOCO: merge-mining?
• 00:28
  THECOCO
  My man, I heard Monero was going to some crypto routine written by wownero developers in collaboration with verge devs!
• 00:29
  UkoeHB
  news to me
• 00:37
  THECOCO
  rehrar and friends were discussing this in some blog video. I'm hoping it isn't official.
• 00:37
  THECOCO
  unless it was somehoe well-tested\
• 08:21
  MeowingCat
  i feel like wallet2 is not getting new blocks
• 08:21
  MeowingCat
  why??????
• 09:00
  MeowingCat
  what could be the reason?
• 09:50
  MeowingCat
  2022-09-05 09:01:17.766 E wrong variant type: N10cryptonote12txout_to_keyE, expected txout_to_tagged_key in transaction id=<ac388d8a1d2aa4692b4513e62959273fbc6796d78990b23260931ce4657eac0a>
• 09:50
  MeowingCat
  what does this mean??
• 09:56
  MeowingCat
  wallet2 is crazy :((((((((
• 12:49
  arnuschky[m]
  Hey there - it's RINO again harping on about multisig ;)
• 12:50
  arnuschky[m]
  We've encountered #8537 which might lead to loss of funds in a multisig wallet. ukoe and j-berman went right at it and fixed it in lightspeed in PR #8545
• 12:51
  arnuschky[m]
  I am now wondering how long it'll take for this fix to percolate through to the end-user? Is there a point-release already planned, where this could go in? Or are we in "who the hell knows" territory?
• 12:56
  dEBRUYNE
  .1.1 is planned as far as I can see
• 12:56
  dEBRUYNE
  Would be prudent to include it there I suppose
• 12:56
  dEBRUYNE
  arnuschky[m]: monero-project/monero #8534
• 13:08
  arnuschky[m]
  thanks dEBRUYNE. Any ETA on this?
• 16:10
  dEBRUYNE
  arnuschky[m]: Not sure
• 16:10
  dEBRUYNE
  cc selsta
• 16:11
  selsta
  arnuschky[m]: I'm waiting for the output export PR to get approved
• 16:11
  selsta
  so release is planned in the next days
• 16:32
  jberman[m]
  <selsta> "arnuschky: I'm waiting for the..." <- Reviewing atm
• 17:02
  rbrunner
  Funny story with "Windows: fix unicode file path support #8547". How did the define change from "WIN32" to "_WIN32"? Side effect?
• 17:40
  selsta
  rbrunner: WIN32 only works when windows.h is included, which wasn't the case anymore after the code got moved around
• 17:59
  rbrunner
  Thanks, selsta, good to know.
• 18:31
  hyc
  yeah, _WIN32 is the only symbol any code should be using
• 21:03
  MeowingCat
  my wallet2 build is stucking at height 2688787
• 21:03
  MeowingCat
  im building Monero v0.18.1.0
• 21:15
  MeowingCat
  CMake Error at C:/proje/monero/build/MINGW64_NT-10.0-22000/_HEAD,_v0.18.1.0_konumunda_ayrıldı_/release/CMakeFiles/CMakeTmp/CMakeLists.txt:17 (add_executable):
• 21:15
  MeowingCat
  No SOURCES given to target: cmTC_e7018
• 21:15
  MeowingCat
  :(((((((((((
• 21:37
  MeowingCat
  i think Git is cauising something weird
• 21:37
  MeowingCat
  i cloned the repository again
• 21:37
  MeowingCat
  it is building now on v0.18.1.0 branch
• 22:31
  MeowingCat
  i was linking older Monero
• 22:31
  MeowingCat
  butttt something is changed in v0.18.1.0
• 22:31
  MeowingCat
  there is no libwallet_merged.a
• 22:32
  MeowingCat
  im linking libwallet.a instead
• 22:32
  MeowingCat
  but getting error
• 22:32
  MeowingCat
  undefined reference to `Monero::WalletManagerFactory::getWalletManager()'
• 22:59
  MeowingCat
  is there anyone here??????? :((((((((((((
• 23:17
  selsta
  MeowingCat: which version did you link previously?
• 23:20
  selsta
  MeowingCat: monero-project/monero #7694
• 23:20
  selsta
  try to undo this to get wallet_merged
• 23:21
  MeowingCat
  selsta, ohhh answer thank youuu
• 23:21
  selsta
  we don't use it really
• 23:22
  selsta
  that's why it was removed
• 23:24
  MeowingCat
  im getting linking errors undefined referencesssssss
• 23:24
  MeowingCat
  undefined reference to `Monero::WalletManagerFactory::getWalletManager()' :((((((((
• 23:25
  MeowingCat
  im linking libwallet.a of v0.18.1.0
• 23:25
  MeowingCat
  noww trying to undo that commit
• 23:27
  MeowingCat
  building.....
• 23:29
  MeowingCat
  selsta, still not making libwallet_merged.a :((((((
• 23:30
  selsta
  did you do a full clean build?
• 23:30
  MeowingCat
  oh no doing nowwww
• 23:39
  MeowingCat
  selsta, didn't make it :((((((((((((((((
• 23:40
  selsta
  that's the only change we had between v0.17 and v0.18
• 23:41
  selsta
  did you link libwallet or libwallet_merged?
• 23:41
  selsta
  i thought the issue was that libwallet_merged was missing?
• 23:48
  MeowingCat
  selsta, in v0.18 libwallet_merged.a is missing i tried to link libwallet.a but im getting undefined reference errors on linking
• 23:49
  selsta
  right, that's why said you should revert it and then link against libwallet_merged
• 23:55
  MeowingCat
  selsta, i reverted but still not making libwallet_merged.a :((((((
• 23:56
  MeowingCat
  oh damn
• 23:56
  MeowingCat
  it is checking for BUILD_GUI_DEPS and it is OFF default
• 23:56
  MeowingCat
  i set it to ON
• 23:56
  MeowingCat
  and building again
• 23:56
  MeowingCat
  i think you were using libwallet_merged.a for GUI wallet and since v0.18 you don't need it anymore
• 23:57
  selsta
  we use wallet_api instead
• 23:57
  MeowingCat
  where is that??
• 23:57
  MeowingCat
  is it same for wallet2_api.h interface?
• 23:57
  selsta
  monero-project/monero-gui #3448
• 23:57
  selsta
  it's a cmake target