Closed NJDFan closed 3 years ago
On MSYS2, I was running GHDL as:
winpty rlwrap tclsh
source
Does this update remove the need to prefix rlwrap with winpty? One less step is always a good thing.
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 https://github.com/OSVVM/OSVVM-Scripts/pull/9#issuecomment-824267843, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACBCACGSSU6KCIPH4XGWHW3TJ4KHZANCNFSM43KSZL6Q .
Pulled and will be in next release
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.