GHDL tweaks for Linux #9
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 2 commits
April 21, 2021 07:31
…ories are now forced to be lowercase, and the tee statements do not try to write to /dev/pty0 if that is not a writable device. Also changed the ToolNameVersion to reflect the actual GHDL version; it had been hardcoded in the script.
|
On MSYS2, I was running GHDL as: Does this update remove the need to prefix rlwrap with winpty? |
|
It's probably been a decade since I've touched MSYS, but I doubt it. All
this update does is remove /dev/pty0 from the tee list if it's not on the
system and make sure that the path for library names winds up in lowercase,
which GHDL gets fussy about.
…On Wed, Apr 21, 2021 at 11:27 AM JimLewis ***@***.***> wrote:
On MSYS2, I was running GHDL as:
winpty rlwrap tclsh
source /OsvvmLibraries/Scripts/StartUp.tcl
Does this update remove the need to prefix rlwrap with winpty?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#9 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACBCACGSSU6KCIPH4XGWHW3TJ4KHZANCNFSM43KSZL6Q>
.
|
|
Pulled and will be in next release |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Updated GHDL vendor script to fix some Linux problems. Library directories are now forced to be lowercase (GHDL wasn't finding mixed-case directories on a case-sensitive filesystem), and the tee statements do not try to write to /dev/pty0 if that is not a writable device.
Also changed the ToolNameVersion to reflect the actual GHDL version; it had been hardcoded in the script.
As a note, I was not able to build RunAllTests.pro, either before or after these changes. This does at least allow me to successfully build OsvvmLibraries.pro, but even on the latest GHDL from the upstream master (2.0.0) I'm getting a SYSTEM.ASSERTIONS.ASSERT_FAILURE : vhdl-nodes.adb:861 from GHDL.