-
tobtoht_
.merges
-
xmr-pr
9938 9984 9989 9990 9992 9993 9995 10000 10001 10002 10014 10015
-
tobtoht_
.merges
-
xmr-pr
10000
-
tobtoht_
done
-
m-relay
<ack-j:matrix.org> @binaryfate would you be able to review this PR to provide fixes for the oss-fuz integration? Plowsof was the first reviewer but two are required
-
m-relay
-
m-relay
<ofrnxmr:monero.social> Asking binaryfate is crazywork :D
-
m-relay
<ofrnxmr:monero.social> I'd leave that invitation more "open"
-
m-relay
<ack-j:matrix.org> Anyone is welcome and encouraged to review but the github bot said “The PR must be approved by known contributors before it can be merged. The past contributors are: plowsof, maflcko, inferno-chromium, binaryFate”
-
m-relay
<ofrnxmr:monero.social> That pr is still marked as a draft
-
m-relay
<ofrnxmr:monero.social> i saw that, but does that mean known contributors to oss-fuzz? I think it would imply known contributors, including trusted members upstream
-
m-relay
<ofrnxmr:monero.social> Anyway, comments from me: 1. is the ci supposed to be green? 2. i dont like the merge commit 3. Its marked as draft
-
m-relay
<ofrnxmr:monero.social> [@xmrack:monero.social](https://matrix.to/#/@xmrack:monero.social)
-
m-relay
<ofrnxmr:monero.social>
google/oss-fuzz #12936
-
m-relay
<ofrnxmr:monero.social> you can see here that selsta and myself approved plowsof's contribution
-
m-relay
<ofrnxmr:monero.social> (you commented there as well). This just to say that the bot's recommendations arent concrete
-
m-relay
<ack-j:matrix.org> I believe CI fails since pr #10004 hasnt been pushed to master
-
m-relay
<ack-j:matrix.org> *merged
-
m-relay
<ofrnxmr:xmr.mx> That might be the same reason it's in draft (waiting for 10004?)
-
m-relay
<ack-j:matrix.org> Yea
-
m-relay
<ack-j:matrix.org> From that PR it looks like any monero contributor can approve. So I extend the invitation to everyone here :D
-
m-relay
<ofrnxmr:xmr.mx> Are the adalogics guys going to maintain this long term? I see they were added to the `auto cc` contacts alongside the fluffy, binary, luigi, and mooo.
-
m-relay
<ack-j:matrix.org> Their emails would only be temporary so they can thoroughly debug the new harnesses over the next few weeks
-
m-relay
<ack-j:matrix.org> I’ll paste my comment advocating for their addition a few MRL meetings ago:
-
m-relay
<ack-j:matrix.org> The benefit of adding david from adalogics (temporarily) to the email list of OSS-Fuzz is that his team will be able to more effectively troubleshoot the harnesses as they can potentially hit edge cases and need to be updated. Monero’s OSS-Fuzz integration has been neglected for a while now and having david (the #1 contributor to the OSS-Fuzz project) making sure everything is r<clipped message>
-
m-relay
<ack-j:matrix.org> unning smoothly would IMHO be a great asset to the monero community.
-
m-relay
<ofrnxmr:xmr.mx> tobtoht should the dependencies be cleaned up here (in dockerfile)
-
m-relay