Skip to content
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

intercept build error handling #580

Open
gyorb opened this issue Feb 16, 2017 · 7 comments
Open

intercept build error handling #580

gyorb opened this issue Feb 16, 2017 · 7 comments

Comments

@gyorb
Copy link
Contributor

gyorb commented Feb 16, 2017

improve intercept build error handling and detection

@gyorb gyorb added this to the release 5.8 milestone Feb 16, 2017
@dkrupp
Copy link
Member

dkrupp commented Feb 16, 2017

intercept-build created build json is empty on Ubuntu 14.04.4 LTS, clang 4.0-master

@dkrupp
Copy link
Member

dkrupp commented Feb 16, 2017

intercept-build logging does not work on Ubuntu 16 either

@whisperity
Copy link
Contributor

How are you invoking intercept-build?

@whisperity
Copy link
Contributor

My investigation reveals that this is an issue when intercept-build is invoked with a direct compiler invocation. It has been reported to @rizsotto. More on the thing here: rizsotto/scan-build#79.

@whisperity
Copy link
Contributor

whisperity commented Mar 13, 2017

Side question @gyorb, @dkrupp: Should we just always build ldlogger, no matter what? It could be that at make package intercpt-build is available, but it moves out of your path when you are using CodeChecker. In this case, the logger automatically tries to fall back to ldlogger but fails horribly.

If we were to build ld-logger even if intercept is availabe, we could make this transition a lot more easier. And building ld-logger doesn't increase CC's build time that much at all.

@whisperity
Copy link
Contributor

@gyorb What did you mean by "improve" in the original summary?
The issue itself with intercept-build has been resolved by #610.

@gyorb
Copy link
Contributor Author

gyorb commented May 8, 2017

Until we support out own ldlogger solution we can build it always into the package (use intercept-build at runtime if available).
By improvement I meant here that we should provide better error/debug messages to the users (if it is possible) when logging the compile commands fail for some reason (not just the message that the compilation command db is empty).

@gyorb gyorb modified the milestones: release 5.9, release 5.8 May 16, 2017
@gyorb gyorb modified the milestone: release 5.9 Jun 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants