Sorry, I misread the order of the git history: I was the one adding a second version of android-tools-conf. However since this recipe doesn't need meta-selinux, it should simply be removed. Only android-tools which has DEPENDS = "... libselinux" needs to be duplicated.
On Sat, Jan 24, 2026 at 11:22 PM Etienne Cordonnier <[email protected]> wrote: > Hi Dmitry, > the reason I added a second version of android-tools is explained in the > pull-request which added android-tools version 10: > https://github.com/openembedded/meta-openembedded/pull/664#issuecomment-1468861667 > (one version depends on meta-selinux and the other does not). > > However I was not the one adding a second version of android-tools-conf. I > don't see a good reason to have two copies of the recipe > android-tools-conf, since this recipe doesn't depend on meta-selinux. This > was probably just a mistake. > > On Sat, Jan 24, 2026 at 9:08 PM Dmitry Baryshkov < > [email protected]> wrote: > >> On Sat, Jan 24, 2026 at 05:40:59PM +0800, Anuj Mittal wrote: >> > On Fri, Jan 23, 2026 at 5:46 PM Jose Quaresma via >> > lists.openembedded.org >> > <[email protected]> wrote: >> > > >> > > From: Shoudi Li <[email protected]> >> > > >> > > For the boards with multiple UDC ports, adb funtion will be impacted >> due >> > > to below error. >> > > >> > > ls /sys/class/udc/ > /sys/kernel/config/usb_gadget/adb/UDC >> > > ls: write error: Device or resource busy >> > > >> > > Signed-off-by: Shoudi Li <[email protected]> >> > > --- >> > > .../android-tools-conf-configfs/android-gadget-start | 6 >> ++++-- >> > > 1 file changed, 4 insertions(+), 2 deletions(-) >> > > >> > > diff --git >> a/meta-oe/dynamic-layers/selinux/recipes-devtool/android-tools/android-tools-conf-configfs/android-gadget-start >> b/meta-oe/dynamic-layers/selinux/recipes-devtool/android-tools/android-tools-conf-configfs/android-gadget-start >> > > index ca6c3df275..76b5e29624 100644 >> > > --- >> a/meta-oe/dynamic-layers/selinux/recipes-devtool/android-tools/android-tools-conf-configfs/android-gadget-start >> > > +++ >> b/meta-oe/dynamic-layers/selinux/recipes-devtool/android-tools/android-tools-conf-configfs/android-gadget-start >> > >> > For some reason, there are two *android-tools-conf-configfs* recipes >> > which seem identical except for this and a few other minor changes. I >> > think this one in dynamic-layers/selinux should be removed instead of >> > aligning it with the other recipe. >> >> Looks like it. Commit d3f4e6851243 ("android-tools 10: import version >> from meta-clang"), which added the second copy, isn't clear, why the >> second copy was necessary. >> >> -- >> With best wishes >> Dmitry >> >
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#123826): https://lists.openembedded.org/g/openembedded-devel/message/123826 Mute This Topic: https://lists.openembedded.org/mt/117415388/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
