On Tue, 24 Sept 2024 at 09:58, Mikko Rapeli <[email protected]> wrote: > > Hi, > > On Fri, Sep 20, 2024 at 03:07:48PM +0530, Raghuvarya S via > lists.openembedded.org wrote: > > To ensure android-tools-adbd.service starts at boot, the path > > for ConditionPathExists must be present at build time. /etc is > > more suitable for build-time files than /var, which is for > > runtime files. Changed ConditionPathExists from > > /var/usb-debugging-enabled to /etc/usb-debugging-enabled > > I know this is a backport but I think change from /var to /etc is wrong. > > /var and /run should be writable also with read-only rootfs. /etc is not > writable. > This change requires read-only rootfs users to symlink > /etc/usb-debugging-enabled > to some other location like /run which isn't nice.
Yeah, that's also a correct point. I think it's all up to the maintainer's discretion. To be fair, in meta-qcom we are using kernel command line arguments (in addition to the flag) in order to enable adbd. Works better for one-time things. -- With best wishes Dmitry
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#112476): https://lists.openembedded.org/g/openembedded-devel/message/112476 Mute This Topic: https://lists.openembedded.org/mt/108555790/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
