-
Notifications
You must be signed in to change notification settings - Fork 34
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
Would you accept a freebsd PR? #43
Comments
Sure thing! We just have to make sure that we test the changes on all platforms (Linux, FreeBSD, Mac). |
I don't have a mac, if I get the PR ready would you be willing to test on Mac? |
Well I would, but I don't have a testing setup for all platforms either. Would be a great addition to the CI though. |
Okay, I'm not sure how to proceed on testin on mac then, but I'll open a PR that works on freebsd and linux at least. Since those are the main platforms, and what I'm doing shouldn't hit mac maybe thats enough, but we can go from there |
I opened #44 I did end up changing things for mac... but looking at the manual pages I'm certain they were not working in the same way it wasn't for freebsd. |
Thx again @duskmoss , I will close this for now. Feel free to open a new issue if something comes up. |
you're welcome! |
Hi just wondering if I made a little pull request that made the script run on freebsd (maybe also a bit in the read me about the dependencies for freebsd) would you accept it?
Just a few minor changes . So far just changes to sed and stat format. I'd test on freebsd and linux.
I know you don't use and rightly said you can't support FreeBSD, so I understand if you'd rather not have anything more that makes it look like you do.
The text was updated successfully, but these errors were encountered: