Commit 6b87017e authored by Andrew Grieve's avatar Andrew Grieve Committed by Commit Bot

Android: fetch_deps.py - don't allow uppercase in CIPD paths.

CIPD does not allow them.

Splitting this off from larger change where the problem was hit:
http://crrev.com/c/1885951

Bug: 693079
Change-Id: I068bf8fd01044f5c80307f7f68de5bd03b1ed1f3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1888373
Auto-Submit: Andrew Grieve <agrieve@chromium.org>
Reviewed-by: default avatarSam Maier <smaier@chromium.org>
Commit-Queue: Sam Maier <smaier@chromium.org>
Cr-Commit-Position: refs/heads/master@{#710483}
parent 7857136b
......@@ -351,7 +351,8 @@ class BuildConfigGenerator extends DefaultTask {
} else {
cipdPath += repoPath
}
cipdPath += "/${depPath}"
// CIPD does not allow uppercase in names.
cipdPath += "/${depPath}".toLowerCase()
sb.append("""\
|
| 'src/${repoPath}/${depPath}': {
......@@ -436,7 +437,8 @@ class BuildConfigGenerator extends DefaultTask {
} else {
cipdPath += repoPath
}
cipdPath += "/${DOWNLOAD_DIRECTORY_NAME}/${dependency.id}"
// CIPD does not allow uppercase in names.
cipdPath += "/${DOWNLOAD_DIRECTORY_NAME}/" + dependency.id.toLowerCase()
// NOTE: the fetch_all.py script relies on the format of this file!
// See fetch_all.py:GetCipdPackageInfo().
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment