This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author eric.araujo
Recipients belopolsky, brett.cannon, eric.araujo, mark.dickinson, pitrou
Date 2010-06-22.10:56:26
SpamBayes Score 0.012246041
Marked as misclassified No
Message-id <1277204189.05.0.915922292968.issue8999@psf.upfronthosting.co.za>
In-reply-to
Content
In the absence of formal testing, a few comments on my patch to help
reviewers.

1) If the code is both an svn and hg checkout, hg wins, since it’s not unheard of to use hg on top of svn, whereas the contrary does not make sense to me.

2) The hg status command does not need post-processing like the svn one, since hg status --added --modified will already filter the output, and --no-status means “don’t print A or M at the start of each line”.

3) I changed a list comp for a genexp, and used startswith instead of getitem+contains+eq; does not change behavior.

4) I suppressed the unneeded shell=True argument to subprocess.Popen (and made the command string into a list); does not change behavior since there were no globbing or path expansion of any kind.

Testing is easy: Apply the patch, add bad indentation in a Python file, run make patchcheck (./python Tools/scripts/patchcheck.py on OSes without make), profit.
History
Date User Action Args
2010-06-22 10:56:29eric.araujosetrecipients: + eric.araujo, brett.cannon, mark.dickinson, belopolsky, pitrou
2010-06-22 10:56:29eric.araujosetmessageid: <1277204189.05.0.915922292968.issue8999@psf.upfronthosting.co.za>
2010-06-22 10:56:27eric.araujolinkissue8999 messages
2010-06-22 10:56:26eric.araujocreate