Skip to content

Conversation

@ddevault
Copy link
Contributor

@ddevault ddevault commented Nov 30, 2017

It's not just __APPLE__ that doesn't support this, but POSIX as a whole.
This doesn't seem necessary at all so I just removed it.

@normanmaurer
Copy link
Contributor

@jrudolph @nitsanw I think this should also be safe to merge FTW

@nitsanw
Copy link
Member

nitsanw commented Jan 31, 2018

@normanmaurer it would be nice if someone rebased it to make the build pass

It's not just __APPLE__ that doesn't support this, but POSIX as a whole.
This doesn't seem necessary at all so I just removed it.
@ddevault
Copy link
Contributor Author

Rebased.

@nitsanw nitsanw merged commit a615063 into jvm-profiling-tools:master Jan 31, 2018
@nitsanw
Copy link
Member

nitsanw commented Jan 31, 2018

Thanks for the contribution!

@jrudolph
Copy link
Member

👍 no good reason to keep those.

@normanmaurer
Copy link
Contributor

normanmaurer commented Jan 31, 2018 via email

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.

5 participants