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

OSLogWriter on macOS #19

Closed
wants to merge 4 commits into from
Closed

OSLogWriter on macOS #19

wants to merge 4 commits into from

Conversation

infinisil
Copy link
Contributor

Removed unnecessary preprocessor condition that prevented OSLogWriter from being compiled and tested on macOS. Test passes on my machine.

@cnoon cnoon self-assigned this Jan 13, 2017
@cnoon cnoon added this to the 3.0.0 milestone Jan 13, 2017
@cnoon
Copy link
Member

cnoon commented Jan 13, 2017

Hi @infinisil, thanks for putting this together...much appreciated! 🍻

After some investigation, I finally figured out what the core issue was. The OSLogWriter test wasn't written correctly and would crash on 10.11 and not 10.12. I ended up making a few small changes to this PR and pushed them into master in 9124f60 and 00aabcb while keeping your attribution. I then pushed up 1657467 which fixes the broken tests. Everything is now working on macOS 10.11 and
10.12.

These changes will be released shortly as part of the Willow 3.0.0 release.

Thanks again!

@cnoon cnoon closed this Jan 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants