-
Notifications
You must be signed in to change notification settings - Fork 11
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
macOS support #2
Comments
I don't know if it would be easily ported to node add ons (I have no idea what I'm doing when it comes to C/C++), but I found a library that works on my machine that works quite well. https://github.com/kfix/ddcctl |
I've looked at that before. Seems like pretty much all the macOS DDC/CI stuff for macOS is ultimately based on @jontaylor/DDC-CI-Tools-for-OS-X. Since that's GPL-licensed, would need to relicense this project to include any of that code. Alternatively, ddcctl could be called from node, but that's not ideal. I think the entire project has to be GPL licensed, but I can dual-license everything except the DDC library code as MIT. |
hi. @hensm - Do you have plans to add support for MacOS in the near future? |
@sabaatworld if not, you can always just call ddcctl using child_process. |
@sabaatworld I had a local build based on the kfix code at one point, but I found it to be very unreliable. Might be worth taking another look at, since there are a couple of newer projects that seem to have made some improvements on it. |
Might I recommend https://github.com/MonitorControl/MonitorControl for macos, works great. |
No description provided.
The text was updated successfully, but these errors were encountered: