[MERGE] More than double the speed of "bzr selftest"

Martin Pool mbp at sourcefrog.net
Tue Jun 12 00:37:12 BST 2007


On 6/12/07, John Arbash Meinel <john at arbash-meinel.com> wrote:
> John Arbash Meinel has voted -0.
> Status is now: Approved
> Comment:
> Updating my vote because of interactions with sys.exit(). If those are
> addressed, this looks good.
>
> My only concerns are:
>
> 1) leaving stuff cluttering $TEMP if the rmtree fails

Thanks for catching that; I should have realized seeing as I did add
the exit hack myself.  Is it ok with you if I use Andrew's approach of
explicitly running  those hooks before we exit (assuming that works)?

> 2) running out of path space on Win32 because IIRC, TEMP is in:
>      C:\Documents and Settings\<username>\Local Settings\Temp
>
> However, if we are going to leave clutter, it is best to put it in '.'
> or TEMP, so I'm okay with this.

My current version of the patch actually does

        candidate_dir = tempfile.mkdtemp(dir=self.TEST_ROOT)

so the top level directory names should be pretty small - 12
characters or so.  We may create something inside their  that uses
long paths, but that's probably a bug anyhow.

> Ian- do you want this for 0.17? As a change to the test infrastructure I
> consider it safe enough, but that is my personal opinion.

I'll go along with Ian's decision to leave it until after 0.17 -- we
should be branching today.


-- 
Martin



More information about the bazaar mailing list