-
m-relay
<vtnerd:monero.social> rsch: I'd have to dig into the specific code. Most likely the keys file changed to the new password but the cache file didn't. The code is supposed to detect when the keys file is selected, but maybe it's buggy. I would make copies of all related files, and then try changing the password of the keys file to the original password, with the goal of making the two in sync again
-
m-relay
<nobfg9000:matrix.org> when FCMP++ rolls out, will there still be timelocks?
-
m-relay
<jeffro256:monero.social> User-specified timelocks are deprecated and will be (or at least should be) prohibited by consensus rule at the next fork, but there will still be the default 10-block-lock for non-coinbase transactions, and the 60-block-lock for coinbase transactions
-
rsch
hi
-
rsch
I would lke to contribute to the source code base. I have found a bug that I would like to go to fix. is there any documentation about the the branches?
-
rsch
how can I checkout the most recent version for testing a bug?
-
moneromooo
master is where most new stuff goes. Branching is made from master for every major release, to have some stability for subsequent minor releases.
-
rsch
as usual I would create a branch for a bug fix and make a PR. is that ok?
-
rsch
I mean from master
-
rsch
or is there a more recent branch to use?
-
moneromooo
If the bug you're fixing is consequential enough, it'll go to both master and the current release branch. If not, master only. Unless it's a bug in code that's only on the current branch, in which case it goes to the branch only.
-
moneromooo
Yes, you do that. This local branch is only on your repo, you manage that however you see fit.
-
rsch
am i wrong to say that the current cli release is since 2023?
-
moneromooo
Sounds old.
-
rsch
yea I am suprised that the bug is noticed so late
-
moneromooo
I see march 2025.
-
moneromooo
commit f1311d4237404ab7da76241dbf10e92a65132cc4 (tag: v0.18.4.0)
-
rsch
CLI and GUI 0.18 "Flourine Fermi" released 2022 july. I guess it referes to the major changes
-
moneromooo
Yes. There were several releases from the 0.18 branch. Last one a couple months ago.
-
rsch
but in simplewallet the last commit is from Sun Nov 24 20:39:26 2024 +0000
-
rsch
do you plan to move to rust in the future?
-
moneromooo
Some future parts might be coded in rust, depending on how maintainable it is, how easy to interface, whether it's in place with an attack surface, etc.
-
moneromooo
"how maintainable" here really means "whether existing monero hackers are conversant in rust" in large part.
-
moneromooo
It's pointless to merge a big rust thing only for the contributor to fade away later.
-
moneromooo
(if noone else around groks rust)
-
rsch
I agree
-
rsch
is there anything the project needs help with most? I see open issues since 2023
-
moneromooo
I believe code review for existing long lived patches. Not fun though.
-
rsch
oh yes there are 250 pr
-
moneromooo
You could also ask in #no-wallet-left-behind, I believe they're rewriting large parts of the wallet code (and hopefully not adding new bugs).
-
moneromooo
There's also the monero gui (no idea how active it is nowadays, see #monero-gui, C++/JS/QML), and haveno, a p2p exchange meant for monero first (#haveno-development, java).
-
rsch
I wonder how active the monero dev is when you have open pr since 2020
-
moneromooo
Well, feel free to tag along and find out.
-
moneromooo
But git log will tell you a lot.
-
m-relay
<ofrnxmr:xmr.mx> @rsch "I have found the bug and will prepare a PR" << did you find the issue?
-
rsch
yes I have a fix already
-
m-relay
<ofrnxmr:xmr.mx> 1. Build branch master, see if you can reproduce on master. If yes, open pr against master
-
m-relay
<ofrnxmr:xmr.mx> 2. Build branch release-v0.18, see if you can reproduce on release-v0.18. If yes, open pr against release-v0.18
-
rsch
ok sure
-
rsch
ofrnxmr: PR pushed
-
rsch
for both branches
-
m-relay
<ofrnxmr:xmr.mx> thanks. nits: change commit name to `simplewallet: cli using wrong filename for storing keys`, append `[0.18]` to the end of the release pr title, and add "closes #9943" to the pr description of master pr
-
m-relay
<binarybaron:matrix.org> Isn’t this really bad for Layer 2 protocols. Bidirectional Atomic Swaps would require timelocks.
-
m-relay
<ofrnxmr:xmr.mx> why would they require timelocks
-
m-relay
<ofrnxmr:xmr.mx> Also, timelocks are already deprecated
-
m-relay
<ofrnxmr:xmr.mx> if you mean "xmr as leader aka first transaction in the swap", xmr doesnt have scripting. What does a timelock have to do with anything
-
plowsof
Are the timelocks in Hash Time Locked Contracts the same timelocks :)
-
rsch
ofrnxmr: applied
-
m-relay
<rbrunner7:monero.social> Once again, because it's such a joy: The timelocks Monero has / soon will not have anymore are **not** suitable to implement atomic swaps.
-
m-relay
<rbrunner7:monero.social> Getting rid of them does not prevent implementing bidirectional atomic swaps, they are useless for that.
-
m-relay
<rbrunner7:monero.social> That we do not have the **right* timelocks is a possible reason to prevent that.
-
selsta
rucknium:
monero-project/monero #9946 there will still be a 2 weeks or so before I aim to tag, but since you asked recently