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

Update qucs_run_hdl.bat to use GHDL on Windows #543

Merged
merged 3 commits into from
Feb 10, 2024

Conversation

iwbnwif
Copy link
Contributor

@iwbnwif iwbnwif commented Feb 9, 2024

*** UNTESTED! ***

This should complete #395 but I am not able to build and test Qucs-S on Windows at the moment. Please can someone test this before proceeding any further.

@ra3xdh ra3xdh added this to the 24.1.0 milestone Feb 10, 2024
@ra3xdh ra3xdh linked an issue Feb 10, 2024 that may be closed by this pull request
3 tasks
@ra3xdh
Copy link
Owner

ra3xdh commented Feb 10, 2024

The BAT doesn't work for me. The error message is shown on the screenshot. I suspect the problem with %TIME% variable. Maybe my GHDL windows package is wrong, because the same circuit works using Linux package. You may use this demo package to test the GHDL on Windows https://drive.google.com/file/d/1-anZcfQK9heIn0d9nmqSGqHlPXDfOjUh/view?usp=sharing Everything should work using 64-bit wine.
image

@ra3xdh ra3xdh added the digital label Feb 10, 2024
@ra3xdh
Copy link
Owner

ra3xdh commented Feb 10, 2024

The simulation operates after setting time=12n without unit in the simulation properties. But GHDL shows an another error coming from the VHDL file itself. It's strange that the same file works on Linux without problem. Maybe GHDL version for Windows requires an extra configuration.

image

@iwbnwif
Copy link
Contributor Author

iwbnwif commented Feb 10, 2024

Please can I check if the Windows build you sent has the fix to qucs/dialogs/simmessage.cpp in faaaff0? There should not be a space between '40' and 'ns'.

I have a workaround, but it would be better to test it against the final code.

But there are also some problems with the batch file. It seems that %DIR% and %TIME% are reserved - at least on my Wine installation. If I change them to %SIMDIR% and %SIMTIME% it seems to work.

@ra3xdh ra3xdh self-requested a review February 10, 2024 13:33
@ra3xdh ra3xdh merged commit ae8250c into ra3xdh:current Feb 10, 2024
2 checks passed
@ra3xdh
Copy link
Owner

ra3xdh commented Feb 10, 2024

Everything works as expected now. Merging.
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Replace FreeHDL by GHDL
2 participants