[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

Re: [tor-bugs] #29792 [Core Tor/Tor]: practracker problems and CI broken on master



#29792: practracker problems and CI broken on master
------------------------------------------+--------------------------------
 Reporter:  asn                           |          Owner:  (none)
     Type:  defect                        |         Status:  new
 Priority:  High                          |      Milestone:  Tor:
                                          |  0.4.1.x-final
Component:  Core Tor/Tor                  |        Version:
 Severity:  Normal                        |     Resolution:
 Keywords:  practracker tech-debt tor-ci  |  Actual Points:
Parent ID:                                |         Points:
 Reviewer:                                |        Sponsor:
------------------------------------------+--------------------------------
Changes (by asn):

 * status:  needs_review => new


Comment:

 Potential solutions to the issue where practracker can break master, when
 we merge-forward bad commits from earlier releases (and the PR was on
 those earlier releases which don't contain practracker):

 a) Add inline comments to denote practracker exceptions. Negative: Ugly.
 b) Add practracker to all the active releases. Negative: This will still
 not solve the problem, since we would need to open PRs for all releases
 everytime we merge-forward to somewhere.
 c) Add practracker as a post-commit git hook. Negative: Might be annoying.
 Not all volunteers will know about that thing.
 d) Disable practracker in CI. Negative: We lose all the positive aspects
 of practracker that will probably be ignored.

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/29792#comment:4>
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