Skip to content

ci: test setup.sh local installation path - #4519

Open
GITTYCAT12 wants to merge 1 commit into
The-OpenROAD-Project:masterfrom
GITTYCAT12:ci/test-setup-sh-local-install
Open

GITTYCAT12 wants to merge 1 commit into
The-OpenROAD-Project:masterfrom
GITTYCAT12:ci/test-setup-sh-local-install

Conversation

@GITTYCAT12

Copy link
Copy Markdown

What does this PR do?

Adds CI coverage for the host-local installation path through setup.sh.

The existing installer workflow exercises DependencyInstaller.sh through Docker, but it does not execute the user-facing setup.sh path directly. This can allow regressions in setup-specific behavior to go unnoticed.

This PR adds an Ubuntu 22.04 job that:

  • checks out ORFS with submodules
  • runs sudo ./setup.sh
  • builds OpenROAD with ./build_openroad.sh --local
  • sources env.sh and verifies the resulting yosys and openroad installations

It also makes changes to the workflow itself trigger the installer workflow and explicitly includes setup.sh in the path filters.

This addresses the intent of upstream issue #4453 without changing the production flow or installer implementation.

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@openroad-ci

openroad-ci commented Sep 5, 2026 •

Copy link
Copy Markdown
Member

🔍 QoR check

Metrics reflect the PR merge build — i.e. what will land on the target branch.

Commit b6fb092 · Jenkins build #19 · Baseline: build · View build on dashboard

62 design(s) checked — 0 with regression(s), 0 without a comparable baseline.

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.

2 participants