Chromium Code Reviews
Help | Chromium Project | Sign in
(2)

Issue 10696047: Pass android_sdk as a variable to gyp (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 year, 9 months ago by Steve Block
Modified:
1 year, 9 months ago
CC:
chromium-reviews_chromium.org
Visibility:
Public.

Description

Pass android_sdk as a variable to gyp

This avoids the need for shell script in the gyp file.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=145195

Patch Set 1 #

Patch Set 2 : Fixed path #

Patch Set 3 : Struggling with escaping #

Patch Set 4 : Fix style #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -8 lines) Lint Patch
M build/android/envsetup.sh View 1 2 3 1 chunk +2 lines, -0 lines 2 comments ? errors Download
M build/java_aidl.gypi View 1 chunk +0 lines, -8 lines 0 comments ? errors Download
Commit:

Messages

Total messages: 14
Steve Block
I can resolve the 'ANDROID_BUILD_TOP' case when this merges downstream.
1 year, 9 months ago #1
navabi
lgtm but someone else must approve. http://codereview.chromium.org/10696047/diff/3003/build/android/envsetup.sh File build/android/envsetup.sh (right): http://codereview.chromium.org/10696047/diff/3003/build/android/envsetup.sh#newcode154 build/android/envsetup.sh:154: " android_sdk=${ANDROID_SDK_ROOT}/platforms/android-${ANDROID_SDK_VERSION}" This ...
1 year, 9 months ago #2
Steve Block
http://codereview.chromium.org/10696047/diff/3003/build/android/envsetup.sh File build/android/envsetup.sh (right): http://codereview.chromium.org/10696047/diff/3003/build/android/envsetup.sh#newcode154 build/android/envsetup.sh:154: " android_sdk=${ANDROID_SDK_ROOT}/platforms/android-${ANDROID_SDK_VERSION}" Right, I'll resolve the 'ANDROID_BUILD_TOP' case when ...
1 year, 9 months ago #3
navabi
lgtm
1 year, 9 months ago #4
I haz the power (commit-bot)
No LGTM from a valid reviewer yet. Only full committers are accepted. Even if an ...
1 year, 9 months ago #5
joth (inactive)
lgtm
1 year, 9 months ago #6
I haz the power (commit-bot)
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/steveblock@chromium.org/10696047/3003
1 year, 9 months ago #7
I haz the power (commit-bot)
Change committed as 145195
1 year, 9 months ago #8
rwalker
Seems to break these instructions: http://code.google.com/p/chromium/wiki/AndroidBuildInstructions
1 year, 9 months ago #9
John Grabowski
LGTM Thanks for paying attention to the small things like this.
1 year, 9 months ago #10
Steve Block
> Seems to break these instructions: Can you be more specific as to the problem ...
1 year, 9 months ago #11
rwalker
On 2012/07/03 09:53:16, Steve Block wrote: > > Seems to break these instructions: > Can ...
1 year, 9 months ago #12
joth (inactive)
chromium on Linux has the same boot strap issue ( http://code.google.com/p/chromium/wiki/UsingNewGit#Initial_checkout -- "Do this to ...
1 year, 9 months ago #13
Steve Block
1 year, 9 months ago #14
Got it, I see what you mean. In fact, this problem existed before my change,
only it didn't manifest itself as an error from gclient sync.

I've updated the documentation.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1280:2d3e6564b7b6