On Wed, Jul 15, 2009 at 06:06:38PM -0400, Lucas Nussbaum wrote: > I think that this information should be stored outside of the patch (in the > history of a VCS, for example).
Can you explain how you envisage such a thing working, outside of the patch? Not all packages are maintained inside a VCS. The system I envisaged in my proposal was (back of the envelope) summary information generated like this * there are <X> debian packages, <Y>/<X> using a recognisable patching system, constitution <Z> patches. <W>/<Z> patches use the patch guidelines. * <V>/<W> of these patches have been reviewed. * <U>/<V> of these patches have been reviewed less than <T> days ago (where <U> is some number defined to try to describe acceptable patch "staleness") and a list of patches in need of review <W> - <V> and a list of patches last reviewed too long ago <V> - <U> And a system for people to search out such patches by package, maintainer, or other criteria, to facilitate reviewing. Such figures could be determined purely from a source archive if the information was encoded in the headers. It would not be feasible I don't think to do something like this if the system needed to query lots of different types of VCSes or other systems too. I suppose a system could be put together that stored all of the patch review data inside of itself. -- Jon Dowland -- To UNSUBSCRIBE, email to debian-devel-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org