All of lore.kernel.org
 help / color / mirror / Atom feed
* [meta-oe][PATCH 0/1] Fix install of adb client when TOOLS is overridden
@ 2021-08-02 16:23 Ben Brown
  2021-08-02 16:23 ` [meta-oe][PATCH 1/1] android-tools: fix " Ben Brown
  0 siblings, 1 reply; 2+ messages in thread
From: Ben Brown @ 2021-08-02 16:23 UTC (permalink / raw)
  To: openembedded-devel

The following changes since commit 5a9eef2f531cfb051661c53de34e4c6ba6915138:

  nginx: upgrade 1.19.6 -> 1.21.1 (2021-07-30 10:42:35 -0700)

are available in the Git repository at:

  https://github.com/benjamb/meta-openembedded.git 92c94b1b5688cc4ac686d3801c7a2efc29b6eb38

for you to fetch changes up to 92c94b1b5688cc4ac686d3801c7a2efc29b6eb38:

  android-tools: fix install of adb client when TOOLS is overridden (2021-08-02 17:02:18 +0100)

----------------------------------------------------------------

Ben Brown (1):
  android-tools: fix install of adb client when TOOLS is overridden

 .../recipes-devtools/android-tools/android-tools_5.1.1.r37.bb   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.32.0


^ permalink raw reply	[flat|nested] 2+ messages in thread

* [meta-oe][PATCH 1/1] android-tools: fix install of adb client when TOOLS is overridden
  2021-08-02 16:23 [meta-oe][PATCH 0/1] Fix install of adb client when TOOLS is overridden Ben Brown
@ 2021-08-02 16:23 ` Ben Brown
  0 siblings, 0 replies; 2+ messages in thread
From: Ben Brown @ 2021-08-02 16:23 UTC (permalink / raw)
  To: openembedded-devel

Overriding TOOLS with 'adb' as the only entry, or with 'adb' as the
final entry in the list fails to match the grep pattern. The current
pattern includes trailing whitespace, likely to distinguish it from
'adbd'. However since `${TOOLS}` is then passed to grep unquoted, any
trailing whitespace would be dropped by the shell.

Fixed by replacing the trailing whitespace with '\>' to mark the end of
the word, which continues to ensure we don't match against 'adbd'.

Signed-off-by: Ben Brown <ben.brown@codethink.co.uk>
---
 .../recipes-devtools/android-tools/android-tools_5.1.1.r37.bb   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-oe/recipes-devtools/android-tools/android-tools_5.1.1.r37.bb b/meta-oe/recipes-devtools/android-tools/android-tools_5.1.1.r37.bb
index e8992346b..fc91e13a5 100644
--- a/meta-oe/recipes-devtools/android-tools/android-tools_5.1.1.r37.bb
+++ b/meta-oe/recipes-devtools/android-tools/android-tools_5.1.1.r37.bb
@@ -133,7 +133,7 @@ do_install() {
         install -m0755 ${B}/ext4_utils/simg2simg ${D}${bindir}
     fi
 
-    if echo ${TOOLS} | grep -q "adb " ; then
+    if echo ${TOOLS} | grep -q "adb\>" ; then
         install -d ${D}${bindir}
         install -m0755 ${B}/adb/adb ${D}${bindir}
     fi
-- 
2.32.0


^ permalink raw reply related	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-08-02 16:24 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-02 16:23 [meta-oe][PATCH 0/1] Fix install of adb client when TOOLS is overridden Ben Brown
2021-08-02 16:23 ` [meta-oe][PATCH 1/1] android-tools: fix " Ben Brown

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.