gtest parameters

[GoogleCodeExporter]

Change environment variables to command line parameters.

Original issue reported on code.google.com by fbarch...@chromium.org on 16 Oct 2015 at 5:45

[GoogleCodeExporter]

r1513 adds gtest to DEPS

Original comment by fbarch...@chromium.org on 16 Oct 2015 at 5:58

• Changed state: Started

[GoogleCodeExporter]

[deleted comment]

[GoogleCodeExporter]

GYP_DEFINES="OS=android" GYP_CROSSCOMPILE=1 ./gyp_libyuv
ninja -j7 -C out/Release libyuv_unittest_apk

ninja: Entering directory `out/Release'
ninja: error: '../../third_party/gflags/src/gflags.cc', needed by 
'obj/third_party/gflags/src/gflags.gflags.o', missing and no known rule to make 
it

if I copy the source from webrtc, it builds.

Original comment by fbarch...@google.com on 17 Oct 2015 at 7:32

[GoogleCodeExporter]

Sorry about that, I accidentally committed a CL syncing gflags into 'src' 
instead of the 'libyuv' directory, not paying attention to that this is 
different for libyuv.
I'm fixing this in https://codereview.chromium.org/1410673004/

After that the source should be put in the right location.

Original comment by kjellander@chromium.org on 18 Oct 2015 at 6:03

[GoogleCodeExporter]

Use -a to pass test arguements

util/android/test_runner.py gtest -s libyuv_unittest --verbose --release 
--gtest_filter=*I420ToARGB* -a "--libyuv_width=1280 --libyuv_height=720 
--libyuv_repeat=999 --libyuv_flags=-1"

Original comment by fbarch...@google.com on 19 Oct 2015 at 9:38

[GoogleCodeExporter]

Original comment by fbarch...@google.com on 19 Oct 2015 at 11:11

[GoogleCodeExporter]

Fixed in r1519
-a bug in chromium fixed. rolled to new chromium and it works

e.g.
util/android/test_runner.py gtest -s libyuv_unittest -t 1800 --verbose 
--release --gtest_filter=*ColorTest* -a "--libyuv_width=1280 
--libyuv_height=720 --libyuv_repeat=999 --libyuv_flags=-1"

Original comment by fbarch...@chromium.org on 22 Oct 2015 at 6:07

• Changed state: Fixed