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

verible-verilog-lint does not work with +incdir+ #2322

Open
sifferman opened this issue Jan 8, 2025 · 1 comment
Open

verible-verilog-lint does not work with +incdir+ #2322

sifferman opened this issue Jan 8, 2025 · 1 comment
Labels
rejects-valid syntax If the parser wrongly rejects syntactically valid code (according to SV-2017).

Comments

@sifferman
Copy link

Hello!

Ubuntu 22.04.4 LTS

verible-verilog-lint --version
v0.0-3899-g75c38daf
Commit  2025-01-06
Built   2025-01-07T04:40:32Z

It seems that verible-verilog-lint does not work with +incdir+.

Here is an example with basejump_stl: https://github.com/bespoke-silicon-group/basejump_stl.

verible-verilog-lint +incdir+bsg_misc bsg_misc/bsg_counter_up_down.sv
E0107 20:33:30.080594  226105 verilog-linter.cc:109] Can't read '+incdir+bsg_misc': +incdir+bsg_misc: No such file or directory
bsg_misc/bsg_counter_up_down.sv:26:41-54: preprocessing error at token "`BSG_INV_PARAM" : Error expanding macro identifier, might not be defined before.

Am I doing doing something wrong? Or does this functionality not exist for verible-verilog-lint yet?

Thanks!

@sifferman sifferman added the rejects-valid syntax If the parser wrongly rejects syntactically valid code (according to SV-2017). label Jan 8, 2025
@hzeller
Copy link
Collaborator

hzeller commented Jan 8, 2025

functionality does not exist yet unfortunately. It is fairly high on my to-do list ... when time permits :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rejects-valid syntax If the parser wrongly rejects syntactically valid code (according to SV-2017).
Projects
None yet
Development

No branches or pull requests

2 participants