[PATCH] send external diff to a file (bug 4047)
James Henstridge
james.henstridge at gmail.com
Fri May 5 08:43:18 BST 2006
On 5/5/06, Martin Pool <mbp at sourcefrog.net> wrote:
> On 05/05/2006, at 5:12 PM, James Henstridge wrote:
> > If the recommended style has changed, I can fix things up.
>
> The general rule is to put them at the top level unless either:
>
> - importing them later would substantially reduce the cost
> - or importing them later is the best way to resolve a dependency
> cycle
Okay. I'll move the imports outside of external_diff(). From what I
can tell the two modules in question get imported unconditionally in
most cases anyway:
* tempfile in bzrlib/osutils.py
* subprocess in bzrlib/gpg.py which gets imported unconditionally by
bzrlib/repository.py
I've moved the imports out of external_diff() in my branch now (at
revision 1696).
James.
More information about the bazaar
mailing list