search

viktorgino / headunit-desktop

HeadUnit Desktop is a car PC software built with Qt 5 and QML
https://github.com/viktorgino/headunit-desktop/wiki
GNU General Public License v3.0
285 stars 64 forks source link

cmd line option to select plugins to load #42

Closed gregd72002 closed 5 years ago

gregd72002 commented 5 years ago

We have different types of plugins and the list will only grow.

This patch enables user to select plugins to load. image

If omitted, all plugins will be loaded (the default behaviour).

viktorgino commented 5 years ago

Would it not make more sense to use a blacklist rather than a whitelist?

viktorgino commented 5 years ago

Actually, this is good as is. A black list based approach should be impemented using settings, so the user can choose plugins from the GUI.

gregd72002 commented 5 years ago

the idea in here was also to keep existing functionality. If the parameter is not used the behaviour is as it always was - load all plugins found. As for UI blacklisting - yes, it does makes sense to me