-
-
Notifications
You must be signed in to change notification settings - Fork 386
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
Native preset disable config flag #99
Comments
Can I work on this issue? |
@Chinmaygoyal go for it! |
I am new to open source and have never made a PR before so can you help me by providing a start? |
Sure, there are lots of helpful articles if you click "Help" at the bottom of this page |
Has this been solved? |
Nope |
Has this been solved? If not then I would like to help |
Please feel free |
Hi, I'd like to contribute to this project, I've forked this already and I'd like to know where I should start working and get some assistance. |
Great! Join the discord! Look through open issues or fix something that bothers you. Documentation needs improving, downstream packages need maintaining. |
Native presets can now be enabled via a CMake parameter and are disabled by default, but there's still no config setting to prevent them from being loaded if they are installed. |
With disabled native preset support in the CMake build (which is the default), the whole native preset loading code won't be compiled into the library at all. As these presets have numerous issues with stability, security and portability, this issue should be fixed in the next release. |
I think the native presets are not very interesting and should be disabled by default. If someone wants to make some cool native presets they can enable it, but normal users have no need for this functionality.
The text was updated successfully, but these errors were encountered: