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

lock to an earlier version of mono #1376

Merged
merged 1 commit into from
Jun 14, 2016
Merged

lock to an earlier version of mono #1376

merged 1 commit into from
Jun 14, 2016

Conversation

shiftkey
Copy link
Member

@shiftkey shiftkey commented Jun 14, 2016

Travis is currently in an interesting spot where latest means it'll use Mono 4.4.0 when building and testing on OS X, and 4.2.3 when building and testing on Linux.

Unfortunately this means the OS X builds are exhibiting some strange behaviour:

  • a wall of Chinese text in the BuildMono step
  • * Assertion at metadata.c:3643, condition 'ptr' not met when running the PCL tests

Example build output: https://travis-ci.org/octokit/octokit.net/jobs/137385011

I found the flag to force a specific version of Mono for both platforms, and that seems to do the trick here.

@dampir @ErikSchierboom @shaggygi @alfhenrik @maddin2016 I think you're all encountering this on your PRs, feel free to merge master or cherry-pick this commit into your branch after I confirm this resolves the issue.

@shiftkey shiftkey merged commit ff01eb4 into master Jun 14, 2016
@shiftkey shiftkey deleted the choose-mono-version branch June 14, 2016 00:24
@ryangribble
Copy link
Contributor

👍 thanks for this, I was scratching my head being unfamiliar with cross platform stuff

alexander-efremov pushed a commit to alexander-efremov/octokit.net that referenced this pull request Jun 14, 2016
alexander-efremov pushed a commit to alexander-efremov/octokit.net that referenced this pull request Jun 14, 2016
alexander-efremov pushed a commit to alexander-efremov/octokit.net that referenced this pull request Jun 14, 2016
alexander-efremov pushed a commit to alexander-efremov/octokit.net that referenced this pull request Jun 14, 2016
alexander-efremov pushed a commit to alexander-efremov/octokit.net that referenced this pull request Jun 14, 2016
alexander-efremov pushed a commit to alexander-efremov/octokit.net that referenced this pull request Jun 14, 2016
alexander-efremov pushed a commit to alexander-efremov/octokit.net that referenced this pull request Jun 14, 2016
alexander-efremov pushed a commit to alexander-efremov/octokit.net that referenced this pull request Jun 14, 2016
alexander-efremov pushed a commit to alexander-efremov/octokit.net that referenced this pull request Jun 14, 2016
alexander-efremov pushed a commit to alexander-efremov/octokit.net that referenced this pull request Jun 14, 2016
alexander-efremov pushed a commit to alexander-efremov/octokit.net that referenced this pull request Jun 14, 2016
alexander-efremov pushed a commit to alexander-efremov/octokit.net that referenced this pull request Jun 14, 2016
alexander-efremov pushed a commit to alexander-efremov/octokit.net that referenced this pull request Jun 14, 2016
alexander-efremov pushed a commit to alexander-efremov/octokit.net that referenced this pull request Jun 14, 2016
alexander-efremov pushed a commit to alexander-efremov/octokit.net that referenced this pull request Jun 14, 2016
@ErikSchierboom
Copy link
Contributor

Thanks! I've rebased my PR.

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.

3 participants