-
Notifications
You must be signed in to change notification settings - Fork 1.4k
[cxxmodules] Teach cling to build modules; Build dependent modules explicitly. #5521
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
base: master
Are you sure you want to change the base?
[cxxmodules] Teach cling to build modules; Build dependent modules explicitly. #5521
Conversation
|
Starting build on |
|
Build failed on ROOT-fedora30/cxx14. Errors:
|
|
Build failed on mac1015/cxx17. Errors:
|
|
Build failed on ROOT-fedora29/python3. Errors:
|
022148b to
a879994
Compare
|
Starting build on |
|
Build failed on mac1015/cxx17. Errors:
|
a879994 to
bec0e46
Compare
|
Starting build on |
|
Build failed on mac1015/cxx17. Errors:
|
|
Build failed on ROOT-fedora30/cxx14. Errors:
|
|
Build failed on ROOT-fedora29/python3. Errors:
|
bec0e46 to
fdca56d
Compare
|
Starting build on |
|
Build failed on ROOT-fedora30/cxx14. Errors:
|
|
Build failed on mac1015/cxx17. Errors:
|
|
@phsft-bot build! |
|
Starting build on |
|
Build failed on mac1015/cxx17. Errors:
|
|
Build failed on ROOT-fedora30/cxx14. Errors:
|
|
Build failed on windows10/cxx14. Errors:
|
|
Build failed on ROOT-fedora29/python3. Failing tests: |
|
@phsft-bot build! |
|
Build failed on ROOT-fedora29/python3. Errors:
|
|
Build failed on mac1015/cxx17. Errors:
|
|
Starting build on |
|
Build failed on mac1015/cxx17. Errors:
|
|
Build failed on ROOT-fedora29/python3. Failing tests:
And 942 more |
84c61c7 to
af6bfc7
Compare
|
Starting build on |
|
Build failed on mac1015/cxx17. Errors:
|
|
Build failed on ROOT-fedora30/cxx14. Failing tests:
And 956 more |
af6bfc7 to
5eed354
Compare
|
Starting build on |
|
Build failed on ROOT-fedora30/cxx14. Failing tests:
And 956 more |
|
Build failed on mac1015/cxx17. Errors:
And 117 more |
|
Build failed on ROOT-fedora29/python3. Failing tests: |
|
Starting build on |
|
Build failed on ROOT-fedora30/cxx14. Failing tests:
And 957 more |
…plicitly.
This allows external users to call rootcling as:
rootcling bare-cling -xc++ -I${ROOTSYS}/etc -I ${headers_location}
-fmodules -Xclang -emit-module -fmodule-name=tinyxml2
-fmodules-cache-path=${CMSSW_LIBDIR} -o ${module_file_location}
${headers_location}/module.modulemap
This will allow us to generalize it to resolve C headers as well.
New versions of OSX has libc in the sdk path and not at the usual /usr/include. Detect the location in a similar way to detection of the C++ headers. This solves the issue with the failure to build the Darwin.pcm file and the diagnostic that there is no module.modulemap for osx 10.15 in /usr/include/
25c9029 to
e627c0b
Compare
|
Starting build on |
|
Build failed on mac1015/cxx17. |
This would allow CMSSW to build its third party modules by calling:
This commit has several issues one is that it does not order the dependencies so if libc.pcm goes out of date the build system does not regenerate std.pcm.
cc: @oshadura, @davidlange6, @smuzaffar