New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tests now pass on MsWin32 platforms #4

Closed
wants to merge 6 commits into
base: master
from

Conversation

Projects
None yet
3 participants
@alabamapaul
Contributor

alabamapaul commented Jan 12, 2016

  • Updated git.t to use binmode() to avoid newline translation to/from LF to CRLF. This allows the tests to pass on the MsWin32 platform
  • Verified tests pass on:
    • Windows 7, 64-bit [(v5.20.1) built for MSWin32-x86-multi-thread-64int]
    • Ubuntu 14.04 LTS [(v5.18.2) built for x86_64-gnu-thread-multi]
  • Added more descriptions to the test names
@chorny

chorny approved these changes Oct 14, 2016

@dolmen

dolmen approved these changes Oct 17, 2016

dolmen added a commit that referenced this pull request Oct 18, 2016

@dolmen

This comment has been minimized.

Show comment
Hide comment
@dolmen

dolmen Oct 18, 2016

Collaborator

I'm sorry to have not noticed your pull request before. It is merged now. Thanks!

Collaborator

dolmen commented Oct 18, 2016

I'm sorry to have not noticed your pull request before. It is merged now. Thanks!

@dolmen dolmen closed this Oct 18, 2016

dolmen added a commit that referenced this pull request Oct 18, 2016

v0.162920
	Fix tests on MSWin32 platform (GH #4, Paul Durden). Thanks!
	Repository moved to github.com/dolmen-perl5/Git-Sub
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment