HDLUtils / hdlregression

MIT License
15 stars 5 forks source link

Problem with com_options #25

Closed rsek1498 closed 4 months ago

rsek1498 commented 5 months ago

Hi, I've the following problem when I set the com_options:

hr.set_code_coverage("bcesf", "code_coverage.ucdb") ... hr.add_files("../../../../01_RTL/bloc1/src/*.vhd", "my_lib", com_options='nofsmresettrans', code_coverage=True)

Compiling library: my_lib QuestaSim-64 vmap 2022.4_2 Lib Mapping Utility 2022.12 Dec 9 2022 vmap my_lib /home/user1/Projet/proj1/01_RTL/bloc1/test/hdl_regression/hdlregression/l ibrary/my_lib Modifying modelsim.ini QuestaSim-64 vcom 2022.4_2 Compiler 2022.12 Dec 9 2022 Start time: 11:49:04 on May 13,2024 vcom "+cover=bcesf" -work /home/user1/Project/proj1/01_RTL/bloc1/test/hdl_regression/hdlre gression/library/my_lib /home/user1/Project/proj1/01_RTL/bloc1/src/my_pkg.vhd Error: (vcom-7) Failed to open design unit file " " in read mode. Error: (vcom-7) Failed to open design unit file " " in read mode.

Without, it's ok: hr.set_code_coverage("bcesf", "code_coverage.ucdb") ... hr.add_files("../../../../01_RTL/bloc1/src/*.vhd", "my_lib", code_coverage=True)

Compiling library: my_lib QuestaSim-64 vmap 2022.4_2 Lib Mapping Utility 2022.12 Dec 9 2022 vmap my_lib /home/user1/Project/proj1/01_RTL/bloc1/test/hdl_regression/hdlregression/library/my_lib Modifying modelsim.ini QuestaSim-64 vcom 2022.4_2 Compiler 2022.12 Dec 9 2022 Start time: 13:10:35 on May 13,2024 vcom -suppress 1346,1236,1090 -2008 "+cover=bcesf" -work /home/user1/Project/proj1/01_RTL/bloc1/test/hdl_regression/hdlregression/library/my_lib /home/user1/Project/proj1/01_RTL/bloc1/src/my_pkg.vhd -- Loading package STANDARD -- Loading package TEXTIO -- Loading package std_logic_1164 -- Compiling package my_pkg End time: 13:10:35 on May 13,2024, Elapsed time: 0:00:00 Errors: 0, Warnings: 0 (I've changed the name of files and directories).

You can noticed that space characters had been added... This problem arises under Unbutu. I've not this problem under Windows but it's not the same files and directories...

Best regards

HDLUtils commented 5 months ago

Hi, thanks for reporting. Could you try again with setting the com_options as a list? It should not be necessary, but will help me investigate. hr.add_files("../../../../01_RTL/bloc1/src/*.vhd", "my_lib", com_options=['nofsmresettrans'], code_coverage=True)

Regards

rsek1498 commented 5 months ago

Hi,

1) First, in my first post, space characters are not displayed correctly: there was 25 space character between "/hdlregression/l" and "ibrary/my_lib". Same for "hdl_regression/hdlr" and "gression/library/my_lib" 2) My vcom option is incorrect: it's "-nofsmresettrans" and no "nofsmresettrans" ! but the result is the same 3) With com_options=['-nofsmresettrans'], it's better: For first file: "vcom -nofsmresettrans"+cover=bcesf" -work /home/user1/Project/proj1/01_RTL/bloc1/test/hdl_regression/hdlre gression/library/my_lib /home/user1/Project/proj1/01_RTL/bloc1/src/my_pkg.vhd => OK But for the second file, I've a syntax error for my file because -2008 has disappeared. I try to add com_options=['-nofsmresettrans -2008'] but I've the message "**Error (suppressible): (vcom-1902) Option "-nofsmresettrans -2008" is not a valid option for this tool". Don't understand why... I try to add " hdl_version='2008' " to hr.add_files but it's not taking into account.

I hope this will help. Best regards

HDLUtils commented 5 months ago

Hi, I am working on this issue and will have a fix for you very soon. In the meanwhile you should be able to proceed if you correct your second com_options list from ['-nofsmresettrans -2008'] to ['-nofsmresettrans', '-2008'].

Br

rsek1498 commented 5 months ago

Hi, Thanks, it works with ['-nofsmresettrans', '-2008']. I will wait for the new version before modifying all my files. I take advantage of this post to indicate that this software is really very useful! A perfect complement to UVVM. Best regards.

HDLUtils commented 4 months ago

Updated to accepting com_options as strings in addition to list of stings: "opt1" or "opt1 opt2" or "opt1, opt2" will now be converted to list by HDLRegression.