Match filename in xvlog linter and add tip #4764
Open
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Match the xvlog stdout for the filename to properly filter out the errors per file.
Suggest using Vivado generated compile.sh file to obtain the xvlog options.
For reference, the sane way of using xvlog is along these lines:
Room for improvement:
The
.prj
uses relative paths and expects xvlog to be called from its path.As is, I'm manually "fixing" the relative paths in the
.prj
to point from the cwd to xsim.Not sure if it's worth a more elaborated solution (e.g. search for the
compile.sh
, set cwd to its location, and extract the xvlog options.