search

graphman65 / linter-vhdl

Atom vhdl linter
https://atom.io/packages/linter-vhdl
MIT License
6 stars 2 forks source link

VHDL2008 selection #10

Closed qarlosalberto closed 5 years ago

qarlosalberto commented 6 years ago

Hello,

It would be nice to be able to select as a global option to use VHDL2008 or pass arguments to GHDL.

graphman65 commented 6 years ago

Hey, You can already do this. https://github.com/graphman65/linter-vhdl#custom-ghdl-arguments Let me know if you cannot get it working.

qarlosalberto commented 6 years ago

Hey, it would be good add custom arguments in linter-vhdl options. With comment on the top I need modify all my files in the project (42) :(

douglasrossi commented 6 years ago

Hi, I totally agree with @qarlosalberto . It would be great to use VHDL-2018 without changing every file in the project.

graphman65 commented 6 years ago

Hi, I'll add this feature asap ! Would you rather want an "vhdl arguments" settings in atom or a .vhdlconfig in the project root directory ? Thanks @qarlosalberto @douglasrossi

douglasrossi commented 6 years ago

Hi @graphman65 , thanks for the quickly reply! I think that in my case the "vhdl arguments" can do the job.

qarlosalberto commented 6 years ago

Yes! "vhdl arguments"

Thank you :)

lcapossio commented 5 years ago

I also vote for VHDL arguments in atom! Thanks for the package it's very convenient.

lcapossio commented 5 years ago

Hi, I've created a pull request for passing arguments to the compiler. Please check it out and leave a comment if necessary. https://github.com/graphman65/linter-vhdl/pull/12

alexg-k commented 5 years ago

+1 I work on a multi-user project and don't want to hassle with user-specific code in all of my project files.

graphman65 commented 5 years ago

Hi, sorry for being so late, I just merged it ! Thanks for your help and your support.