commit 5808c18f49006e1ab2cccc946b20e3e4c8d501a7
author David Munro <davidmunro@google.com> Wed Jun 16 12:15:05 2021 +1000
committer Commit Bot <commit-bot@chromium.org> Wed Jun 16 04:20:22 2021 +0000
tree 78fab652ca3313f530f4d2afb143fc8f310b3676
parent b7d3ed3b4dc7777bbbe49cdc7ec876ae850bca89

get_binhost.py: Support manifest paths without trailing slashes

At least for the internal manifest on tatl, I'd see 'Unknown manifest
source
https://chrome-internal.googlesource.com/chromeos/manifest-internal'
with no trailing slash. Support with or without.

BUG=None
TEST=get_binhost.py tatl

Change-Id: I102c248baaa17cb91c6f35bf1c92eb740bce467c
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/dev-util/+/2963802
Reviewed-by: David Munro <davidmunro@google.com>
Reviewed-by: Fergus Dall <sidereal@google.com>
Commit-Queue: David Munro <davidmunro@google.com>
Tested-by: David Munro <davidmunro@google.com>
Auto-Submit: David Munro <davidmunro@google.com>

contrib/get_binhost.py

diff --git a/contrib/get_binhost.py b/contrib/get_binhost.py
index 5bd948b..d1c2372 100755
--- a/contrib/get_binhost.py
+++ b/contrib/get_binhost.py
@@ -38,10 +38,10 @@
         cwd='/mnt/host/source/.repo/manifests/',
         check=True).stdout.decode().strip()
 
-    if url == 'https://chrome-internal.googlesource.com/chromeos/manifest-internal/':
+    if 'chrome-internal.googlesource.com/chromeos/manifest-internal' in url:
         return True
 
-    if url == 'https://chromium.googlesource.com/chromiumos/manifest/':
+    if 'chromium.googlesource.com/chromiumos/manifest' in url:
         return False
 
     print(f'Unknown manifest source {url}')