Skip to content

Conversation

@quinnj
Copy link
Member

@quinnj quinnj commented Jul 8, 2015

No description provided.

@quinnj quinnj mentioned this pull request Jul 8, 2015
@quinnj
Copy link
Member Author

quinnj commented Jul 8, 2015

The OSX failure looks unrelated (further up in the build process); can anyone confirm before I merge this?

@tkelman
Copy link
Contributor

tkelman commented Jul 8, 2015

was a timeout, I restarted the PR build

@IainNZ
Copy link
Member

IainNZ commented Jul 8, 2015

Seems to have happened again?

@tkelman
Copy link
Contributor

tkelman commented Jul 8, 2015

did someone else restart the pr build a second time? i didn't restart the push build, that one was a strange clang internal error? https://gist.github.com/tkelman/b5ef2226c575c2613933

@quinnj
Copy link
Member Author

quinnj commented Jul 8, 2015

Yeah, the push build has the weird OSX clang error. We're good to merge, right?

@tkelman
Copy link
Contributor

tkelman commented Jul 8, 2015

lgtm

quinnj added a commit that referenced this pull request Jul 8, 2015
@quinnj quinnj merged commit 7fa43ed into master Jul 8, 2015
@quinnj quinnj deleted the jq/mmapdeprecate branch July 8, 2015 20:57
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