[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #27164 [Core Tor/Tor]: rust protover accepts "Link=1-5-foo"
#27164: rust protover accepts "Link=1-5-foo"
----------------------------------------------+----------------------------
Reporter: cyberpunks | Owner: (none)
Type: defect | Status:
| needs_revision
Priority: Medium | Milestone: Tor:
| 0.3.5.x-final
Component: Core Tor/Tor | Version: Tor:
| 0.3.3.9
Severity: Normal | Resolution:
Keywords: rust, 033-backport, 034-backport | Actual Points:
Parent ID: | Points:
Reviewer: teor | Sponsor:
----------------------------------------------+----------------------------
Changes (by teor):
* status: needs_review => needs_revision
* reviewer: => teor
Comment:
Thanks for this patch, it looks good to me.
Here's what needs to change before it merges:
* code changes need a changes file that passes `make check-changes`
* (doc and comment changes don't get a changes file)
* the branch needs to be rebased onto maint-0.3.3
Let us know if you don't want to make these changes, and someone else will
do them eventually.
And also:
* we need to wait for the CI to pass:
* I have pushed your branches to my github, which is set up to run
travis
* https://travis-ci.org/teor2345/tor/builds/417209334
* if you want to run travis automatically, you can configure it on
gitgud, or submit a pull request to https://github.com/torproject/tor
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/27164#comment:3>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs