Build just chrome to shorten the Coverity build and analysis...

Build just chrome to shorten the Coverity build and analysis
time.

Remove a stale TODO comment.

R=bevc,jimhebert
BUG=none
TEST=none

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

[wtc, 2010-08-10 21:36:07]

Bev, Jim:

Here are the command-line changes to build just
chrome.  Let's coordinate and make these changes
to our three Coverity machines.

[Mark Mentovai, 2010-08-10 21:43:53]

http://codereview.chromium.org/3175001/diff/1/2
File tools/coverity/coverity.py (right):

http://codereview.chromium.org/3175001/diff/1/2#newcode184
tools/coverity/coverity.py:184: os.chdir('src/build')
This directory…

http://codereview.chromium.org/3175001/diff/1/2#newcode186
tools/coverity/coverity.py:186: cmd = ('%s/cov-build --dir %s xcodebuild
-project chrome.xcodeproj '
…does not contain chrome.xcodeproj

If you were able to use src/build for all.xcodeproj in the past, you should
change it to src/chrome to use chrome.xcodeproj.

[jimhebert, 2010-08-10 23:10:31]

FWIW I don't know enough about the build to sort out between your change and
mark's comment about the chdir.

Pending resolution of that one point, LGTM though.

On Tue, Aug 10, 2010 at 2:43 PM, <mark@chromium.org> wrote:

>
> http://codereview.chromium.org/3175001/diff/1/2
> File tools/coverity/coverity.py (right):
>
> http://codereview.chromium.org/3175001/diff/1/2#newcode184
> tools/coverity/coverity.py:184: os.chdir('src/build')
> This directory…
>
> http://codereview.chromium.org/3175001/diff/1/2#newcode186
> tools/coverity/coverity.py:186: cmd = ('%s/cov-build --dir %s xcodebuild
> -project chrome.xcodeproj '
> …does not contain chrome.xcodeproj
>
> If you were able to use src/build for all.xcodeproj in the past, you
> should change it to src/chrome to use chrome.xcodeproj.
>
>
> http://codereview.chromium.org/3175001/show
>

[bevc, 2010-08-10 23:51:34]

lgtm

[Mark Mentovai, 2010-08-11 15:09:50]

LGTM