Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(2318)

Issue 5023042: Refactor the bench and common gyp/gypi files to allow for building bench within (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 9 months ago by seanpaul
Modified:
12 years, 9 months ago
Reviewers:
bungeman, epoger, bsalomon
CC:
skia-review_googlegroups.com
Base URL:
http://skia.googlecode.com/svn/trunk/
Visibility:
Public.

Description

Refactor the bench and common gyp/gypi files to allow for building bench within a chromium checkout (using chromium build configuration). BUG=362 TEST=Build bench from within the chromium and skia source tree, ensured it built and ran successfully. Committed: http://code.google.com/p/skia/source/detail?r=2270

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+244 lines, -207 lines) Patch
M gyp/bench.gyp View 1 2 chunks +2 lines, -67 lines 0 comments Download
A gyp/bench.gypi View 1 1 chunk +78 lines, -0 lines 0 comments Download
M gyp/common.gypi View 3 chunks +21 lines, -140 lines 0 comments Download
A gyp/common_conditions.gypi View 1 1 chunk +122 lines, -0 lines 0 comments Download
A gyp/common_variables.gypi View 1 1 chunk +21 lines, -0 lines 0 comments Download

Messages

Total messages: 6
seanpaul
PTAL
12 years, 9 months ago (2011-09-14 15:56:11 UTC) #1
bungeman
On 2011/09/14 15:56:11, seanpaul wrote: > PTAL LGTM
12 years, 9 months ago (2011-09-14 16:10:52 UTC) #2
bsalomon
On 2011/09/14 16:10:52, bungeman wrote: > On 2011/09/14 15:56:11, seanpaul wrote: > > PTAL > ...
12 years, 9 months ago (2011-09-14 17:19:08 UTC) #3
epoger
On 2011/09/14 17:19:08, bsalomon wrote: > On 2011/09/14 16:10:52, bungeman wrote: > > On 2011/09/14 ...
12 years, 9 months ago (2011-09-14 17:48:45 UTC) #4
bsalomon
On Wed, Sep 14, 2011 at 1:48 PM, <epoger@google.com> wrote: > On 2011/09/14 17:19:08, bsalomon ...
12 years, 9 months ago (2011-09-14 17:53:51 UTC) #5
epoger
12 years, 9 months ago (2011-09-14 18:15:21 UTC) #6
On 2011/09/14 17:53:51, bsalomon wrote:
> On Wed, Sep 14, 2011 at 1:48 PM, <mailto:epoger@google.com> wrote:
> 
> > On 2011/09/14 17:19:08, bsalomon wrote:
> >
> >> On 2011/09/14 16:10:52, bungeman wrote:
> >> > On 2011/09/14 15:56:11, seanpaul wrote:
> >> > > PTAL
> >> >
> >> > LGTM
> >>
> >
> >  LGTM
> >>
> >
> > I'm glad to see that this works, but at the same time I'm not happy with
> > the overarching direction (adding yet another layer of complexity to an
> > already-complex build system).
> >
> > I would rather see us go in the direction of skia providing one or two
> > main gyp files that any skia-dependent software package can depend on to
> > build commonly useful targets.
> >
> > What's the use case here?  Who wants to build bench within a chromium
> > checkout?
> >
> >
> >
>
http://codereview.appspot.com/**5023042/%3Chttp://codereview.appspot.com/5023...>
> >
> 
> 
> The goal is to build bench within a CrOS checkout in order to assess
> performance. We'd like to benchmark the skia lib as built for chrome, with
> its decisions about what #define flags to set, optional files to build, and
> compiler flags.

LGTM then.  If I were a stickler for such things, I would say "where's the bug
indicating the need for this"?  Luckily I'm not.  Or am I?
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b