documentation for windows should mention to add c:\path\to\git\bin to PATH

Issue #198 resolved
domruf created an issue

I added c:\path\to\git\cmd instead of c:\path\to\git\bin to PATH which caused test_archives.py to fail because gzip and bzip2 were not available.

Comments (4)

  1. Thomas De Schampheleire

    Is this issue just a short-term reminder for yourself? Otherwise, since you already are contributing to Kallithea, I think it would be great if you could send a patch instead.

  2. Mads Kiilerich

    I guess the real problem here is that the test assumes these tools are available ... and often they are not on Windows.

    It is not obvious that taking them from the git porcelain is the right solution ... but it might be an ok pragmatic solution.

    I guess there might be other such dependencies. The test suite should perhaps explicitly require a full msys environment for Windows testling like Mercurial does.

    However, in this case, I guess it could be better to replace the requirement for these two tools with direct invocation of the corresponding Python module.

  3. domruf reporter

    Well it is not only the tests. In case of test_archives.py it is not the test itself that needs gzip. The git backend needs gzip.

    I agree that it would be better to not depend on these command line tools (see #194).

    But as a short term solution some words in the documentation would be good.

    I'll write something and create a PR.

  4. Log in to comment