You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your feature request related to a problem? Please describe.
I'm using make-style logic in my project. Specifically, there's only a need to re-run analysis / elaboration / simulation if the underlying VHDL files changed.
However, if I just run a blanket vsg --fix all-my-files.vhd, the file timestamps will be touched even if there is nothing to do.
Describe the solution you'd like
Either with a new flag, or just with a change to the default behavior, running vsg --fix ... should only update the file timestamps if something changed.
Describe alternatives you've considered
I've also considered a hack, which is to copy all the files to a temporary directory and then run vsg --fix on them and only copy them back if they've really changed. And this can work as a workaround for sure.
I only file this issue because I suspect others will run into similar situations.
Additional context
N / A
The text was updated successfully, but these errors were encountered:
I pushed an update for this issue to the issue-1407 branch. When you get a chance could you check it out on your end and let me know if it works for you?
Is your feature request related to a problem? Please describe.
I'm using make-style logic in my project. Specifically, there's only a need to re-run analysis / elaboration / simulation if the underlying VHDL files changed.
However, if I just run a blanket
vsg --fix all-my-files.vhd
, the file timestamps will be touched even if there is nothing to do.Describe the solution you'd like
Either with a new flag, or just with a change to the default behavior, running
vsg --fix ...
should only update the file timestamps if something changed.Describe alternatives you've considered
I've also considered a hack, which is to copy all the files to a temporary directory and then run
vsg --fix
on them and only copy them back if they've really changed. And this can work as a workaround for sure.I only file this issue because I suspect others will run into similar situations.
Additional context
N / A
The text was updated successfully, but these errors were encountered: