-
Notifications
You must be signed in to change notification settings - Fork 102
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Build failure with ROOT 6.22 new python structure #625
Comments
Interesting, we are testing in our CI against dev3 and dev4 and we did not observe this. How can we test this particular ROOT configuration of yours? |
But just for clarification dev3 has ROOT built with both python2 and 3 or is there dev3(py2) and dev3python3(py3)? |
@lastCent I couldn't find the cmake command and output in the logs or the cdash website |
So this is related to this
and the relevant commit root-project/root@71c42ac |
@lastCent but this affects only root master and not the 6-20 branch, so this change is for ROOT 6.22? |
Hello,
You can source the complete view of a particular nightly from any CERN CVMFS-enabled machine. To do so run something like:
We aim to build ROOT with python2 in the dev3 and dev4 builds, and python3 in dev3python3. We have used to specify this with the -Dpython3 flag, which is set to ON for dev3python3 (not for dev3/dev4). But I understand that ROOT changes the python versioning system with the 6.20 release, such that more than one version can be specified. Could this be the issue perhaps?
Apologies for the bad UI. The logs can be accessed by clicking the yellow package icons next to the name of a failing build (which is a build where the error column contains errors). So for example, in the section dev3 press one of the package icons, then DD4hep-master-install.log. I'm unsure how widespread access rights are. If some of you don't have those, let me know and I'll fetch some of those logs for you. The build uses our LCGCMake build system, which is built on top of CMake. This makes the exact command a it difficult to read, but you can see the flags+environment used here https://gitlab.cern.ch/sft/lcgcmake/blob/master/projects/CMakeLists.txt#L73 . Let me know which parts are unclear, or if nothing about it is clear at all.
We build every night, so the exact version of ROOT may change. For yesterday's (Wednesday) build
|
The error I get on wednesday dev3 for configure
This does not seem like a DD4hep error? |
Oh sorry, I should have given you the CMake command we usually use for local testing: Additionally you can take prebuilt packages from the stack by setting Edit: The views may not be a proper build environment for views, especially if they don't finish. Please use this script to set up a local environment for building instead: |
Assuming that the build of ROOT in Wed dev3 is I do this
and then I create a nearly empty cmake_minimum_required(VERSION 3.12 FATAL_ERROR)
PROJECT( DD4hep LANGUAGES CXX)
find_package (ROOT) and when invoking cmake I get this
|
Hi all, sorry for the confusion. The issue with You can, for now, use the following script to set up environment: https://stikked.web.cern.ch/stikked/view/5a6cc1fa (save it as I have done some debugging on my own, and would like to share some results. First, here is the error message we observe (full log ):
Then, I compared CMake configuration logs (/cvmfs/sft-nightlies.cern.ch/lcg/nightlies/dev3/Mon/DD4hep/master/x86_64-centos7-gcc8-opt/logs/DD4hep-master-configure.log ) from last successful build with the failing build (copy ): Was
Now:
DD4Hep detects PyRoot by looking for Another question is why does DD4Hep attempt to install dictionaries that are not built? Thanks, |
For clarification, we detect PyROOT by looking for the target There is a bug in the DDG4 that would ask for dictionary creation, but not actually call the command unless a library is build that uses the dictionary.cxx file, thus also the pcm file was not created. (Should be fixed with #631 ) The relevant part of the log is actually this part
There is no message if pyroot is not found. |
@iarspider @lastCent can you confirm that the nightly builds worked over the weekend? |
@andresailer yes, the nightly builds (of DD4Hep master) worked. |
@iarspider
Or will you let us know once that is fixed? |
@andresailer the views were not recreated due to build failures. You can use the other solution (dedicated setup script) for now. |
I have added a JIRA issue, but relaxed the importance, since it is the ROOT HEAD |
DD4hep fails to build in our nightly builds, probably because of the way in which multiple Python versions now are organized in the lib/ directory, and the splitting of PyROOT from TPython. ROOT 6.20 changelog: https://root.cern/doc/v620/release-notes.html
OS version: e.g. SLC6
Compiler version: e.g GCC 6.2
DD4hep version: tag or commit ID, or GitHub branch
Reproduced by: exact steps to reproduce the problem: checkout, setup environment (Geant, ROOT versions, iLCSoft release), cmake, build, run...
Input: link to input files if applicable
Output: full build and run log output
Goal: A short description of what you are trying to achieve
The text was updated successfully, but these errors were encountered: