Make sure roottest tests ROOT being build, not ROOT in env by Axel-Naumann · Pull Request #7937 · root-project/root

Axel-Naumann · 2021-04-20T14:00:41Z

phsft-bot · 2021-04-20T14:00:49Z

Starting build on ROOT-debian10-i386/cxx14, ROOT-performance-centos8-multicore/default, ROOT-fedora30/cxx14, ROOT-fedora31/noimt, ROOT-ubuntu16/nortcxxmod, mac1014/python3, mac11.0/cxx17, windows10/cxx14
How to customize builds

phsft-bot · 2021-04-20T14:21:17Z

Build failed on ROOT-fedora31/noimt.
Running on root-fedora-31-2.cern.ch:/home/sftnight/build/workspace/root-pullrequests-build
See console output.

Failing tests:

projectroot.roottest-build.cling.parsing.roottest_cling_parsing_semicolon

phsft-bot · 2021-04-20T14:30:17Z

Build failed on mac1014/python3.
Running on macphsft17.dyndns.cern.ch:/build/jenkins/workspace/root-pullrequests-build
See console output.

Failing tests:

projectroot.roottest-build.cling.parsing.roottest_cling_parsing_semicolon

oshadura · 2021-04-20T14:33:08Z

CMakeLists.txt

    if(IS_DIRECTORY ${roottestdir})
      file(REMOVE_RECURSE ${CMAKE_CURRENT_BINARY_DIR}/roottest)
-      add_subdirectory(${roottestdir} roottest)
+      add_subdirectory(${roottestdir} roottest-build)


Why we have here roottest-build?

It's the build directory that's passed as the second argument. And I was always confused why I checked out roottest into my build dir ;-) I prefer to have the directory name clarify it's the roottest build dir. I'm okay to drop this if you find this wrong!

nono, I was just curious...Let's use it!

phsft-bot · 2021-04-20T14:35:43Z

Build failed on ROOT-performance-centos8-multicore/default.
Running on olbdw-01.cern.ch:/data/sftnight/workspace/root-pullrequests-build
See console output.

Failing tests:

projectroot.roottest-build.cling.parsing.roottest_cling_parsing_semicolon

phsft-bot · 2021-04-20T14:37:35Z

Build failed on ROOT-fedora30/cxx14.
Running on root-fedora30-2.cern.ch:/home/sftnight/build/workspace/root-pullrequests-build
See console output.

Failing tests:

projectroot.roottest-build.cling.parsing.roottest_cling_parsing_semicolon

phsft-bot · 2021-04-20T14:37:43Z

Build failed on windows10/cxx14.
Running on null:C:\build\workspace\root-pullrequests-build
See console output.

Errors:

[2021-04-20T14:33:47.528Z] C:\build\workspace\root-pullrequests-build\root\interpreter\llvm\src\lib\CodeGen\PHIElimination.cpp(665,1): fatal error C1060: compiler is out of heap space [C:\build\workspace\root-pullrequests-build\build\interpreter\llvm\src\lib\CodeGen\LLVMCodeGen.vcxproj]
[2021-04-20T14:33:47.528Z] C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.23.28105\include\memory(1820,1): fatal error C1060: compiler is out of heap space (compiling source file C:\build\workspace\root-pullrequests-build\root\interpreter\llvm\src\lib\CodeGen\PHIEliminationUtils.cpp) [C:\build\workspace\root-pullrequests-build\build\interpreter\llvm\src\lib\CodeGen\LLVMCodeGen.vcxproj]
[2021-04-20T14:33:47.529Z] C:\build\workspace\root-pullrequests-build\root\interpreter\llvm\src\include\llvm/Support/TrailingObjects.h(233,69): fatal error C1060: compiler is out of heap space (compiling source file C:\build\workspace\root-pullrequests-build\root\interpreter\llvm\src\lib\CodeGen\PostRAHazardRecognizer.cpp) [C:\build\workspace\root-pullrequests-build\build\interpreter\llvm\src\lib\CodeGen\LLVMCodeGen.vcxproj]
[2021-04-20T14:33:47.529Z] C:\build\workspace\root-pullrequests-build\root\interpreter\llvm\src\lib\CodeGen\PatchableFunction.cpp(89,1): fatal error C1060: compiler is out of heap space [C:\build\workspace\root-pullrequests-build\build\interpreter\llvm\src\lib\CodeGen\LLVMCodeGen.vcxproj]
[2021-04-20T14:33:47.529Z] C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.23.28105\include\tuple(332,1): fatal error C1060: compiler is out of heap space (compiling source file C:\build\workspace\root-pullrequests-build\root\interpreter\llvm\src\lib\CodeGen\SelectionDAG\StatepointLowering.cpp) [C:\build\workspace\root-pullrequests-build\build\interpreter\llvm\src\lib\CodeGen\SelectionDAG\LLVMSelectionDAG.vcxproj]
[2021-04-20T14:33:47.529Z] C:\build\workspace\root-pullrequests-build\root\interpreter\llvm\src\include\llvm/ADT/SmallVector.h(581,1): fatal error C1060: compiler is out of heap space (compiling source file C:\build\workspace\root-pullrequests-build\root\interpreter\llvm\src\tools\clang\lib\Frontend\FrontendAction.cpp) [C:\build\workspace\root-pullrequests-build\build\interpreter\llvm\src\tools\clang\lib\Frontend\obj.clangFrontend.vcxproj]
[2021-04-20T14:33:47.529Z] C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.23.28105\include\tuple(564,1): fatal error C1060: compiler is out of heap space (compiling source file C:\build\workspace\root-pullrequests-build\root\interpreter\llvm\src\lib\Transforms\Scalar\SimpleLoopUnswitch.cpp) [C:\build\workspace\root-pullrequests-build\build\interpreter\llvm\src\lib\Transforms\Scalar\LLVMScalarOpts.vcxproj]
[2021-04-20T14:33:47.529Z] C:\build\workspace\root-pullrequests-build\root\interpreter\llvm\src\include\llvm/ADT/DenseMap.h(1237,1): fatal error C1060: compiler is out of heap space (compiling source file C:\build\workspace\root-pullrequests-build\root\interpreter\llvm\src\lib\Transforms\Scalar\SeparateConstOffsetFromGEP.cpp) [C:\build\workspace\root-pullrequests-build\build\interpreter\llvm\src\lib\Transforms\Scalar\LLVMScalarOpts.vcxproj]
[2021-04-20T14:33:47.529Z] C:\build\workspace\root-pullrequests-build\root\interpreter\llvm\src\include\llvm/Analysis/ScalarEvolutionExpressions.h(576,29): fatal error C1060: compiler is out of heap space (compiling source file C:\build\workspace\root-pullrequests-build\root\interpreter\llvm\src\lib\Transforms\Scalar\Scalarizer.cpp) [C:\build\workspace\root-pullrequests-build\build\interpreter\llvm\src\lib\Transforms\Scalar\LLVMScalarOpts.vcxproj]
[2021-04-20T14:33:47.529Z] C:\build\workspace\root-pullrequests-build\root\interpreter\llvm\src\include\llvm/IR/Instruction.def(158,1): fatal error C1060: compiler is out of heap space (compiling source file C:\build\workspace\root-pullrequests-build\root\interpreter\llvm\src\lib\Transforms\Scalar\SROA.cpp) [C:\build\workspace\root-pullrequests-build\build\interpreter\llvm\src\lib\Transforms\Scalar\LLVMScalarOpts.vcxproj]

And 8 more

phsft-bot · 2021-04-20T14:40:34Z

Build failed on ROOT-ubuntu16/nortcxxmod.
Running on sft-ubuntu-1604-2.cern.ch:/build/workspace/root-pullrequests-build
See console output.

Failing tests:

projectroot.roottest-build.cling.parsing.roottest_cling_parsing_semicolon

phsft-bot · 2021-04-20T14:50:45Z

Build failed on ROOT-debian10-i386/cxx14.
Running on pcepsft11.dyndns.cern.ch:/home/sftnight/build/workspace/root-pullrequests-build
See console output.

Failing tests:

projectroot.roottest-build.cling.parsing.roottest_cling_parsing_semicolon

phsft-bot · 2021-04-20T15:24:56Z

Build failed on mac11.0/cxx17.
Running on macphsft23.dyndns.cern.ch:/Users/sftnight/build/workspace/root-pullrequests-build
See console output.

Failing tests:

projectroot.roottest-build.cling.parsing.roottest_cling_parsing_semicolon

oshadura · 2021-04-21T10:54:29Z

@Axel-Naumann projectroot.roottest-build.cling.parsing.roottest_cling_parsing_semicolon failure looks 'real'?

Axel-Naumann · 2021-04-21T11:57:07Z

failure looks 'real'?

Totally - debugging...

This fixes issue-7081, as now, with the correct spelling of the env var, the build-ROOTSYS is picked up by roottest.

phsft-bot · 2021-04-21T13:41:05Z

Starting build on ROOT-debian10-i386/cxx14, ROOT-performance-centos8-multicore/default, ROOT-fedora30/cxx14, ROOT-fedora31/noimt, ROOT-ubuntu16/nortcxxmod, mac1014/python3, mac11.0/cxx17, windows10/cxx14
How to customize builds

Axel-Naumann · 2021-04-21T13:51:50Z

CTestCustom.cmake suppresses roottest.cling.parsing not roottest-build.cling.parsing. This looks fragile (also wrt Jenkins) so I dropped the rename of the build dir. Too bad.

Axel-Naumann requested a review from oshadura April 20, 2021 14:00

Axel-Naumann self-assigned this Apr 20, 2021

oshadura reviewed Apr 20, 2021

View reviewed changes

[cmake] Fix export of Env{ROOTSYS}:

7f41301

This fixes issue-7081, as now, with the correct spelling of the env var, the build-ROOTSYS is picked up by roottest.

Axel-Naumann force-pushed the issue-7081 branch from d209bb6 to 7f41301 Compare April 21, 2021 13:40

oshadura self-requested a review April 21, 2021 13:48

oshadura approved these changes Apr 21, 2021

View reviewed changes

Axel-Naumann merged commit f5cb2c1 into root-project:master Apr 21, 2021

Axel-Naumann deleted the issue-7081 branch April 21, 2021 18:00

Conversation

Axel-Naumann commented Apr 20, 2021

Uh oh!

phsft-bot commented Apr 20, 2021

Uh oh!

phsft-bot commented Apr 20, 2021

Failing tests:

Uh oh!

phsft-bot commented Apr 20, 2021

Failing tests:

Uh oh!

oshadura Apr 20, 2021

Choose a reason for hiding this comment

Uh oh!

Axel-Naumann Apr 20, 2021

Choose a reason for hiding this comment

Uh oh!

oshadura Apr 20, 2021

Choose a reason for hiding this comment

Uh oh!

phsft-bot commented Apr 20, 2021

Failing tests:

Uh oh!

phsft-bot commented Apr 20, 2021

Failing tests:

Uh oh!

phsft-bot commented Apr 20, 2021

Errors:

Uh oh!

phsft-bot commented Apr 20, 2021

Failing tests:

Uh oh!

phsft-bot commented Apr 20, 2021

Failing tests:

Uh oh!

phsft-bot commented Apr 20, 2021

Failing tests:

Uh oh!

oshadura commented Apr 21, 2021

Uh oh!

Axel-Naumann commented Apr 21, 2021

Uh oh!

phsft-bot commented Apr 21, 2021

Uh oh!

Axel-Naumann commented Apr 21, 2021

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants