Skip to content

ci: update qemu-hisilicon clone URLs to OpenIPC org - #212

Merged
widgetii merged 2 commits into
mainfrom
ci/qemu-hisilicon-openipc-org
Aug 16, 2026
Merged

widgetii merged 2 commits into
mainfrom
ci/qemu-hisilicon-openipc-org

Conversation

@widgetii

Copy link
Copy Markdown
Member

The qemu-hisilicon repo was transferred from a personal account to the OpenIPC organisation (OpenIPC/qemu-hisilicon). GitHub creates a redirect from the old URL, but CI should reference the canonical location.

Changes

  • qemu-ive-ops job: update clone URL and comments
  • qemu-nnie-forward job: update clone URL and comments
  • Boot-test job: update clone URL

All three jobs cloned from the old personal-account URL; all now point to github.com/OpenIPC/qemu-hisilicon.

The repo was transferred from a personal account to the OpenIPC
organisation. Update all three clone steps and associated comments
in the qemu-ive-ops, qemu-nnie-forward, and boot-test jobs.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@qodo-free-for-open-source-projects

Copy link
Copy Markdown

PR Summary by Qodo

ci: point qemu-hisilicon clones at OpenIPC org repository

⚙️ Configuration changes 🕐 Less than 5 minutes

Grey Divider

AI Description

• Update CI clone URLs to use the canonical OpenIPC/qemu-hisilicon repository.
• Refresh related workflow comments to reflect the repo transfer.
• Apply consistently across qemu-ive-ops, qemu-nnie-forward, and boot-test jobs.
Diagram

graph TD
  A["GitHub Actions workflow"] --> B["qemu-ive-ops job"] --> E{{"OpenIPC/qemu-hisilicon"}}
  A --> C["qemu-nnie-forward job"] --> E
  A --> D["boot-test job"] --> E
Loading
High-Level Assessment

Updating the workflow to reference the canonical OpenIPC/qemu-hisilicon URL is the right approach; it avoids relying on GitHub redirects and reduces future fragility. A centralized env var for the repo URL could reduce repetition, but the current change is already clear and low-risk.

Files changed (1) +8 / -8

Other (1) +8 / -8
build.ymlSwitch qemu-hisilicon clone URLs to OpenIPC org (and update comments) +8/-8

Switch qemu-hisilicon clone URLs to OpenIPC org (and update comments)

• Replaces all occurrences of the widgetii/qemu-hisilicon clone URL with OpenIPC/qemu-hisilicon across the qemu-ive-ops, qemu-nnie-forward, and boot-test jobs. Updates adjacent comments describing the pinned SHA and upstream references to match the new org location.

.github/workflows/build.yml

@qodo-free-for-open-source-projects

qodo-free-for-open-source-projects Bot commented Aug 16, 2026

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0) 🎨 UX issues (0) 🔗 Cross-repo conflicts (0) 📜 Skill insights (0)

Grey Divider


Remediation recommended

1. Update stale upstream references ✓ Resolved 🐞 Bug ⚙ Maintainability
Description
The workflow now clones OpenIPC/qemu-hisilicon, but local runbooks and the QEMU test helper still
instruct users to obtain qemu-hisilicon from widgetii/qemu-hisilicon. Maintainers following those
instructions can use a different/stale upstream location than CI, causing local reproduction and
synchronization guidance to diverge.
Code

.github/workflows/build.yml[728]

+          git clone --depth 1 https://github.com/OpenIPC/qemu-hisilicon.git /tmp/qemu-hisilicon
Evidence
The changed workflow explicitly establishes OpenIPC/qemu-hisilicon as the source used by CI, while
read repository files retain old operational references: README links to the old repository for
generation labels and QEMU smoke tests; the IVE README names it as the source, links to it, and says
CI builds from it; run-qemu.sh labels it as the required source; and bootrom instructions require
the old repository and describe its CI job. These are actionable setup/documentation references
rather than only historical attribution.

README.md[22-22]
README.md[433-433]
bootrom/README[65-68]
bootrom/README[109-115]
kernel/ive_neo/test/qemu/README.md[24-33]
kernel/ive_neo/test/qemu/README.md[61-70]
kernel/ive_neo/test/qemu/run-qemu.sh[7-10]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
CI now uses `OpenIPC/qemu-hisilicon`, but operational documentation and helper comments still point to `widgetii/qemu-hisilicon`, leaving local setup and synchronization guidance inconsistent with CI.
## Issue Context
Update active links, clone/source instructions, and CI descriptions to the transferred repository. Preserve historical issue/PR attribution where it is intentionally documenting the old namespace.
## Fix Focus Areas
- README.md[22-22]
- README.md[433-433]
- bootrom/README[65-68]
- bootrom/README[109-115]
- kernel/ive_neo/test/qemu/README.md[24-33]
- kernel/ive_neo/test/qemu/README.md[61-70]
- kernel/ive_neo/test/qemu/run-qemu.sh[7-10]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


Grey Divider

Tip of the day
💡 Did you know, you can add REVIEW.md to your repo root and Qodo follows it on every PR

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

Comment thread .github/workflows/build.yml
Follow-up to the CI URL update: fix stale references in docs and
run-qemu.sh that still pointed to the old personal-account location.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@widgetii

Copy link
Copy Markdown
Member Author

/request

@widgetii

Copy link
Copy Markdown
Member Author

/review

@qodo-free-for-open-source-projects

Copy link
Copy Markdown

Code review by qodo was updated up to the latest commit b40743f

@widgetii
widgetii merged commit f5e64d3 into main Aug 16, 2026
35 checks passed
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