-
selsta
.merge+ 3684
-
xmr-pr
Added
-
tauntaum
hello
-
tauntaum
i had a segfault yesterday
paste.debian.net/1210153
-
tauntaum
any help?
-
moneromooo
That looks related to a recent race fix, though not quite the same stack I've seen.
-
moneromooo
IIRC you paste the link to the PR yesterday for something else.
-
tauntaum
this segfault is strange because happens on 4 of 6 nodes that i have
-
tauntaum
2 of nodes dont crash at all!
-
tauntaum
i had connection problems and desperated i "did the hacker" as u said
-
tauntaum
and gdb revealed this
-
tauntaum
can we fix this because my4nodes will be useless!
-
moneromooo
Same as any other patch: patch -p1 or git am
-
moneromooo
Actually that should be in the recent release I expect.
-
moneromooo
So maybe get that.
-
tauntaum
patch number?
-
moneromooo
I don't have it here. You posted it yesterday.
-
moneromooo
(I think it was you ?)
-
tauntaum
-
tauntaum
this one
-
tauntaum
-
tauntaum
moneromooo 7873 ?
-
moneromooo
Yes, 7873
-
tauntaum
thnx
-
tauntaum
moneromooo what to do when 2 commits conflict
-
tauntaum
7873 and 7803
-
tauntaum
i add before 7803 and now the only way is to remove 7803 if i want to insert 7873
-
moneromooo
You look at the diffs and fix so it is what it should be.
-
tauntaum
im not c++ guru
-
moneromooo
I can have a quick look if you paste the conflict.
-
moneromooo
to paste.debian.net please
-
moneromooo
All the text between <<<< and >>>>
-
tauntaum
-
tauntaum
when i add again waht is need i have
paste.debian.net/1210241
-
moneromooo
No context. I'll try to do it for you. 7873 and 7803 on top of master ?
-
tauntaum
yes
-
tauntaum
u want all file?
-
tauntaum
after being patched
-
moneromooo
Just wait. I'll apply those both and send you a merged patch.
-
» moneromooo starting dev vm
-
tauntaum
ok
-
moneromooo
Yes, bad luck, half a dozen conlicts :/
-
tauntaum
both this patches are very important
-
tauntaum
what to do
-
moneromooo
Wait. I'm trying to resolve the conflicts.
-
tauntaum
ok
-
moneromooo
ffs, I hate those double refs, I never know how to use them -_-
-
tauntaum
hmm
-
tauntaum
why 7803 is not inserted yet
-
tauntaum
is very important i think
-
moneromooo
OK, sorry, too much work to go through for code I don't know.
-
moneromooo
Ask either vtnerd or wfaressuissia in #monero-dev, they're the authors of each so they'll at least know one of the patches.
-
tauntaum
ok thank u anyway
-
moneromooo
Ask the author of th epatch that's not merged yet, if one is merged.
-
tauntaum
no one is merged until now
-
moneromooo
Odd, I'd have thought the race would be important to get in.
-
tauntaum
if u c here
monero-project/monero #7873 vtnerd knows that patches has conflicts
-
tauntaum
-
tauntaum
-
tauntaum
-
moneromooo
Maybe. Ask one of the patch creators.
-
tauntaum
yes im asking no one is answering
-
moneromooo
vtnerd will likely answer within a couple days, he's on and off.
-
tauntaum
-
selsta
15:54 <moneromooo> Odd, I'd have thought the race would be important to get in. <-- stuck in review
-
tauntaum
selsta yes i took the patch and run but nothing chnaged!
-
tauntaum
same error
-
tauntaum
0x00007ffff7ce54cf in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:95
-
tauntaum
i think that is smth else
-
tauntaum
some libc wrong version etc
-
selsta
ignore that clone.S thing
-
tauntaum
node crashes
-
tauntaum
why?
-
selsta
it must be something on your local enviroment
-
selsta
environment*
-
tauntaum
i have all debian
-
selsta
no one else reports these crashes
-
tauntaum
in 4 of 6 nodes crashes
-
tauntaum
lack of memory?
-
selsta
.merge+ 3685
-
xmr-pr
Added
-
selsta
.merge+ 3682
-
xmr-pr
Added
-
selsta
.merges
-
xmr-pr
3096 3282 3452 3471 3472 3473 3477 3481 3572 3679 3682 3683 3684 3685
-
selsta
w0w so many prs
-
selsta
lots of work for luigi1111w
-
luigi1111w
wen next release
-
selsta
tomorrow
-
selsta
rating89us: can you check 3435 again? I updated it, should work now
-
rating89us
ok, will check soon
-
selsta
.merge+ 3435
-
xmr-pr
Added
-
selsta
.merge+ 3686
-
xmr-pr
Added