Commit Graph

63 Commits

Author SHA1 Message Date
Ravi Mistry
5fc8912087 Add bin/try-clients to trigger client tryjobs
Will trigger the following:
  skia.primary:
    Build-Debian9-Clang-arm-Release-Flutter_Android
    Build-Debian9-Clang-cf_x86_phone-eng-Android_Framework
    Build-Debian9-Clang-host-sdk-Android_Framework
    Build-Debian9-Clang-TAP-Presubmit-G3_Framework
  luci.chromium.try:
    mac_chromium_compile_dbg_ng

Bug: skia:9154
Change-Id: I88a21865777f7da52e8333920ad9d10da5e788ec
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/219482
Reviewed-by: Eric Boren <borenet@google.com>
Commit-Queue: Ravi Mistry <rmistry@google.com>
2019-06-11 16:55:53 +00:00
Hal Canary
ba1602896d SkQP: new docker test code
`tools/skqp/docker_run_apk.sh` executes SkQP tests in an emulator

`tools/skqp/run_apk.sh` is used by `docker_run_apk.sh` and
`tools/skqp/test_apk.sh` to factor out common code.

Also: `bin/sysopen` now pipes output to `/dev/null` and doesn't block.

TODO: harmonize with test code executed by bots in `infra/skqp/`.

No-Try: true
Change-Id: Ia212a84565ff52279a845e20372a0ad7cc0726a4
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/209401
Reviewed-by: Hal Canary <halcanary@google.com>
Commit-Queue: Hal Canary <halcanary@google.com>
2019-04-19 18:50:03 +00:00
Hal Canary
8c2ccf4764 tools: bin/sync just alias for git-sync-deps now
No-Try: true
Change-Id: Id323d99c4ad0526980330daf41752452e6b09c95
Reviewed-on: https://skia-review.googlesource.com/c/194381
Auto-Submit: Hal Canary <halcanary@google.com>
Commit-Queue: Kevin Lubick <kjlubick@google.com>
Reviewed-by: Kevin Lubick <kjlubick@google.com>
2019-02-22 20:05:05 +00:00
Hal Canary
b45d512481 bin/sysopen: script moved to bin
No-Try: true
Change-Id: I1b9b1be36c9ed2d574de681b90b98e56f45e70fa
Reviewed-on: https://skia-review.googlesource.com/c/174588
Auto-Submit: Hal Canary <halcanary@google.com>
Reviewed-by: Hal Canary <halcanary@google.com>
Commit-Queue: Hal Canary <halcanary@google.com>
2018-12-06 14:21:33 +00:00
Hal Canary
ed18c93d1e bin/gerrit-number (no longer experimental)
Change-Id: Iadbef27074201173be2c71447c5d6f721afffbb3
Reviewed-on: https://skia-review.googlesource.com/c/174585
Reviewed-by: Ben Wagner <bungeman@google.com>
Commit-Queue: Hal Canary <halcanary@google.com>
2018-12-05 16:47:29 +00:00
Brian Salomon
4db65d4d39 Add script to pull svgs to svgs/ and add to .gitignore
Change-Id: I77f78e7b27963a087ace30b8ce41118dd76c9026
Reviewed-on: https://skia-review.googlesource.com/150914
Reviewed-by: Hal Canary <halcanary@google.com>
Reviewed-by: Florin Malita <fmalita@chromium.org>
Commit-Queue: Brian Salomon <bsalomon@google.com>
2018-08-31 20:02:24 +00:00
Hal Canary
9db44ed0ef bin/droid script: fix wierd mkdir error on my device
Here's the error:
    adb: error: failed to copy '...' to '...':
    remote secure_mkdirs failed: Operation not permitted

I also cleaned up the script.

No-Try: true
Change-Id: Ie2dec7b42eaf2a19475cdd0a04074f8bec47b2fb
Reviewed-on: https://skia-review.googlesource.com/148906
Auto-Submit: Hal Canary <halcanary@google.com>
Commit-Queue: Mike Klein <mtklein@google.com>
Reviewed-by: Mike Klein <mtklein@google.com>
2018-08-23 14:46:19 +00:00
Mike Klein
25e2559656 add --sync to adb push in bin/droid
This skips pushing files that are already present.

Change-Id: Ibe86be1e553da3cbe9d02d00764164d9e0745f8d
Reviewed-on: https://skia-review.googlesource.com/138164
Commit-Queue: Mike Klein <mtklein@chromium.org>
Commit-Queue: Hal Canary <halcanary@google.com>
Auto-Submit: Mike Klein <mtklein@chromium.org>
Reviewed-by: Hal Canary <halcanary@google.com>
2018-06-28 15:46:57 +00:00
Ben Wagner
bb9cad053b Update gn
The currrent gn is quite old. The new gn, among other fixes, now supports
checking includes.

The values come from
893eb86b02

Change-Id: Ia2e47e18c7bbd3219a258fd6cc9063bf38588d00
Reviewed-on: https://skia-review.googlesource.com/129720
Commit-Queue: Mike Klein <mtklein@google.com>
Reviewed-by: Mike Klein <mtklein@google.com>
2018-05-23 12:11:06 +00:00
Mike Klein
cdcadf7a29 remove fetch-gn dependency on buildtools
This makes it possible to fetch GN without syncing DEPS.

To roll GN now, just update the three hashes in bin/fetch-gn.

buildtools is still a dependency of fetch-clang-format.

Bug: skia:7634

Change-Id: Ida6b516cfb0b306da73550875ab1ff97c9da6a64
Reviewed-on: https://skia-review.googlesource.com/107882
Reviewed-by: Hal Canary <halcanary@google.com>
Commit-Queue: Mike Klein <mtklein@chromium.org>
2018-02-21 20:42:26 +00:00
Hal Canary
ec97ac9be2 fetch-clang-format: better error checking
Change-Id: Idc0e984275056128301d4f1ae66aa6ace67b05d5
Reviewed-on: https://skia-review.googlesource.com/98322
Reviewed-by: Hal Canary <halcanary@google.com>
Commit-Queue: Hal Canary <halcanary@google.com>
2018-01-22 20:33:49 +00:00
Herbert Derby
ff889f1339 Add directions for when trouble strikes in fetch-skps.
Change-Id: I53c02c12a327f6484f4e45777c0168065adfae75
Reviewed-on: https://skia-review.googlesource.com/91080
Reviewed-by: Ravi Mistry <rmistry@google.com>
Commit-Queue: Herb Derby <herb@google.com>
2018-01-04 21:31:01 +00:00
Eric Boren
cbe99c050e bin/try: No prompt for a single matching job
NOTRY: true
Bug: skia:
Change-Id: I14e71f96b56b4288807bcc5189593b22d3078824
Reviewed-on: https://skia-review.googlesource.com/83443
Commit-Queue: Eric Boren <borenet@google.com>
Reviewed-by: Ravi Mistry <rmistry@google.com>
2017-12-11 18:32:08 +00:00
Eric Boren
c9080c8846 bin/try: Add interactive mode
Bug: skia:
Change-Id: Iceb1834f1a3aa7594227ff3adf187c650aa30367
Reviewed-on: https://skia-review.googlesource.com/61480
Commit-Queue: Eric Boren <borenet@google.com>
Reviewed-by: Ravi Mistry <rmistry@google.com>
2017-10-18 17:16:16 +00:00
Eric Boren
d5c128bd7e bin/try: add support for internal bots
Bug: skia:
Change-Id: Iae6b2dc1098c9763ceec0289129818e1948a974c
Reviewed-on: https://skia-review.googlesource.com/60543
Reviewed-by: Ravi Mistry <rmistry@google.com>
Commit-Queue: Eric Boren <borenet@google.com>
2017-10-17 21:10:15 +00:00
Eric Boren
cff9f956c7 bin/try: Add support for select Chromium trybots
These are hard-coded in update_meta_config.py, which is used to set the
list of trybots in Gerrit.

Bug: skia:
Change-Id: I6e512e4cb59974e27e8954cf5134c70068e716f3
Reviewed-on: https://skia-review.googlesource.com/60521
Reviewed-by: Ravi Mistry <rmistry@google.com>
Commit-Queue: Eric Boren <borenet@google.com>
2017-10-17 13:43:42 +00:00
Eric Boren
6376517472 Add bin/try script
Wrapper for "git cl try" which allows searching jobs using regular
expressions.

Bug: skia:
Change-Id: I07a1b487f2eca9da7df4c4d4c65e22e459b76fed
Reviewed-on: https://skia-review.googlesource.com/60261
Commit-Queue: Eric Boren <borenet@google.com>
Reviewed-by: Kevin Lubick <kjlubick@google.com>
Reviewed-by: Ravi Mistry <rmistry@google.com>
2017-10-16 19:37:11 +00:00
Mike Klein
8e3c42b1ee add a Win/Clang build bot
BUG=skia:2679

CQ_INCLUDE_TRYBOTS=skia.primary:Build-Win-Clang-x86_64-Release

Change-Id: I861366ae1340abcc67e87041904d08337b465bbe
Reviewed-on: https://skia-review.googlesource.com/28864
Reviewed-by: Eric Boren <borenet@google.com>
2017-07-31 19:53:51 +00:00
Mike Klein
c722f79b6c clang on windows support
1) Run python bin/fetch-clang-win
2) Set clang_win = "../bin/clang_win"
3) ???
4) Profit

Most changes here are to pass the right -mfoo flags to Clang
to enable advanced instruction sets, or fixed warning-as-errors.

BUG=skia:2679

Change-Id: Ieed145d35c209131c7c16fdd3ee11a3de4a1a921
Reviewed-on: https://skia-review.googlesource.com/28740
Reviewed-by: Ben Wagner <bungeman@google.com>
Commit-Queue: Mike Klein <mtklein@chromium.org>
2017-07-31 18:39:23 +00:00
Hal Canary
b357dea546 bin/fetch-clang-format
Change-Id: I7deb9b49b4a3a361888a6afc07b4c6341a4b16cf
Reviewed-on: https://skia-review.googlesource.com/9410
Reviewed-by: Mike Klein <mtklein@chromium.org>
Commit-Queue: Hal Canary <halcanary@google.com>
2017-03-08 16:43:49 +00:00
Kevin Lubick
0168e0442e Add a way to compute the code coverage of fuzzers
This also fixes bin/coverage.

fuzz/coverage borrows heavily from bin/coverage.
BUG=skia:

Change-Id: I9e353d1f5ea3bca1d57d66b1c1ecabc6f9b23cee
Reviewed-on: https://skia-review.googlesource.com/8414
Reviewed-by: Joe Gregorio <jcgregorio@google.com>
Commit-Queue: Kevin Lubick <kjlubick@google.com>
2017-02-15 13:19:40 +00:00
Mike Klein
160907f8de Clean up more references to GYP.
Delete files only used by GYP, and files that used GYP.  Neither can
possibly be actively used.  Beyond that, just a couple doc tweaks.

Change-Id: I0220d7226e7bb9ed7c54a7d8f2906a718313c521
Reviewed-on: https://skia-review.googlesource.com/8062
Commit-Queue: Mike Klein <mtklein@chromium.org>
Reviewed-by: Hal Canary <halcanary@google.com>
2017-02-06 17:55:35 +00:00
Hal Canary
cae159e760 bin/list-skia-bots
NOTRY=true
Change-Id: Ie123bf796ff062972fd86319f0a20f3a81a9b2ff
Reviewed-on: https://skia-review.googlesource.com/7982
Reviewed-by: Kevin Lubick <kjlubick@google.com>
Commit-Queue: Hal Canary <halcanary@google.com>
2017-02-02 21:23:48 +00:00
Hal Canary
7f660e8418 bin/fetch-gn is 3.5x faster
Change-Id: I8098a38eb81efe0a73201c1f00dff8363b9c2075
Reviewed-on: https://skia-review.googlesource.com/7025
Commit-Queue: Hal Canary <halcanary@google.com>
Reviewed-by: Mike Klein <mtklein@chromium.org>
2017-01-30 16:22:22 +00:00
Hal Canary
4d3adb6b0d bin/sync : call tools/git-sync-deps now
Also, update some docs.

NOTRY=true
Change-Id: I7ad3375fc1cbf8f71ed42a460ecfe29ef6c1d85e
Reviewed-on: https://skia-review.googlesource.com/7657
Reviewed-by: Hal Canary <halcanary@google.com>
Commit-Queue: Hal Canary <halcanary@google.com>
2017-01-27 16:03:35 +00:00
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