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

Problematic issues with build directories #1

Open
stsquad opened this issue Jun 17, 2015 · 3 comments
Open

Problematic issues with build directories #1

stsquad opened this issue Jun 17, 2015 · 3 comments

Comments

@stsquad
Copy link
Contributor

stsquad commented Jun 17, 2015

Due to the way the builder works around calling clang-check directly it will get confused by not being in the right build directory (e.g. in QEMU it fails to find config-target.h which is in the target build directory). I'm working on a fix to pass the build directory as an option.

@stsquad
Copy link
Contributor Author

stsquad commented Jun 17, 2015

After a bunch of experimentation this seems tricky to solve. In my version of the checker I had simply used source-original and relied on a :predicate to ensure we had saved the buffer. Then clang-check could be called normally and resolve everything. Because we are trying to maintain the on-the-fly checking we need to somehow ensure clang-check is called in the build-directory referenced in compile_commands.json. Maybe this means messing with default-directory while building up the command unless there is someway to define a prequel set of commands before running the checker.

@Sarcasm
Copy link

Sarcasm commented Jul 20, 2015

FYI, in irony-mode I use the -working-directory DIR compile option for the compile_commands.json's directory. I don't remember this ever being an issue.

@stsquad
Copy link
Contributor Author

stsquad commented Feb 15, 2017

I have a feeling newer clang-checks are much better at picking up compile_commands.json. I'm trying to find out when this changed because I could remove a lot of the parsing junk if that is the case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants