New issue
Advanced search Search tips

Issue 836838 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2018
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

Compile error: compile_resources.py

Project Member Reported by pdr@chromium.org, Apr 25 2018

Issue description

I'm getting a compile error on my patch[1]:
FAILED: gen/arsc/apks/HttpWebApk.ap_ gen/arsc/apks/HttpWebApk.ap_.info gen/chrome/android/webapk/shell_apk/http_webapk__compile_resources_R.txt gen/chrome/android/webapk/shell_apk/http_webapk__compile_resources.srcjar gen/chrome/android/webapk/shell_apk/http_webapk/http_webapk.resources.proguard.txt 
python ../../build/android/gyp/compile_resources.py --depfile gen/chrome/android/webapk/shell_apk/http_webapk__compile_resources_1.d --android-sdk-jar ../../third_party/android_tools/sdk/platforms/android-27/android.jar --aapt-path ../../third_party/android_tools/sdk/build-tools/27.0.3/aapt --dependencies-res-zips=@FileArg\(gen/chrome/android/webapk/shell_apk/http_webapk.build_config:resources:dependency_zips\) --extra-res-packages=@FileArg\(gen/chrome/android/webapk/shell_apk/http_webapk.build_config:resources:extra_package_names\) --extra-r-text-files=@FileArg\(gen/chrome/android/webapk/shell_apk/http_webapk.build_config:resources:extra_r_text_files\) --android-manifest gen/chrome/android/webapk/shell_apk/http_webapk_manifest/AndroidManifest.xml --version-code 1 --version-name Developer\ Build --apk-path gen/arsc/apks/HttpWebApk.ap_ --apk-info-path gen/arsc/apks/HttpWebApk.ap_.info --debuggable --r-text-out gen/chrome/android/webapk/shell_apk/http_webapk__compile_resources_R.txt --srcjar-out gen/chrome/android/webapk/shell_apk/http_webapk__compile_resources.srcjar --check-resources-pkg-id=0x7f --proguard-file gen/chrome/android/webapk/shell_apk/http_webapk/http_webapk.resources.proguard.txt
  File "../../build/android/gyp/compile_resources.py", line 19
    faydimport shutil

[1] https://chromium-review.googlesource.com/c/chromium/src/+/1026941

This seems to be recently changed:
https://chromium.googlesource.com/chromium/src/+/2aea307c1ffa0468aad8962b7049862e80fca97e
 
Reverted. Bad merge, not sure how it happened.
Status: Fixed (was: Assigned)

Sign in to add a comment