Junio and Hannes: thanks for the comments four months ago; I've
been slow getting back to this. I incorporated all your
suggestions.
Junio: this merges okay with Brandon's v2.4 support series.
This series fixes problems in git-p4, and its tests, so that
git-p4 works on the cygwin platform.
See the wiki for info on how to get started on cygwin:
https://git.wiki.kernel.org/index.php/GitP4
Testing by people who use cygwin would be appreciated. It would
be good to support cygwin more regularly. Anyone who had time
to contribute to testing on cygwin, and reporting problems, would
be welcome.
There's more work requried to support msysgit. Those patches
are not in good enough shape to ship out yet, but a lot of what
is in this series is required for msysgit too.
These patches:
- fix bugs in git-p4 related to issues found on cygwin
- cleanup some ugly code in git-p4 observed in error paths while
getting tests to work on cygwin
- simplify and refactor code and tests to make cygwin changes easier
- handle newline and path issues for cygwin platform
- speed up some aspects of git-p4 by removing extra shell invocations
Changes from v1:
http://thread.gmane.org/gmane.comp.version-control.git/206557
- Addressed comments from Junio and Hannes:
- Removed "git p4: fix error message when "describe -s" fails";
it was fixed as part of 18fa13d (git p4: catch p4 describe
errors, 2012-11-23), with messages like "p4 describe -s ...
failed".
- Removed extranneous "grep -q" in "git p4: generate better
error message for bad depot path".
- Added "git p4 test: avoid loop in client_view" after a
suggestion from Junio.
- Made the test-path-utils removal explicit.
- Modify the chmod test to use test_chmod, and verify at
least the p4 bits on cygwin, although not the filesystem.
- Retested on latest cygwin
Pete Wyckoff (21):
git p4: temp branch name should use / even on windows
git p4: remove unused imports
git p4: generate better error message for bad depot path
git p4 test: use client_view to build the initial client
git p4 test: avoid loop in client_view
git p4 test: use client_view in t9806
git p4 test: start p4d inside its db dir
git p4 test: translate windows paths for cygwin
git p4: remove unreachable windows \r\n conversion code
git p4: scrub crlf for utf16 files on windows
git p4 test: newline handling
git p4 test: use LineEnd unix in windows tests too
git p4 test: avoid wildcard * in windows
git p4: cygwin p4 client does not mark read-only
git p4 test: use test_chmod for cygwin
git p4: disable read-only attribute before deleting
git p4: avoid shell when mapping users
git p4: avoid shell when invoking git rev-list
git p4: avoid shell when invoking git config --get-all
git p4: avoid shell when calling git config
git p4: introduce gitConfigBool
git-p4.py | 119 ++++++++++++++++++++++++++++--------------
t/lib-git-p4.sh | 64 ++++++++++++++++-------
t/t9800-git-p4-basic.sh | 5 ++
t/t9802-git-p4-filetype.sh | 117 +++++++++++++++++++++++++++++++++++++++++
t/t9806-git-p4-options.sh | 51 ++++++++----------
t/t9807-git-p4-submit.sh | 14 ++++-
t/t9809-git-p4-client-view.sh | 16 ++++--
t/t9812-git-p4-wildcards.sh | 37 ++++++++++---
t/t9815-git-p4-submit-fail.sh | 11 ++--
t/test-lib.sh | 3 ++
10 files changed, 332 insertions(+), 105 deletions(-)
--
1.8.1.1.460.g6fa8886
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
