Add 2 more Appveyor matrix-combinations forCygwin, andhide 2+2 (see below) TCs failing -you have to check from time to time if they hve been fixed.
Addgit.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
Fix launching ofMINGW'sgit-daemon also for Cygwin - reqork it as resource-managed (with statement.
"Polish" some urls sent togit commands to inlude'/cygdrive/...' prefix.
de-cygpathize repo-paths when creatinggit.Repo() instances to accept also'/cygdrive/...' paths.

Appveyor Cygwin TC failures

Reporting changes in the Appvayor cygwin-combinationbottom-line for significant commits, in reverse chronological order:

5962373: cygwin, appveyor,Support Cygwin's Git on Windows #533: Enable actual failures, hide certain 2+2 cases
- PY2: skip: 12
- PY3: skip: 16
b2efa1b: cygwin,Support Cygwin's Git on Windows #533: FIX submodules detection (~10TCs fixed)
- PY2: err: 2, fail: 1
- PY3: err: 2, fail: 1
9afebf67: cygwin,Support Cygwin's Git on Windows #533: FIX submodules detection (~10TCs fixed)
- PY2: err: 2, fail: 2
- PY3: err: 2, fail: 2
a2d248b: cygwin,Support Cygwin's Git on Windows #533: polish abs-paths ingit add commands
- PY2: err: 12, fail: 2
- PY3: err: 11, fail: 2
3b1cfcc: cygwin,Fix "known" Windows errored test cases #525: Allow '/cygdrive/c/' paths on repo init
- PY2: err: 13, fail: 2
- PY3: err: 12, fail: 2
57d0537: cygwin,Fix "known" Windows errored test cases #525: Polish also --git-separate-dir
- PY2: err: 13, fail: 3
- PY3: err: 12, fail: 3
0bce7cc: cygwin,Fix "known" Windows errored test cases #525: FIX daemon launching
- PY2: err: 13, fail: 3
- PY3: err: 12, fail: 3
e6e23ed: cygwin,Fix "known" Windows errored test cases #525: Try to make it work with Cygwin's Git.
- PY2: err: 14, fail: 3
- PY3: err: 13, fail: 3
ba7c2a0: appveyor,Fix "known" Windows errored test cases #525: enable CYGWIN TCs without failing (START: no Cygwin specific code)
- PY2: err: 44, fail: 0
- PY3: err: 13, fail: 0

ankostis added5 commits

October 14, 2016 17:43

appveyor,gitpython-developers#533: enable CYGWIN TCs without failing

ba7c2a0

- Cygwin TCs failing (start, no Cygwin specific code):  - PY2: err: 44, fail: 0  - PY3: err: 13, fail: 0

cygwin,gitpython-developers#533: Try to make it work with Cygwin's Git.

e6e23ed

+ Make `Git.polish_url()` convert paths into Cygwin-friendly paths.+ Add utility and soe TCs for funcs for detecting cygwin and convertingabs-paths to `/cygdrive/c/...`.- Cygwin TCs failing:  - PY2: err: 14, fail: 3  - PY3: err: 13, fail: 3

cygwin,gitpython-developers#533: FIX daemon launching

0bce7cc

+ 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,gitpython-developers#533: Polish also --git-separate-dir

57d0537

- Cygwin TCs failing:  - PY2: err: 13, fail: 3  - PY3: err: 12, fail: 3

cygwin,gitpython-developers#533: Allow '/cygdrive/c/' paths on repo …

3b1cfcc

…init- Cygwin TCs failing:  - PY2: err: 13, fail: 2  - PY3: err: 12, fail: 2

Byron added the in progress label

Oct 14, 2016

Copy link

codecov-io commentedOct 14, 2016•
edited
Loading

Current coverage is 94.33% (diff: 93.65%)

Merging#533 intomaster will decrease coverage by0.08%

@@             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

Powered byCodecov. Last update9e4a454...cc77e6b

ankostis added a commit to ankostis/GitPython that referenced this pull request

Oct 14, 2016

cygwin,gitpython-developers#533: polish abs-paths ingit addcommands

5895270

+ 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

ankostis added tag.Windows tag.cygwin labels

Oct 15, 2016

ankostis self-assigned this

Oct 15, 2016

ankostis added this to thev2.1.0 - proper windows support milestone

Oct 15, 2016

ankostis force-pushed thecygwin branch fromd8fc2e4 toefbedffCompare

October 16, 2016 00:23

ankostis added a commit to ankostis/GitPython that referenced this pull request

Oct 16, 2016

cygwin,gitpython-developers#533: FIX submodules detection

7fa6a8d

+ Decygpath sm's `.git` file contents.

ankostis added4 commits

October 16, 2016 02:45

cygwin,gitpython-developers#533: polish abs-paths ingit addcommands

a2d248b

+ 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

src: import os.path as osp

0210e39

src: reduce needless deps togitdb.util

b02662d

ci, deps: no PY26, ddt>=1.1.1, CIspip install test-requirements

4486bcb

+ Use environment-markers in requirement files (seehttp://stackoverflow.com/a/33451105/548792).

ankostis force-pushed thecygwin branch fromefbedff to997ea34Compare

October 16, 2016 00:56

ankostis added a commit to ankostis/GitPython that referenced this pull request

Oct 16, 2016

cygwin,gitpython-developers#533: FIX submodules detection (~10TCs fi…

997ea34

…xed)+ Decygpath sm's `.git` file contents.- Cygwin TCs failing:  - PY2: err: 2, fail: 2  - PY3: err: 2, fail: 2

ankostis added2 commits

October 16, 2016 03:04

cygwin,gitpython-developers#533: FIX submodules detection (~10TCs fi…

b2efa1b

…xed)+ Decygpath sm's `.git` file contents.+ Polish another path in `git add`; actually no main-code changes, justa replace \-->/ on a relative(!) path to make cygwin-git to work.- REGRESSION `test_git_submodules_and_add_sm_with_new_commit` asks foruser/email settings.- Cygwin TCs failing:  - PY2: err: 2, fail: 1  - PY3: err: 2, fail: 1

Merge withgitpython-developers#532, fix unicode filenames with escap…

ec731f4

…esurogates

ankostis added a commit to ankostis/GitPython that referenced this pull request

Oct 16, 2016

cygwin, appveyor,gitpython-developers#533: Enable actual failures, h…

8bdf92e

…ide certain 2+2 cases

ankostis added a commit to ankostis/GitPython that referenced this pull request

Oct 16, 2016

cygwin, appveyor,gitpython-developers#533: Enable actual failures, h…

06fa996

…ide certain 2+2 cases

ankostis added a commit to ankostis/GitPython that referenced this pull request

Oct 16, 2016

cygwin, appveyor,gitpython-developers#533: Enable actual failures, h…

4198593

…ide certain 2+2 cases

cygwin, appveyor,gitpython-developers#533: Enable actual failures, h…

5962373

…ide certain 2+2 cases

ankostis force-pushed thecygwin branch from997ea34 to5962373Compare

October 16, 2016 20:20

Copy link

ContributorAuthor

ankostis commentedOct 16, 2016

@Byron you may merge this one, I'm done withWindows, and franky I'm exhausted.
The CHANGES for a "Windows" version are summarized in the OPs of#519,#525 and this issue.

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 added2 commits

October 17, 2016 11:37

helper: minor fix prefix of temp-dirs

08e0d5f

tc-helper: fix minor contexlib abuse

cc77e6b

Byron merged commitcc77e6b intogitpython-developers:master

Oct 22, 2016

Byron removed the in progress label

Oct 22, 2016

Copy link

Member

Byron commentedOct 22, 2016

@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.
I will go ahead and try to make the summary for the changes.rst file.

Copy link

Member

Byron commentedOct 22, 2016

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 :).