search

OSVVM / OSVVM-Scripts

OSVVM project simulation scripts. Scripts are tedious. These scripts simplify the steps to compile your project for simulation
Other
10 stars 14 forks source link

Build command doesn't report an error if *.pro file is not found #3

Closed Paebbels closed 3 years ago

Paebbels commented 4 years ago

Problem

When calling the build command provided by OSVVM-Scripts, no error is reported when the provided *.pro doesn't exist.

Current Behavior

Some TCL actions are performed, but the script immediately returns without any result or error message,

Expected Behavior

Report a file not found error

Example log

Executing build with non-existing file

build ../OsvvmLibraries/Axi4/Axi4Stream/RunAllTests.pro
# transcript file C:/git/GitHub/OSVVM/sim_riviera/logs/2020.04/Axi4Stream_RunAllTests.log
# Start time 08:08:08
# set CURRENT_WORKING_DIRECTORY C:/git/GitHub/OSVVM/OsvvmLibraries/AXI4/Axi4Stream
# Start time 08:08:08
# Finish time 08:08:08
# Elasped time 0 minutes

Executing build with existing file

build ../OsvvmLibraries/Axi4/AxiStream/RunAllTests.pro
# transcript file C:/git/GitHub/OSVVM/sim_riviera/logs/2020.04/AxiStream_RunAllTests.log
# Start time 08:08:56
# set CURRENT_WORKING_DIRECTORY C:/git/GitHub/OSVVM/OsvvmLibraries/AXI4/AxiStream
# source C:/git/GitHub/OSVVM/OsvvmLibraries/AXI4/AxiStream/RunAllTests.pro
# library osvvm_TbAxiStream
# analyze ./testbench/TbAxiStream.vhd
# vcom -2008 -dbg -relax -work osvvm_TbAxiStream C:/git/GitHub/OSVVM/OsvvmLibraries/AXI4/AxiStream/testbench/TbAxiStream.vhd
...
...
...
JimLewis commented 3 years ago

Fixed in current revision. Thanks for the issue report.