Description

ankostis

opened on Oct 3, 2016

Issue body actions

Currently, Windows TestCases on Appveyor established on #519 are "artifically" ok.
Because in the code, the Test Cases known to fail on have been marked with the HIDE_WINDOWS_KNOWN_ERRORS variable so that we get an ALL GREEN response; that way we can detect when new commits break additional TCs on Windows.

Currently there are 10 failing TCs (search them with the above var-name), and you can view the errors in this Appveyor job:

https://ci.appveyor.com/project/ankostis/gitpython/build/1.0.196

The errors are mostly related to 2 issues:

1. Unicode: for those with reading time, I would suggest these 2 contradictory sources:
   • problem: Everything you did not want to know about Unicode in Python 3
   • solution: PEP 383 -- Non-decodable Bytes in System Character Interfaces - so using text-mode
2. Leaked resources: these are (possibly memmory-mapped) files that are not deallocated (and hence, not deleted afterwards). Unfortunately, dealing with them might mean that you have to jump projects, as some are inherited from the library projects smmp and (mostly) gitdb.

In order to reproduce the failed TCs on your PC, change locally git/test/lib/helper.py:#L38 from this:

    HIDE_WINDOWS_KNOWN_ERRORS = os.environ.get('HIDE_WINDOWS_KNOWN_ERRORS', True)

into this:

    HIDE_WINDOWS_KNOWN_ERRORS = os.environ.get('HIDE_WINDOWS_KNOWN_ERRORS', False)

This issue is about not forgetting these TCs - asking for help to fix them.

Activity

1. ankostis

   added

   critical

   help wanted

   tag.Windows

   on Oct 3, 2016
2. ankostis

   added 4 commits that reference this issue on Oct 3, 2016

   doc: Explain Windows compatibility status, mention #525 asking for help

   86aa873

   MINGW, gitpython-developers#525: FIX remote urls in config-files

   d2e6f30

   MINGW, gitpython-developers#525: FIX remote urls in config-files

   58a3318

   MINGW, gitpython-developers#525: FIX remote urls in config-files

   3ded48c
3. Byron commented on Oct 9, 2016

   

   Byron

   on Oct 9, 2016

   Member

   More actions

   @ankostis Even though I won't be of much help here, I have just provided you with full write permissions on all the gitpython repositories that matter. Previously, I believe you might have been missing write access to smmap, for no other reason than me having forgotten to set it up right away :).
4. ankostis

   added

   tag.leaks

   on Oct 11, 2016
5. ankostis

   added 6 commits that reference this issue on Oct 11, 2016

   MINGW, gitpython-developers#525: FIX remote urls in config-files

   1900e7e

   MINGW, gitpython-developers#525: FIX remote urls in config-files

   ce344df

   MINGW, gitpython-developers#525: FIX remote urls in config-files

   b565b0e

   MINGW, gitpython-developers#525: FIX remote urls in config-files

   dc5b0e5

   daemon, gitpython-developers#525: simplify exception handling

   0f131a0

   daemon, gitpython-developers#525: FIX remote urls in config-files

   aedb065
6. 40 remaining items
7. ankostis

   added 10 commits that reference this issue on Oct 14, 2016

   cygwin, gitpython-developers#525: Polish also --git-separate-dir

   ec75656

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

   d57b53b

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

   e2a8978

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

   6b6661d

   cygwin, gitpython-developers#525: FIX daemon launching

   1fe86a3

   cygwin, gitpython-developers#525: Polish also --git-separate-dir

   b2749b1

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

   5179f87

   remote, gitpython-developers#525: FIX BUG push-cmd misses error messages

   5e6827e

   cmd, gitpython-developers#525: Always include stdout+stderr in except…

   c8e914e

   Merge pull request #530 fixing some #525 Windows errors

   e12ef59
8. ankostis

   added 3 commits that reference this issue on Oct 22, 2016

   fix(MapWindow): unicode foes in read_into_memory() used by gitpython TCs

   1bb6ff0

   fix(MapWindow): unicode foes in read_into_memory() used by gitpython TCs

   91814e6

   fix(MapWindow): unicode foes in read_into_memory() used by gitpython TCs

   b2eafcc
9. Byron

   added the

   Task

   issue type on Jul 20, 2026
10. Byron

    closed this as not plannedon Jul 20, 2026

Issue actions