[native-toolchain-CR] Work around flatbuffers flaky build
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12886 ) Change subject: Work around flatbuffers flaky build .. Patch Set 4: Verified+1 Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/12886 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: native-toolchain Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I57e95617ccfc2d7d63cc277ab2b858f28ccad44c Gerrit-Change-Number: 12886 Gerrit-PatchSet: 4 Gerrit-Owner: Hector Acosta Gerrit-Reviewer: Laszlo Gaal Gerrit-Reviewer: Tim Armstrong Gerrit-Comment-Date: Mon, 01 Apr 2019 16:20:44 + Gerrit-HasComments: No
[native-toolchain-CR] Work around flatbuffers flaky build
Tim Armstrong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12886 ) Change subject: Work around flatbuffers flaky build .. Work around flatbuffers flaky build The flatbuffers build ocasionally fails with the following error: /flatbuffers/flatbuffers-1.6.0/src/idl_parser.cpp /mnt/source/flatbuffers/flatbuffers-1.6.0/samples/sample_binary.cpp:19:17: error: 'MyGame' has not been declared using namespace MyGame::Sample; ^ /mnt/source/flatbuffers/flatbuffers-1.6.0/samples/sample_binary.cpp:19:25: error: 'Sample' is not a namespace-name using namespace MyGame::Sample; ^ /mnt/source/flatbuffers/flatbuffers-1.6.0/samples/sample_binary.cpp:19:31: error: expected namespace-name before ';' token using namespace MyGame::Sample; ^ /mnt/source/flatbuffers/flatbuffers-1.6.0/samples/sample_binary.cpp: In function 'int main(int, const char**)': /mnt/source/flatbuffers/flatbuffers-1.6.0/samples/sample_binary.cpp:25:3: error: 'flatbuffers' has not been declared flatbuffers::FlatBufferBuilder builder; ^ /mnt/source/flatbuffers/flatbuffers-1.6.0/samples/sample_binary.cpp:28:26: error: 'builder' was not declared in this scope auto weapon_one_name = builder.CreateString("Sword"); ^ ... Disabling the BUILD_TESTS target removes the flakiness and makes the compilation faster. Change-Id: I57e95617ccfc2d7d63cc277ab2b858f28ccad44c Reviewed-on: http://gerrit.cloudera.org:8080/12886 Reviewed-by: Tim Armstrong Tested-by: Tim Armstrong --- M source/flatbuffers/build.sh 1 file changed, 6 insertions(+), 1 deletion(-) Approvals: Tim Armstrong: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/12886 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: native-toolchain Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I57e95617ccfc2d7d63cc277ab2b858f28ccad44c Gerrit-Change-Number: 12886 Gerrit-PatchSet: 5 Gerrit-Owner: Hector Acosta Gerrit-Reviewer: Laszlo Gaal Gerrit-Reviewer: Tim Armstrong
[native-toolchain-CR] Work around flatbuffers flaky build
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12886 ) Change subject: Work around flatbuffers flaky build .. Patch Set 3: Verified+1 Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/12886 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: native-toolchain Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I57e95617ccfc2d7d63cc277ab2b858f28ccad44c Gerrit-Change-Number: 12886 Gerrit-PatchSet: 3 Gerrit-Owner: Hector Acosta Gerrit-Reviewer: Laszlo Gaal Gerrit-Reviewer: Tim Armstrong Gerrit-Comment-Date: Mon, 01 Apr 2019 16:18:29 + Gerrit-HasComments: No
[native-toolchain-CR] Work around flatbuffers flaky build
Laszlo Gaal has posted comments on this change. ( http://gerrit.cloudera.org:8080/12886 ) Change subject: Work around flatbuffers flaky build .. Patch Set 3: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/12886 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: native-toolchain Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I57e95617ccfc2d7d63cc277ab2b858f28ccad44c Gerrit-Change-Number: 12886 Gerrit-PatchSet: 3 Gerrit-Owner: Hector Acosta Gerrit-Reviewer: Laszlo Gaal Gerrit-Comment-Date: Fri, 29 Mar 2019 21:11:57 + Gerrit-HasComments: No
[native-toolchain-CR] Work around flatbuffers flaky build
Hector Acosta has uploaded a new patch set (#3). ( http://gerrit.cloudera.org:8080/12886 ) Change subject: Work around flatbuffers flaky build .. Work around flatbuffers flaky build The flatbuffers build ocasionally fails with the following error: /flatbuffers/flatbuffers-1.6.0/src/idl_parser.cpp /mnt/source/flatbuffers/flatbuffers-1.6.0/samples/sample_binary.cpp:19:17: error: 'MyGame' has not been declared using namespace MyGame::Sample; ^ /mnt/source/flatbuffers/flatbuffers-1.6.0/samples/sample_binary.cpp:19:25: error: 'Sample' is not a namespace-name using namespace MyGame::Sample; ^ /mnt/source/flatbuffers/flatbuffers-1.6.0/samples/sample_binary.cpp:19:31: error: expected namespace-name before ';' token using namespace MyGame::Sample; ^ /mnt/source/flatbuffers/flatbuffers-1.6.0/samples/sample_binary.cpp: In function 'int main(int, const char**)': /mnt/source/flatbuffers/flatbuffers-1.6.0/samples/sample_binary.cpp:25:3: error: 'flatbuffers' has not been declared flatbuffers::FlatBufferBuilder builder; ^ /mnt/source/flatbuffers/flatbuffers-1.6.0/samples/sample_binary.cpp:28:26: error: 'builder' was not declared in this scope auto weapon_one_name = builder.CreateString("Sword"); ^ ... Disabling the BUILD_TESTS target removes the flakiness and makes the compilation faster. Change-Id: I57e95617ccfc2d7d63cc277ab2b858f28ccad44c --- M source/flatbuffers/build.sh 1 file changed, 6 insertions(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/native-toolchain refs/changes/86/12886/3 -- To view, visit http://gerrit.cloudera.org:8080/12886 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: native-toolchain Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I57e95617ccfc2d7d63cc277ab2b858f28ccad44c Gerrit-Change-Number: 12886 Gerrit-PatchSet: 3 Gerrit-Owner: Hector Acosta