Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(138)

Issue 145313005: Fix temporary_android_dummy_target in build/all_android.gyp. (Closed)

Created:
6 years, 11 months ago by Bernhard Bauer
Modified:
6 years, 11 months ago
Reviewers:
Nico
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Visibility:
Public.

Description

Fix temporary_android_dummy_target in build/all_android.gyp. TBR=thakis@chromium.org BUG=none NOTRY=true Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=246883

Patch Set 1 #

Patch Set 2 : x #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -0 lines) Patch
M build/all_android.gyp View 1 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
Bernhard Bauer
Apologies for the double-commit; I thought fixing the syntax error would be sufficient.
6 years, 11 months ago (2014-01-24 13:43:11 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/bauerb@chromium.org/145313005/30001
6 years, 11 months ago (2014-01-24 13:43:22 UTC) #2
commit-bot: I haz the power
Change committed as 246883
6 years, 11 months ago (2014-01-24 13:43:33 UTC) #3
Nico
Lgtm Thanks & sorry :-/ since you saw this: How often do you source envsetup? ...
6 years, 11 months ago (2014-01-24 14:09:22 UTC) #4
Bernhard Bauer
On Fri, Jan 24, 2014 at 2:09 PM, Nico Weber <thakis@chromium.org> wrote: > Lgtm > ...
6 years, 11 months ago (2014-01-24 14:32:53 UTC) #5
Nico
On Fri, Jan 24, 2014 at 6:32 AM, Bernhard Bauer <bauerb@chromium.org> wrote: > On Fri, ...
6 years, 11 months ago (2014-01-24 15:11:24 UTC) #6
Bernhard Bauer
6 years, 11 months ago (2014-01-24 15:43:58 UTC) #7
Once every computer restart, which varies, but every couple of weeks sounds
about right.

Bernhard.


On Fri, Jan 24, 2014 at 3:11 PM, Nico Weber <thakis@chromium.org> wrote:

> On Fri, Jan 24, 2014 at 6:32 AM, Bernhard Bauer <bauerb@chromium.org>wrote:
>
>> On Fri, Jan 24, 2014 at 2:09 PM, Nico Weber <thakis@chromium.org> wrote:
>>
>>> Lgtm
>>>
>>> Thanks & sorry :-/
>>>
>>> since you saw this: How often do you source envsetup? (This wasn't
>>> happening on bots, right?)
>>>
>> Yeah, it didn't happen on the bots.
>>
>> I almost never source envsetup, because I keep a shell open for Android
>> building.
>>
>> Is there some way we could automatize re-sourcing envsetup, similar to
>> using landmines for clobbering?
>>
>
> I'm trying to remove envsetup. Once it's gone, it doesn't really matter I
> suppose.
>
> What's "almost never"? Like, every 4 weeks?
>
>
>>
>>
>>>  On Jan 24, 2014 5:43 AM, <commit-bot@chromium.org> wrote:
>>>
>>>> Change committed as 246883
>>>>
>>>> https://codereview.chromium.org/145313005/
>>>>
>>>
>>
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

Powered by Google App Engine
This is Rietveld 408576698