[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