Commit Graph

38 Commits

Author SHA1 Message Date
Mike Klein
b621a35b43 drop depot_tools dependency in bin/fetch-gn
... and copy gn to bin/ when done to make it easy to find.

Change-Id: I1ec405b4c45efb828626ff7d904a417f69b39cb2
Reviewed-on: https://skia-review.googlesource.com/6962
Reviewed-by: Ben Wagner <bungeman@google.com>
Commit-Queue: Mike Klein <mtklein@chromium.org>
2017-01-12 19:54:50 +00:00
Hal Canary
a4a3381790 tools and docs: clean up gyp-specific things
BUG=skia:

DOCS_PREVIEW= https://skia.org/?cl=5770

Change-Id: Iadc436a68cbf7ec0d1dd3c019072eb28bf589bb6
Reviewed-on: https://skia-review.googlesource.com/5770
Commit-Queue: Hal Canary <halcanary@google.com>
Reviewed-by: Mike Klein <mtklein@chromium.org>
2016-12-09 21:15:27 +00:00
Mike Klein
82364bae16 Build for Android from Windows, work in progress.
Pretty vanilla stuff here, mostly just making the gcc-like toolchain Windows friendly.

I was having trouble getting rm -r {{output}} && $ar rcs {{output}} @$rspfile to work without deleting my ar.exe, so I chickened out the usual way by adding gn/ar.py.

I've also updated bin/droid to work with Git Bash on Windows.

BUG=skia:

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=3751

Change-Id: I04c34ccc91e6a291c11ac4e7a7a0ffe41d879fe6
Reviewed-on: https://skia-review.googlesource.com/3751
Reviewed-by: Brian Osman <brianosman@google.com>
Commit-Queue: Mike Klein <mtklein@chromium.org>
2016-10-26 14:37:43 +00:00
Hal Canary
23510f2c29 bin/sync: call fetch-gn when DEPS changes
Change-Id: Ifb2a364df1a8817152a2c1e79748d14534a9e523
Reviewed-on: https://skia-review.googlesource.com/3904
Reviewed-by: Mike Klein <mtklein@chromium.org>
Commit-Queue: Hal Canary <halcanary@google.com>
2016-10-25 16:15:12 +00:00
halcanary
3948a1bf18 bin/coverage: a GN version of the coverage script
intended to replace tools/coverage when GYP is gone.

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2343243002

Review-Url: https://codereview.chromium.org/2343243002
2016-09-26 10:30:39 -07:00
egdaniel
c19440dcaf Update GN droid script
Splits the pushes into different lines since old adbs cannot handle the one
line command

TBR=mtklein@google.com

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2339293002

Review-Url: https://codereview.chromium.org/2339293002
2016-09-14 14:42:03 -07:00
mtklein
f3155ad97c GN: add a helper script for running Android builds
Think of this as the spiritual equivalent of platform_tools/android/bin/android_run_skia, but for GN and easier to type.

Cutting down the debug symbols makes Android builds about 1/4 the size.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2323833002
DOCS_PREVIEW= https://skia.org/user/quick/gn?cl=2323833002

Review-Url: https://codereview.chromium.org/2323833002
2016-09-08 10:03:21 -07:00
mtklein
cdc8942fd6 cleanup dead nanomsg and build_overrides
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2287013003
TBR=

Review-Url: https://codereview.chromium.org/2287013003
2016-08-27 06:50:11 -07:00
herb
35f3e59d1c Pick the right download tool for windows.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2197423002

Review-Url: https://codereview.chromium.org/2197423002
2016-08-01 12:26:51 -07:00
borenet
1543487fc6 Change SKPs to new asset management format
BUG=skia:5427
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2169653003
DOCS_PREVIEW= https://skia.org/?cl=2169653003

Review-Url: https://codereview.chromium.org/2169653003
2016-07-22 08:31:07 -07:00
mtklein
6af0c958a0 GN: restore gn.py partly as fetch-gn
Same as before, except it explicitly only fetches GN instead of acting
like a proxy for it.  I found myself always typing 'gn' anyway.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2167353002

Review-Url: https://codereview.chromium.org/2167353002
2016-07-22 03:34:25 -07:00
halcanary
298ab76f1a bin/fetch-skps: print instruction on error
NOTRY=true

Review URL: https://codereview.chromium.org/1919133003
2016-04-26 12:38:47 -07:00
halcanary
21736bd8b5 bin/sync
Review URL: https://codereview.chromium.org/1915223002
2016-04-25 13:34:06 -07:00
halcanary
5c4685e0b0 bin/sync-and-gyp: more dirs to search
Review URL: https://codereview.chromium.org/1914993003
2016-04-25 12:39:39 -07:00
bsalomon
3d7b02c728 Fix gyp-and-sync for windows
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1715423003

Review URL: https://codereview.chromium.org/1715423003
2016-02-22 12:59:53 -08:00
halcanary
68af68a0ef bin/sync-and-gyp: better error checking
NOTRY=true

Review URL: https://codereview.chromium.org/1704143002
2016-02-17 13:07:37 -08:00
halcanary
117672db3f bin/sync-and-gyp: pass --deps='...' to gclient sync.
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1705063002

Review URL: https://codereview.chromium.org/1705063002
2016-02-17 08:39:23 -08:00
mtklein
dc469adf04 come on, this is how you drive-by
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1708563002

Review URL: https://codereview.chromium.org/1708563002
2016-02-17 06:37:51 -08:00
halcanary
2362c476ef bin/sync-and-gyp: start using gclient
stop using hard-to-maintain git-sync-deps

BUG=skia:4885

GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1693963005

Review URL: https://codereview.chromium.org/1693963005
2016-02-16 11:48:06 -08:00
halcanary
a4c26c0d3a bin/sync-and-gyp: sh->py (make more cross-platform)
Review URL: https://codereview.chromium.org/1425593011
2015-11-09 08:28:13 -08:00
herb
dad837ace1 compare has a syntax error where it is missing the path to sem().
BUG=skia:

Review URL: https://codereview.chromium.org/1420963010
2015-11-06 10:35:37 -08:00
mtklein
143fd5547c bin/deps-and-gyp -> bin/sync-and-gyp
NOTREECHECKS=true

BUG=skia:

Review URL: https://codereview.chromium.org/1423463005
2015-11-03 12:07:47 -08:00
halcanary
4a40a46c65 bin/deps-and-gyp: deal with corner cases cleanly
TBR=mtklein@google.com
NOTRY=true

Review URL: https://codereview.chromium.org/1415193006
2015-11-03 11:18:11 -08:00
halcanary
0575131e57 bin/deps-and-gyp: gclient-free solution
Review URL: https://codereview.chromium.org/1426103004
2015-11-03 10:30:05 -08:00
halcanary
1f17364dc8 bin/sync-and-gyp: gclient hackery
Review URL: https://codereview.chromium.org/1427183003
2015-11-02 10:22:43 -08:00
mtklein
24a592c2ef Make bin/c and bin/compare work on Windows.
- Call python explicitly.
  - Drop numpy dependency (on numpy.mean.... come on.)
  - Make scipy dependency optional.

Depends on https://codereview.chromium.org/1419073003 to really work.

BUG=skia:

Doesn't change code.
NOTRY=true

Review URL: https://codereview.chromium.org/1416833004
2015-10-28 09:45:44 -07:00
Hal Canary
94475cdbfb sync-and-gyp: Update shell script to correct the syntax of functions
R=halcanary@gmail.com

Review URL: https://codereview.chromium.org/1342543003 .
2015-09-11 20:49:00 -04:00
herb
ea73fa529f Add instructions to authenticate for fetch.
BUG=skia:

Review URL: https://codereview.chromium.org/1305143002
2015-08-21 08:38:06 -07:00
herb
af4edf9ccc Change to use mean and to use stderr.
BUG=skia:

Review URL: https://codereview.chromium.org/1228783003
2015-07-09 10:50:24 -07:00
cdalton
2c56ba5cde Don't suppress nanobench output table in verbose mode
Changes verbose mode to print both the table and the individual sample
values. No need to hold back information in verbose mode.

BUG=skia:

Review URL: https://codereview.chromium.org/1208763003
2015-06-26 13:32:53 -07:00
halcanary
43315bbe4b sync-and-gyp: make smarter
This fixes two issues: If gyp is run with the CXX and CC environment
variables set, it will produce different ninja build files.
sync-and-gyp now tracks the values of those variables; if they change,
re-run gyp.

Secondly, we introduced find.py, which is used inside gyp files.  If
the contents of certain directories changes, gyp should be re-run.  So
now we track those directories too.

Review URL: https://codereview.chromium.org/1153083002
2015-05-26 08:01:19 -07:00
kkinnunen
c77f6af7c3 Make the c and ac scripts work with sh and without bin in PATH
Make the bin/c and bin/ac scripts work with sh. The scripts are run with
/bin/sh shebang, which fails atleast on ubuntu 12.04 /bin/sh. The sh in
Ubuntu 12.04 is dash.

The fixes are according to the suggestions in http://mywiki.wooledge.org/Bashism

Also run "compare" script with explicit ./bin/ path to support people
who do not have skia/bin in PATH.

Review URL: https://codereview.chromium.org/1139033005
2015-05-15 05:35:50 -07:00
halcanary
8440baa5d5 sync-and-gyp script: fix to work in MSYS shell
NOTRY=true
TBR=scroggo@google.com

Review URL: https://codereview.chromium.org/1104613003
2015-04-24 14:33:06 -07:00
halcanary
135b7ecaa8 tools: add sync-and-gyp script
NOTRY=true

Review URL: https://codereview.chromium.org/1035003004
2015-03-27 12:11:49 -07:00
mtklein
674cd7e05c Add a script to fetch the latest SKPs.
I keep forgetting how best to do this.

NOTRY=true
NOTREECHECKS=true

Review URL: https://codereview.chromium.org/880283002
2015-01-28 09:39:10 -08:00
mtklein
205de46789 run clean branch baseline only once
BUG=skia:

Review URL: https://codereview.chromium.org/874643002
2015-01-23 10:39:55 -08:00
Mike Klein
8a84db909a Don't count a leading 1 as a signficant digit in the ratio.
What used to look like this:
            desk_pokemonwiki.skp    9.38ms -> 9.76ms    1x
                 tabl_pravda.skp     237us ->  241us    1x
          desk_css3gradients.skp     249us ->  254us    1x
            ....
               desk_fontwipe.skp    39.6us -> 38.7us    0.98x
                   tabl_digg.skp     922us ->  893us    0.97x
                  tabl_gmail.skp    20.7us ->   20us    0.96x

Now will print more like this:

            desk_pokemonwiki.skp    9.38ms -> 9.76ms    1.04x
                 tabl_pravda.skp     237us ->  241us    1.02x
          desk_css3gradients.skp     249us ->  254us    1.02x
            ....
               desk_fontwipe.skp    39.6us -> 38.7us    0.98x
                   tabl_digg.skp     922us ->  893us    0.97x
                  tabl_gmail.skp    20.7us ->   20us    0.96x

BUG=skia:

Review URL: https://codereview.chromium.org/756643004
2014-11-24 17:44:23 -05:00
mtklein
7ba39cb9a6 Add nanobench stats scripts to Skia repo.
These are the scripts I've been homegrowing for measuring perf impact.  I think we found them useful today as a way of sifting through the noise.

BUG=skia:

Review URL: https://codereview.chromium.org/703713002
2014-11-24 12:39:59 -08:00