-
Notifications
You must be signed in to change notification settings - Fork 1.4k
[cxxmodules] CIFactory now uses MultiplexConsumer #988
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
[cxxmodules] CIFactory now uses MultiplexConsumer #988
Conversation
|
Starting build on |
|
I moved this into an extra PR because it somehow the CIFactory patch from #930 fails a certain test. If this also fails this test, then I at least know it's related to this ASTConsumer (as everything else should be behind a |
|
@phsft-bot build |
|
Starting build on |
This is a preparation for C++ modules as they will require that we attach multiple consumers to the CI.
b21f490 to
90ed8a9
Compare
|
Starting build on |
|
Build failed on slc6/gcc62. Failing tests: |
|
Build failed on slc6/gcc49. Failing tests: |
|
Build failed on ubuntu14/native. Failing tests: |
|
Build failed on centos7/gcc49. Failing tests: |
|
@vgvassilev @Axel-Naumann any clue why this is failing this test? The only outside visible change should be that you now no longer can |
|
Build failed on mac1012/native. Failing tests: |
|
No clues from me. But I hear gdb calling... |
|
Closing this, it's already part of the #930 PR |
This is a preparation for C++ modules as they will require that
we attach multiple consumers to the CI.