Bluetooth dependency check breaks Shelly usage about core HOT 4 CLOSED

Hey there @bdraco, mind taking a look at this issue as it has been labeled with an integration (bluetooth) you are listed as a code owner for? Thanks!

_{^{(message by CodeOwnersMention)}}

bluetooth documentation
bluetooth source
_{^{(message by IssueLinks)}}

from core.

Found a workaround. Not sure how this is connected to the reboot because this is nothing which have changed but anyway it would be nice if bluetooth dependency is not checked if deactivated. It is not really needed for the Shellys which are connected via network.

/etc/dbus-1/system.d/Bluetooth.conf

<policy user="homeassistant">
  <allow own="org.bluez"/>
  <allow send_destination="org.bluez"/>
  <allow send_interface="org.bluez.GattCharacteristic1"/>
  <allow send_interface="org.bluez.GattDescriptor1"/>
  <allow send_interface="org.freedesktop.DBus.ObjectManager"/>
  <allow send_interface="org.freedesktop.DBus.Properties"/>
</policy>  

There was also a policy for the group which is used already because of group assignment to the user but with less allow lines. Seems to be the allows for the group are not enough.

  <!-- allow users of bluetooth group to communicate -->
  <policy group="bluetooth">
    <allow send_destination="org.bluez"/>
  </policy>

from core.

Found a workaround. Not sure how this is connected to the reboot because this is nothing which have changed but anyway it would be nice if bluetooth dependency is not checked if deactivated. It is not really needed for the Shellys which are connected via network.

Its currently needed for Bluetooth in Shelly devices support https://www.home-assistant.io/integrations/shelly/#bluetooth-support , and since Home Assistant doesn't have a way to list a dep as conditional based on a config entry setting, its not possible at this time. It is not likely a feature we would add since it would significantly increase development complexity and dependency resolution, increasing risk.

from core.

Hey there @balloob, @bieniu, @thecode, @chemelli74, mind taking a look at this issue as it has been labeled with an integration (shelly) you are listed as a code owner for? Thanks!

_{^{(message by CodeOwnersMention)}}

shelly documentation
shelly source
_{^{(message by IssueLinks)}}

from core.