Skip to content

Commit e5a79bd

Browse files
committed
claude: fix ~/.local/bin/openroad cruft
1 parent 88ece56 commit e5a79bd

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

.claude/commands/openroad-issue.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -226,7 +226,7 @@ Whittling tips:
226226

227227
Critical whittle.py notes:
228228
- `openroad` must be on PATH (whittle uses it internally to read ODB counts).
229-
Use `ln -sf $(pwd)/bazel-bin/openroad ~/.local/bin/`
229+
Use `PATH="$(pwd)/bazel-bin:$PATH"` — never install to `~/.local/bin/`.
230230
- Use absolute paths for `--base_db_path` and in `--step` (whittle changes cwd)
231231
- Don't pipe whittle output through `tail` — use `tee` with `pipefail`
232232
- Use `--use_stdout` when errors go to stdout (not just stderr)

.claude/commands/untar-and-run-report.md

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -175,10 +175,12 @@ Binary at `bazel-bin/openroad`. Record the hash for the report:
175175
bin/git-read upstream/OpenROAD-flow-scripts/tools/OpenROAD rev-parse HEAD
176176
```
177177

178-
Put it on PATH (or use the full path in the verify step):
178+
**Never install openroad to ~/.local/bin/ or anywhere on the default PATH.**
179+
That leaves cruft behind from previous runs. Instead, use `PATH=` to
180+
prepend the bazel-bin directory for each invocation:
179181

180182
```bash
181-
ln -sf $(realpath upstream/OpenROAD-flow-scripts/tools/OpenROAD/bazel-bin/openroad) ~/.local/bin/openroad
183+
OPENROAD_BIN=$(realpath upstream/OpenROAD-flow-scripts/tools/OpenROAD/bazel-bin)
182184
```
183185

184186
### 7. Verify the archive reproduces the bug
@@ -189,7 +191,7 @@ cp tmp/<project>/<module>_<stage>_deps/<stage>_<tag>.tar.gz tmp/verify-<tag>/
189191
cd tmp/verify-<tag>
190192
tar xf <stage>_<tag>.tar.gz
191193
cd <stage>_<tag>
192-
time echo exit | ./run-me-*.sh 2>&1 | tail -50
194+
time echo exit | PATH="$OPENROAD_BIN:$PATH" ./run-me-*.sh 2>&1 | tail -50
193195
```
194196

195197
Run this in the background — it may take minutes.

0 commit comments

Comments
 (0)