Open GoogleCodeExporter opened 9 years ago
https://webrtc-codereview.appspot.com/1027010/ will probably solve this with
the rolling of the chromium_revision.
Original comment by kjellan...@google.com
on 16 Jan 2013 at 9:55
chromium deps doesnt fix gtest
d:\src\libjingle\trunk>call python build/gyp_chromium --depth=. -fninja -G
msvs_version=2012 talk/libjingle_all.gyp
Updating projects from gyp files...
ninja: Entering directory `out\Debug'
[401/2167] CXX obj\third_party\gtest\src\gunit.gtest-all.obj
FAILED: ninja -t msvc -o obj\third_party\gtest\src\gunit.gtest-all.obj -e
environment.x86 -- cl.exe /nologo /showIncludes /FC @obj\t
hird_party\gtest\src\gunit.gtest-all.obj.rsp /c
..\..\third_party\gtest\src\gtest-all.cc
/Foobj\third_party\gtest\src\gunit.gtest-a
ll.obj /Fdgunit.pdb
d:\src\libjingle\trunk\third_party\gtest\include\gtest\gtest-printers.h(550) :
error C2977: 'std::tuple' : too many template argumen
ts
c:\program files (x86)\microsoft visual studio 11.0\vc\include\utility(73) : see declaration of 'std::tuple'
d:\src\libjingle\trunk\third_party\gtest\include\gtest\gtest-printers.h(558) :
error C2977: 'std::tuple' : too many template argumen
Original comment by fbarch...@google.com
on 16 Jan 2013 at 11:01
I think this failure is caused by the special handling of gtest in the
libjingle_test.gyp. I think Windows have problems compiling the
'<(DEPTH)/third_party/gtest/src/gtest-all.cc',
Ronghua: can you have a look at using
'<(DEPTH)/testing/gtest.gyp:gtest',
as a dependency instead in the .gyp?
That would work if the DEPS file is updated according to the patch attached to
this comment. It moves gtest from third_party/ to testing/ instead, so the
testing/gtest.gyp from Chrome can be used.
I was looking at creating a CL to resolve this, but I wasn't sure how you use
the <(android_ndk_include) variable in the libjingle_tests.gyp, so I think
you'll have to do it.
Original comment by kjellan...@google.com
on 18 Jan 2013 at 3:03
Attachments:
Original issue reported on code.google.com by
fbarch...@google.com
on 13 Jan 2013 at 6:58