ipbus/ipbb

ipbb vivado reset seems to be broken in v0.5.2

Closed this issue · 3 comments

Hi,

after moving to ipbb v0.5.2 the reset command seems to be broken. The .gitlab-ci.yml that triggered the problem can be found at [1], I've also pasted the error message below. (Not critical because I called reset somewhat unnecessarily anyway.. ) All other commands I tried (ipbb vivado project as well as synthesis/implementation) worked without problems.

Cheers,
Dinyar

[1] https://gitlab.cern.ch/dinyar/p2gt_demultiplexing/blob/7fbddee2ba10f3ae893156cbabe77e326e5f8dce/.gitlab-ci.yml#L56

Error message:

$ ipbb vivado reset
[snip]
reset | Vivado% open_project /home/gitlab-runner/builds/d44ced6a/0/dinyar/p2gt_demultiplexing/build/demuxing_tests/proj/p2gt_demultiplexing/top/top
reset | open_project /home/gitlab-runner/builds/d44ced6a/0/dinyar/p2gt_demultiplexing/build/demuxing_tests/proj/p2gt_demultiplexing/top/top
reset | ERROR: [Coretcl 2-155] Invalid project file name '/home/gitlab-runner/builds/d44ced6a/0/dinyar/p2gt_demultiplexing/build/demuxing_tests/proj/p2gt_demultiplexing/top/top'. File must have a valid Vivado project extension (.xpr/.ppr).
reset | Vivado% quit
reset | quit
reset | INFO: [Common 17-206] Exiting Vivado at Tue Oct  1 10:10:19 2019...
reset | ----------------------------------------
Vivado error/critical warnings detected
ERROR: [Coretcl 2-155] Invalid project file name '/home/gitlab-runner/builds/d44ced6a/0/dinyar/p2gt_demultiplexing/build/demuxing_tests/proj/p2gt_demultiplexing/top/top'. File must have a valid Vivado project extension (.xpr/.ppr).

Hi, it surely is. Patch coming up soon. Thanks for reporting!

Fixed in master. Can you give it a go?

Looks good now, thanks!