Skip to content

Conversation

stefansedich
Copy link
Contributor

Fixes for #1100

@Aaronontheweb
Copy link
Member

@stefansedich could you squash these commits?

@Aaronontheweb
Copy link
Member

Pull in latest from dev - #1102 might have fixed this.

@stefansedich
Copy link
Contributor Author

Ah awesome! Wil give that a shot today sometime.

On Sun, 28 Jun 2015 09:07 Aaron Stannard notifications@github.com wrote:

Pull in latest from dev - #1102
#1102 might have fixed this.


Reply to this email directly or view it on GitHub
#1103 (comment).

@Aaronontheweb
Copy link
Member

@stefansedich mind including a short update about where you're at with the Mono stuff? All of the fun stuff from our Skype chat ;)

@stefansedich
Copy link
Contributor Author

Sure sorry @Aaronontheweb forgot to write an update :)

So thus far this has been mostly a task of one step forwards 500 back, after various small fixes to actually get the build running on Mono (fix some output paths, ignore MSTest tests under mono) we have now run into an issue with xunit2 on mono:

xunit/xunit#158

According to this thread the master branch (4.3.x) actually works with xunit2 (this is something I am currently validating as being true for akka.net).

I will update as soon as I get the latest mono compiling and have a chance to run the build.

@stefansedich
Copy link
Contributor Author

Ok running the latest nightly mono build the xunit tests at least start, but at the first cluster test it goes boom and seems to hang.

Will investigate this further!

@stefansedich
Copy link
Contributor Author

@Aaronontheweb should I kill this PR for now and shelve my changes? I am not sure I see a fix in the immediate term, I prefer to not just change the tests to mask the issue we are seeing in Mono :(.

@Aaronontheweb
Copy link
Member

@stefansedich yes, that's probably for the best - but keep #1110 up to date.

@stefansedich
Copy link
Contributor Author

Ok no worries, closing this off!

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