On Wed, 2007-08-15 at 16:04 +0200, Peter Baumann wrote: > On Wed, Aug 15, 2007 at 02:27:28PM +0100, Alex Bennee wrote: > > Hi, > > > > It turns out CVS doesn't always give the status output in the order > > requested. According to my local CVS gurus this is a known CVS issue. > <snip> > I inlined the patch for easier commenting. Please inline further > patches. Will do. I assumed Evolution would do something sensible. My mistake :-( > > > --- > > git-cvsexportcommit.perl | 30 ++++++++++++++++++++---------- > > 1 files changed, 20 insertions(+), 10 deletions(-) > > > > diff --git a/git-cvsexportcommit.perl b/git-cvsexportcommit.perl > <snip> > This is extremly wastefull, because it will spawn a CVS process for each file. > A better fix would be to parse the filename from the output of > 'cvs status' and use that as input for $cvsstat. > > (And/or you could use an hash instead of an array for 'cvsoutput', so > you could double check that you only get the status for those files you > asked for.) I agree it's wasteful and could be done better however I'm no perl hacker so I just went for something that was correct and worked. The path that is echoed later in the status output is however the CVS file path which may not be directly related to the actual path in your source tree. For example I have one status reported as: $ cvs status src/proj_version =================================================================== File: proj_version Status: Up-to-date Working revision: 1.1.380.1 Repository revision: 1.1.380.1 /export/cvsroot/project/src/Attic/proj_version,v Sticky Tag: ATAG (branch: 1.1.380) Sticky Date: (none) Sticky Options: (none) This makes the matching more than a little problematic. It depends on how much people that use this script care about performance? For my part it's a fire and forget script once I've finished my hacking in a git tree so I don't mind it taking some time. I'm not particularly minded to dig further in perl to make it faster unless there is a real clamour - or perhaps someone with a bigger itch and more perl foo can tackle it. In the meantime it does fix a bug in the script so I would say it's applying. -- Alex, homepage: http://www.bennee.com/~alex/ Blessed is he who has reached the point of no return and knows it, for he shall enjoy living. -- W. C. Bennett - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
