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

Use GHDL instead of FreeHDL for digital simulation [Linux Only] #503

Merged
merged 2 commits into from
Jan 29, 2024

Conversation

iwbnwif
Copy link
Contributor

@iwbnwif iwbnwif commented Jan 26, 2024

The approach taken follows the advice given at Qucs/qucs#830.
Includes syntax fixes to accommodate the slight differences between GHDL and FreeHDL.
It is not the full solution to #395 and does not address #396, but it allows basic digital simulation as previously.

The approach taken follows the advice given at
Qucs/qucs#830. Includes syntax fixes to accomodate the slight
differences between GHDL and FreeHDL.
@ra3xdh ra3xdh added this to the 24.1.0 milestone Jan 28, 2024
@ra3xdh ra3xdh mentioned this pull request Jan 28, 2024
3 tasks
@ra3xdh ra3xdh merged commit d1a438c into ra3xdh:current Jan 29, 2024
2 checks passed
@ra3xdh
Copy link
Owner

ra3xdh commented Jan 29, 2024

Thanks for the contribution! Merged. The remaining thing is to update the Windows BAT files until #396 will be implemented.

@iwbnwif
Copy link
Contributor Author

iwbnwif commented Jan 29, 2024

I need to set up a build environment for Windows, but updating the .bat script should be trivial thereafter. I think there are also some further enhancements possible regarding the way in which subcircuits are passed to the simulator.

@ra3xdh
Copy link
Owner

ra3xdh commented Jan 29, 2024

but updating the .bat script should be trivial thereafter

I will try to update the BAT myself in the next week.

@iwbnwif iwbnwif deleted the ghdl_digital_scripts branch January 30, 2024 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants