-
Notifications
You must be signed in to change notification settings - Fork 49
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
support for OSX #60
Comments
I guess nowadays a lot of people running their databases in the docker containers, so with the native Docker for OS X this is less an issue. However, adopting pg_view to non-Linux systems would be more or less straightforward with psutil, while we don't currently plan to do that, we'd welcome contributions. |
@hunleyd Hi! Thanks for your interest in this project -- were you interested in making the contribution that @alexeyklyukin? |
interested? yes. capable? not until i learn the language :( |
@hunleyd Understood. Good luck! |
I'm interested in making this contribution, actually I've already started, so hope it's still valid ;) Btw. I'll be willing to add tests and do some refactor - currently I am working on that in one branch, but will be able to send separate pull requests. |
@rsiera sure, you are welcome. Could you describe briefly how are you going to solve the problem? Are you planning on incorporating https://github.com/giampaolo/psutil, or reimplementing the /proc routines in a MacOS compatible way? |
Ok, great, I've added psutil to the project so it can be platform-independent. Need some time to verify if everything will work. Is it acceptable solution, or do you prefer the second one ? |
@rsiera psutil is what I had in mind as well, so, please, go ahead. |
Currently if you try to run the tool on OSX, it outputs
Non Linux database hosts are not supported at the moment. Can not continue
which is understandable for a new tool, but I know a lot of people who do their devel on their macbooks and then deploy onto Linux servers/vms and being able to use the tool on OSX would be great.The text was updated successfully, but these errors were encountered: