openbase / bco.dal

Domotic Abstraction Layer
Other
1 stars 2 forks source link

launcher exclution does not work anymore #98

Closed DivineThreepwood closed 6 years ago

DivineThreepwood commented 6 years ago

e.g.

bco-openhab --exclude-launcher app-manager

does not exlucde the app-manager anymore.

pLeminoq commented 6 years ago

The launcher exclusion still works. The issue is that you added the minus in 'app-manager'. The test for exclusion is as follows:

if (launcherEntry.getKey().getName().toLowerCase().contains(exclusionPatter.toLowerCase())) {
    exclude = true;
}

where launcherEntry.getKey() is the launcher class. For the app manager this is AppManagerLauncher. So if your commando should work maybe the test has to be adapted to exclude the minus. Otherwise I would suggest to use:

bco --print-launcher

to get the names of all launcher that can be excluded.

pLeminoq commented 6 years ago

Minuses and underscores are ignored for this check since commit openbase/jul@029a30e5a5a1057344d654c6193df13a6a7790c2