chromite: Split long strings on lines > 80 chars.
Black will only split strings at spaces.
We can tweak its `SPLIT_SAFE_CHARS` to split elsewhere -
https://github.com/psf/black/blob/main/src/black/trans.py#L76
Allowing splits at "-","/", and "." lets it deal with a few more
long strings in chromite automatically, so there will be fewer
suppressions when the line-too-long lint is introduced.
BUG=b:233893248
TEST=cros lint + CQ
Change-Id: Ic092be1d0a80a4c13fac30d19756db715b42b8dd
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/4535497
Commit-Queue: Trent Apted <tapted@chromium.org>
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
Tested-by: Trent Apted <tapted@chromium.org>
diff --git a/scripts/cros_generate_breakpad_symbols.py b/scripts/cros_generate_breakpad_symbols.py
index ae63dd6..321ec16 100644
--- a/scripts/cros_generate_breakpad_symbols.py
+++ b/scripts/cros_generate_breakpad_symbols.py
@@ -73,7 +73,10 @@
# jacuzzi, scarlet, kukui, etc.
"usr/bin/rma_reset",
# Virtual dynamic shared object, not expected to have STACK records.
- "opt/google/containers/android/ndk_translation/lib/arm/libndk_translation_vdso.so",
+ (
+ "opt/google/containers/android/ndk_translation/lib/arm/"
+ "libndk_translation_vdso.so"
+ ),
# TODO(b/279665879): Figure out why this ndk_translation libraries is not
# getting STACK records.
"opt/google/containers/android/ndk_translation/lib/arm/libdexfile.so",
@@ -968,8 +971,10 @@
"file_list",
nargs="*",
default=None,
- help="generate symbols for only these files "
- "(e.g. /build/$BOARD/usr/bin/foo)",
+ help=(
+ "generate symbols for only these files "
+ "(e.g. /build/$BOARD/usr/bin/foo)"
+ ),
)
opts = parser.parse_args(argv)