On Aug 6, 2007, at 1:51 PM, Phillip J. Eby wrote:
At 11:31 AM 8/6/2007 -0600, Jeff Shell wrote:
On 8/6/07, Phillip J. Eby email@example.com wrote:
At 07:57 AM 8/6/2007 -0600, Jeff Shell wrote:
A basic fix appears to be easy: add another `re_finder` for `CVS/Entries.Log` with a regex like ``^A\s\w?/([^/]+)/``.
I'd add this, but since I don't have data to test it on and you apparently do, please send a tested patch and I'll add it to the
By "tested patch", do you want a unit test to accompany the patch? Or do you just want it to have been tested by me and compared against the current release?
This would be a mistake. The former. :)
However, there are no existing tests for this feature, so there's no place to plug in. It would require generating fake packages with fake CVS (and complementary tests should do subversion) files and directories and yada yada yada. I looked and looked for something I might use as a starting point. Since I know next to nothing about the overall structure of setuptools and distutils, I feel rather unqualified to write such a test completely from scratch in the time I have available.