Skip to content
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

fix builds on windows #4425

Merged
merged 1 commit into from
Nov 25, 2017
Merged

fix builds on windows #4425

merged 1 commit into from
Nov 25, 2017

Conversation

whyrusleeping
Copy link
Member

cc @keks @Stebalien somehow we missed updating this hash. I'm pretty sure that gx doesnt skip any files because due to build constraints.

License: MIT
Signed-off-by: Jeromy jeromyj@gmail.com

License: MIT
Signed-off-by: Jeromy <jeromyj@gmail.com>
@djdv
Copy link
Contributor

djdv commented Nov 25, 2017

Referencing this: #4422 which also references related PRs.

@keks
Copy link
Contributor

keks commented Nov 25, 2017

I'm not sure how this happened, but it must have been during the massive update commit 76e1da0.

I think we can safely merge this now, since the tests in #4422 pass and it's the same change.

@keks keks self-assigned this Nov 25, 2017
@keks keks requested a review from Stebalien November 25, 2017 15:07
@Stebalien
Copy link
Member

I'm pretty sure that gx doesnt skip any files because due to build constraints.

A rebase? But that would have broken everything... I'm not sure. I take it we don't build on windows in CI?

@Stebalien
Copy link
Member

#4427

@whyrusleeping whyrusleeping merged commit 96b8723 into master Nov 25, 2017
@whyrusleeping whyrusleeping deleted the fix/windows-build-deps branch November 25, 2017 18:58
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.

4 participants