On 14/02/2009 05:24, Jonathan Watt wrote: > +try: > + import subprocess > +except ImportError: > + pass Not sure this works in the face of demandimport. It would be better to add wrappers for this in mercurial.util, as we have for the various popens. Cheers, Dirkjan