"Robert Blum" <rob.blum@xxxxxxxxx> writes:
> P4 on Windows expects the PWD environment variable to be set to the
> current working dir, but os.chdir in python doesn't do that by default
Missing full stop at the end of sentence aside, this comment makes me
wonder if there is an optional way to have it set it, as opposed to the
inconvenient way it behaves "by defualt". If there is none, I think your
patch, even though it looks ugly, is the least evil approach. Another way
might be to wrap callsites of system() by introducing a "run_p4" function,
like:
def run_p4(arg):
os.environ['PWD'] = os.getcwd() if os.name == 'nt'
return system(arg)
> ---
>
> Pushing it out to the list since I'm not entirely sure who the git-p4 owner
> even is. CC'ed likely suspects for ownership ;)
Thanks. I've been waiting for an Ack from somewhere or success reports
from p4 users on Windows.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
