graphman65 / linter-vhdl

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

no error messages on Linux and MAC #7

Closed Gatherer closed 6 years ago

Gatherer commented 6 years ago

linter-vhdl do not show any warnings / errors. tested on Linux and MAC.

access to ghdl is tested by using the "compile on lint" option. If a there are no errors the cf file is created. Command Line access to ghdl works and all errors are shown correct.

atom 1.25.1 linter-vhdl 1.1.4 linter 2.2.0 linter-ui-default 1.7.1

sww1235 commented 6 years ago

I can also confirm this. Tried with both the compile on lint checkbox checked and not. Runnning ghdl directly in a terminal, I can the expected errors.

graphman65 commented 6 years ago

Hey, thanks for the report. It seems related to a change in the format of the errors output. It's actually working with ghdl 0.35-dev on my machine. I'll fix this so that it can work with newer versions.

graphman65 commented 6 years ago

Hummm, might be my bad but I can't reproduce this bug. I tried with 0.36 as well and it's working. Can you provide me some code example that should display errors but doesn't ?

sww1235 commented 6 years ago

I don't know what version of ghdl @Gatherer is running, but I am on 0.34 from homebrew cask. looks like homebrew needs to update as 0.35 is over a year old.

What versions were you building it for before you tested 0.35 and 0.36?

graphman65 commented 6 years ago

I built it for 0.35 at first but I just tried with 0.34 and it's working on my Ubuntu. Might be a MAC only issue. I can't fix it for now as I'm not able to reproduce it :/

Gatherer commented 6 years ago

On MAC I use the ghdl homebrew 0.34 version on Linux I use ghdl 0.36. Is there a way I can provide more informations?

It's wired all other linter, verilog, c/c++ and md do work on both systems, MAC and Linux.

graphman65 commented 6 years ago

Can you send me a code example ?

Gatherer commented 6 years ago

The example you have in your example directory is fine. Example: "Compile on Lint is set"

I can repeat this on MAC and Linux Mint 18.3. Just updated Atom to 1.26.0, no change, all plugins are up to date.

Gatherer commented 6 years ago

Not sure why but after updating to Atom 1.27.0 everything works fine on Linux Mint 18.3 and MAC. No other changes

Atom 1.27.0 linter 2.2.0 linter-ui-default 1.7.1 linter-vhdl 1.1.4

ghld 0.34 (MAC, on Linux not sure but works too)

graphman65 commented 6 years ago

Thanks for the feedback, I was a too busy to find out what was going on but it definitely seems that it was an atom/linter issue.

Hope it will still work as expected with the next atom updates !