Re: [clang-tools-extra] r283534 - Fix buildbot error.
On 7 October 2016 at 13:06, Renato Golin wrote: > On 7 October 2016 at 10:23, Haojian Wu via cfe-commits > wrote: >> Author: hokein >> Date: Fri Oct 7 04:23:28 2016 >> New Revision: 283534 >> >> URL: http://llvm.org/viewvc/llvm-project?rev=283534&view=rev >> Log: >> Fix buildbot error. >> >> The error maybe caused by the mixed environment of the two lint tests. >> Cleanup the environment before running each test. > > Hi, > > Can you revert all changes and test it again on your machine before > pushing it upstream? > > Your changes and reverts are really upsetting all our buildbots and > making them completely unstable. Right, your tests were breaking all over the place on ranges that had nothing to do with your commits, so I reverted on r283553. Please let me know if you need help with tests before you commit it again. cheers, --renato ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Re: [clang-tools-extra] r283534 - Fix buildbot error.
On 7 October 2016 at 10:23, Haojian Wu via cfe-commits wrote: > Author: hokein > Date: Fri Oct 7 04:23:28 2016 > New Revision: 283534 > > URL: http://llvm.org/viewvc/llvm-project?rev=283534&view=rev > Log: > Fix buildbot error. > > The error maybe caused by the mixed environment of the two lint tests. > Cleanup the environment before running each test. Hi, Can you revert all changes and test it again on your machine before pushing it upstream? Your changes and reverts are really upsetting all our buildbots and making them completely unstable. cheers, --renato ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[clang-tools-extra] r283534 - Fix buildbot error.
Author: hokein Date: Fri Oct 7 04:23:28 2016 New Revision: 283534 URL: http://llvm.org/viewvc/llvm-project?rev=283534&view=rev Log: Fix buildbot error. The error maybe caused by the mixed environment of the two lint tests. Cleanup the environment before running each test. Modified: clang-tools-extra/trunk/test/clang-move/move-class.cpp clang-tools-extra/trunk/test/clang-move/move-multiple-classes.cpp Modified: clang-tools-extra/trunk/test/clang-move/move-class.cpp URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/clang-move/move-class.cpp?rev=283534&r1=283533&r2=283534&view=diff == --- clang-tools-extra/trunk/test/clang-move/move-class.cpp (original) +++ clang-tools-extra/trunk/test/clang-move/move-class.cpp Fri Oct 7 04:23:28 2016 @@ -1,3 +1,4 @@ +// RUN: rm -rf %T/clang-move // RUN: mkdir -p %T/clang-move/build // RUN: sed 's|$test_dir|%/T/clang-move|g' %S/Inputs/database_template.json > %T/clang-move/compile_commands.json // RUN: cp %S/Inputs/test* %T/clang-move/ Modified: clang-tools-extra/trunk/test/clang-move/move-multiple-classes.cpp URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/clang-move/move-multiple-classes.cpp?rev=283534&r1=283533&r2=283534&view=diff == --- clang-tools-extra/trunk/test/clang-move/move-multiple-classes.cpp (original) +++ clang-tools-extra/trunk/test/clang-move/move-multiple-classes.cpp Fri Oct 7 04:23:28 2016 @@ -1,3 +1,4 @@ +// RUN: rm -rf %T/clang-move // RUN: mkdir -p %T/clang-move/build // RUN: sed 's|$test_dir|%/T/clang-move|g' %S/Inputs/database_template.json > %T/clang-move/compile_commands.json // RUN: cp %S/Inputs/multiple_class_test* %T/clang-move/ ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
