Skip to content

Makefile: fix paths in the VCS flow - #3596

Open
sgbird wants to merge 1 commit into
openhwfoundation:masterfrom
sgbird:fix_vcs_build
Open

sgbird wants to merge 1 commit into
openhwfoundation:masterfrom
sgbird:fix_vcs_build

Conversation

@sgbird

@sgbird sgbird commented Sep 27, 2026

Copy link
Copy Markdown

Two paths in the VCS flow are resolved against the wrong location:

  • vcs_build changes into $(vcs-library) before invoking vlogan, but the default flist is relative to the repository root (core/Flist.cva6). vlogan then fails with:

    Error-[P1ARGS-CANTOPN-F] Cannot open file
      Unable to open 'core/Flist.cva6'
    

    Resolve the flist with $(abspath ...) so it is independent of the current working directory.

  • the vcs run target passes '$(root-dir)/init_testharness.do' to Verdi, but the file actually lives in util/. The do-file is therefore never loaded and no FSDB is dumped. Point it to '$(root-dir)/util/init_testharness.do'.

  • [y] I have searched for similar pull requests
  • [y] I am a human engaging in an interpersonal interaction. During this interaction, my words are my own and are not generated. If relevant, I provide links to my sources.

It is helpful when VCS/Verdi were used as compile/debug tools

Two paths in the VCS flow are resolved against the wrong location:

- vcs_build changes into $(vcs-library) before invoking vlogan, but the
  default flist is relative to the repository root (core/Flist.cva6).
  vlogan then fails with:

      Error-[P1ARGS-CANTOPN-F] Cannot open file
        Unable to open 'core/Flist.cva6'

  Resolve the flist with $(abspath ...) so it is independent of the
  current working directory.

- the vcs run target passes '$(root-dir)/init_testharness.do' to Verdi,
  but the file actually lives in util/. The do-file is therefore never
  loaded and no FSDB is dumped. Point it to
  '$(root-dir)/util/init_testharness.do'.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant