lowRISC / lowRISC/opentitan

[dvsim] Honor BUILD_ROOT for placing scratch directory

Open
#5,031 4 comments 0 reactions 1 assignee View on GitHub

@sriyerg is already working on this.

Since Mar 25, 2022.

Component:DV Component:Tooling Earlgrey-PROD Triaged Priority:P3 Type:Enhancement
Dominant language
SystemVerilog
Stars
3.6k
Forks
1.1k
Avg merge
2d 22h
Merged PRs (30d)
141

Description

BUILD_ROOT is the designated environment variable for build outputs, see `util/build_consts.sh`. In CI environments, we set this variable to a directory outside of the source tree. dvsim currently uses SCRATCH_ROOT instead.

We should
- Improve dvsim to honor BUILD_ROOT
- Source util/build_consts.sh before running dvsim (e.g. in lint jobs)

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.