Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upSupport Cygwin's Git on Windows #533
Conversation
- Cygwin TCs failing (start, no Cygwin specific code): - PY2: err: 44, fail: 0 - PY3: err: 13, fail: 0
+ Make `Git.polish_url()` convert paths into Cygwin-friendly paths. + Add utility and soe TCs for funcs for detecting cygwin and converting abs-paths to `/cygdrive/c/...`. - Cygwin TCs failing: - PY2: err: 14, fail: 3 - PY3: err: 13, fail: 3
+ Rework git-daemon launching with `with` resource-management. + cmd: add `is_cygwin` optional override kwd on `Git.polish_url()`. - Cygwin TCs failing: - PY2: err: 13, fail: 3 - PY3: err: 12, fail: 3
- Cygwin TCs failing: - PY2: err: 13, fail: 3 - PY3: err: 12, fail: 3
- Cygwin TCs failing: - PY2: err: 13, fail: 2 - PY3: err: 12, fail: 2
Current coverage is 94.33% (diff: 93.65%)@@ master #533 diff @@
==========================================
Files 63 63
Lines 9736 9872 +136
Methods 0 0
Messages 0 0
Branches 0 0
==========================================
+ Hits 9193 9313 +120
- Misses 543 559 +16
Partials 0 0
|
+ Modify TCs - no main-code changes. + FIXed: + `TestSubmodule.test_git_submodules_and_add_sm_with_new_commit()` + TestDiff.test_diff_with_staged_file() - Cygwin TCs failing: - PY2: err: 11, fail: 2 - PY3: err: 10, fail: 2
+ Decygpath sm's `.git` file contents.
+ Modify TCs - no main-code changes. + FIXed: + `TestSubmodule.test_git_submodules_and_add_sm_with_new_commit()` + TestDiff.test_diff_with_staged_file() - Cygwin TCs failing: - PY2: err: 12, fail: 2 - PY3: err: 11, fail: 2
+ Use environment-markers in requirement files (see http://stackoverflow.com/a/33451105/548792).
…xed) + Decygpath sm's `.git` file contents. - Cygwin TCs failing: - PY2: err: 2, fail: 2 - PY3: err: 2, fail: 2
+ Decygpath sm's `.git` file contents. + Polish another path in `git add`; actually no main-code changes, just a replace \-->/ on a relative(!) path to make cygwin-git to work. - REGRESSION `test_git_submodules_and_add_sm_with_new_commit` asks for user/email settings. - Cygwin TCs failing: - PY2: err: 2, fail: 1 - PY3: err: 2, fail: 1
…ide certain 2+2 cases
…ide certain 2+2 cases
…ide certain 2+2 cases
|
@Byron you may merge this one, I'm done with Windows, and franky I'm exhausted. I would appreciate if you tried to make an even better summary and add them into CHANGES, to get an overview of my changes needed for making Windows work. |
|
@ankostis Thanks for the hint ! I just pushed the merge-commit, bringing in all the goodness you have been working on. Thanks so much for all the work you put in, GitPython is so much better now. |
|
A new release was made. As I felt incapable of summing up the details of the changes concisely, except for general bits about unicode handling and context managers. Thus I ended up praising you, and linked to the respective milestone, hoping that this is alright with you :). |
git.util.cygpath()&git.util.decygpath()based on material from these 2 resources:+https://msdn.microsoft.com/en-us/library/windows/desktop/aa365247(v=vs.85).aspx
+https://www.cygwin.com/cygwin-ug-net/using.html#unc-paths
git-daemonalso for Cygwin - reqork it as resource-managed (withstatement.gitcommands to inlude'/cygdrive/...'prefix.git.Repo()instances to accept also'/cygdrive/...'paths.Appveyor Cygwin TC failures
Reporting changes in the Appvayor cygwin-combination bottom-line for significant commits, in reverse chronological order:
git addcommands