fast-export-from-git doesn't find git

Bug #418792 reported by Erik Hahn
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Bazaar Fast Import
Fix Released
Medium
Ian Clatworthy

Bug Description

bzr fast-export-from-git doesn't find git although version 1.6 installed in %PATH%:

D:\tmp\ordnung>bzr fast-export-from-git . ordnung.fi
bzr: ERROR: git missing. Please install Git 1.6 or later and try again.

D:\tmp\ordnung>git --version
git version 1.6.2.1217.gd7bc3

D:\tmp\ordnung>dir /b c:\Programme\Git\cmd (This directory is listed in %PATH%)
git.cmd
gitk.cmd

Related branches

Revision history for this message
Erik Hahn (erikhahn) wrote :

I've found that replacing "git" by "git.cmd" in /exporters/__init.py removed the error, apparently subprocess cannot handle windows scripts.

Revision history for this message
John A Meinel (jameinel) wrote : Re: [Bug 418792] Re: fast-export-from-git doesn't find git

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Erik Hahn wrote:
> I've found that replacing "git" by "git.cmd" in /exporters/__init.py
> removed the error, apparently subprocess cannot handle windows scripts.
>

I believe it can, but you have to pass "shell=True" for that to work.

The other alternative is to rename "git.cmd" into "git.exe"...

John
=:->

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Cygwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEUEARECAAYFAkqUTpUACgkQJdeBCYSNAAP0owCY9h1uEr1sROB7QlqjbsonUeyo
FQCffiWk+N6Kl4Czkjct5zc/uNwM6Nk=
=kQx/
-----END PGP SIGNATURE-----

Revision history for this message
Ian Clatworthy (ian-clatworthy) wrote :

John A Meinel wrote:
> Erik Hahn wrote:
>> I've found that replacing "git" by "git.cmd" in /exporters/__init.py
>> removed the error, apparently subprocess cannot handle windows scripts.
>
>
> I believe it can, but you have to pass "shell=True" for that to work.

Ah - ok. I'll add that tweak before I release 0.9.

Ian C.

Revision history for this message
Ian Clatworthy (ian-clatworthy) wrote :

Try the fix in rev 227 please.

Changed in bzr-fastimport:
assignee: nobody → Ian Clatworthy (ian-clatworthy)
importance: Undecided → Medium
status: New → Fix Released
Revision history for this message
Ian Clatworthy (ian-clatworthy) wrote :

Make that rev 228.

Revision history for this message
Erik Hahn (erikhahn) wrote :

Works fine.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.