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
[csolution] compile_commands.json
not referenced in generated *.cbuild.yml
#1210
Comments
Tools reading the cbuild and cbuild-idx files should not make a blind assumption about output files and directories to exist. Just assume a build fails and an axf, lib, bin or hex file is not created. We will not go back and update the cbuild-idx file depending on the success of a build. |
It seems we should clarify and align expectations.
Calling cbuild with target Generating compile commands is a pure build manager feature, the project manager currently doesn't know about it, so I wouldn't simply add it to project manager results, unless we decide to make the project manager aware of it. |
Thanks for clarifying. |
compile_commands.json
not referenced in generated *.cbuild.yml
@brondani @jkrech Sorry I had completely missed your answers here. From our previous discussions, I understood that to retrieve artefacts produced by a job (whether it is artefacts from builds or for intellisense), I should read the cbuild-idx file to get the cbuild.yml file and from there find out what has been produced. From your answers here, I imagine I had misunderstood the purpose of those two files as they seem to be primarily for internal use of the build process itself and not for describing the result of a job. Is there a way from the output directory to determine what constitutes an output artefact rather than an intermediary build artefact? |
@acabarbaye these files are not just for internal use. They contain details about the build process. IMHO these are the files that the IDE should use to understand the location of build artifacts and individual sources files of a component. |
in that case, should not the |
@acabarbaye I agree, we need to make this more useful for IDE usage. Not sure if |
they are the results of |
@acabarbaye I believe I understand your point, I would suggest to add it to
But for consistency its presence/absence and the |
When we add it this way, the I would recommend to build the IDE specific information as suggested here: #1265 (comment) - effectively with a command-line option. |
@ReinhardKeil - |
I don't believe this is required to reflect the With |
We ran
cbuild
with-t=database
to generate the compile_commands.json from a csolution project. This file is not referenced in the generated cbuild.yml file as we would expect.See the files below
hello.Debug+B-U585I-IOT02A.cbuild.yml.txt
hello.cbuild-idx.yml.txt
As can be seen in the file, the output information mentions an hex file and an axf file which actually have not been generated:
The text was updated successfully, but these errors were encountered: