Skip to content

ARROW-839: [Python] Use mktime variant that is reliable on MSVC#559

Closed
wesm wants to merge 3 commits intoapache:masterfrom
wesm:ARROW-839
Closed

ARROW-839: [Python] Use mktime variant that is reliable on MSVC#559
wesm wants to merge 3 commits intoapache:masterfrom
wesm:ARROW-839

Conversation

@wesm
Copy link
Copy Markdown
Member

@wesm wesm commented Apr 18, 2017

This also reverts an unintentional regression from #544 when code from config.h was moved to platform.h

Copy link
Copy Markdown
Member

@xhochy xhochy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, LGTM

@asfgit asfgit closed this in 0bcb785 Apr 18, 2017
@wesm wesm deleted the ARROW-839 branch April 18, 2017 14:33
jeffknupp pushed a commit to jeffknupp/arrow that referenced this pull request Jun 3, 2017
This also reverts an unintentional regression from apache#544 when code from config.h was moved to platform.h

Author: Wes McKinney <wes.mckinney@twosigma.com>

Closes apache#559 from wesm/ARROW-839 and squashes the following commits:

2e9b300 [Wes McKinney] Use _mkgmtime64 on MSVC
f182bab [Wes McKinney] Restore include order in platform.h
38c29bf [Wes McKinney] Add Windows build instructions for Python
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants