New issue
Advanced search Search tips
Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Dec 14
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug

Blocking:
issue 914264



Sign in to add a comment
link

Issue 914707: confirm no-op failure in android-rel builder

Reported by tikuta@chromium.org, Dec 13 Project Member

Issue description

https://ci.chromium.org/b/8927276008359552288


ninja explain: output gen/chrome/android/chrome_public_bundle/chrome_public_bundle__unsplit_dex.zip doesn't exist
ninja explain: gen/chrome/android/chrome_public_bundle/chrome_public_bundle__unsplit_dex.zip is dirty
ninja explain: gen/chrome/android/chrome_public_bundle/chrome_public_bundle__unsplit_dex.zip is dirty
ninja explain: obj/chrome/android/chrome_public_bundle__unsplit_dex.stamp is dirty
ninja explain: gen/chrome/android/chrome_public_bundle__unsplit_dex__dexspliter.stamp is dirty
ninja explain: obj/chrome/android/chrome_public_bundle__unsplit_dex__dexspliter.stamp is dirty
ninja explain: gen/chrome/android/chrome_public_bundle/base.zip is dirty
ninja explain: gen/chrome/android/chrome_public_bundle/base.zip is dirty
ninja explain: obj/chrome/android/chrome_public_bundle__base__create.stamp is dirty
ninja explain: apks/ChromePublic.aab is dirty
ninja explain: obj/chrome/android/chrome_public_bundle__bundle.stamp is dirty
ninja explain: obj/chrome/android/chrome_public_bundle.stamp is dirty
ninja explain: obj/chrome/android/chrome_public_bundle__base__create.stamp is dirty
ninja explain: obj/chrome/android/chrome_public_bundle__bundle.stamp is dirty
ninja explain: obj/chrome/android/chrome_public_bundle__unsplit_dex.stamp is dirty
ninja explain: obj/chrome/android/chrome_public_bundle__unsplit_dex__dexspliter.stamp is dirty


agrieve, could you take a look this?
 

Comment 1 by tikuta@chromium.org, Dec 14

Owner: tikuta@chromium.org
Status: Started (was: Untriaged)
Taking a look.

Comment 2 by bugdroid1@chromium.org, Dec 14

Project Member
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/86b46190c5b00c75e4bd4a8597964d769a866df6

commit 86b46190c5b00c75e4bd4a8597964d769a866df6
Author: Takuto Ikuta <tikuta@chromium.org>
Date: Fri Dec 14 15:03:41 2018

[android] use copy in dexsplitter instead of move

Without this, following command always re-run dexsplitter.py.
$ ninja -C out/android-rel/ -dexplain -v chrome_public_bundle__unsplit_dex__dexspliter

Also fix a typo.

Bug:  914707 
Change-Id: Ia827d1b0f87c4993e2837af720febefe22cef85f
Reviewed-on: https://chromium-review.googlesource.com/c/1378150
Commit-Queue: agrieve <agrieve@chromium.org>
Reviewed-by: agrieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/master@{#616672}
[modify] https://crrev.com/86b46190c5b00c75e4bd4a8597964d769a866df6/build/android/gyp/dexsplitter.py
[modify] https://crrev.com/86b46190c5b00c75e4bd4a8597964d769a866df6/build/config/android/rules.gni

Comment 3 by tikuta@chromium.org, Dec 14

Status: Fixed (was: Started)
No warning in 'compile confirm no-op' step.
https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/android-rel/5625

Sign in to add a comment