Skip to content

recipes: use nobranch=1 for upstream git sources#41

Open
kvinwang wants to merge 1 commit intomainfrom
fix/ovmf-nobranch
Open

recipes: use nobranch=1 for upstream git sources#41
kvinwang wants to merge 1 commit intomainfrom
fix/ovmf-nobranch

Conversation

@kvinwang
Copy link
Copy Markdown
Collaborator

Summary

  • Replace branch=master with nobranch=1 in OVMF recipe
  • Replace branch=zfs-2.2-release with nobranch=1 in ZFS recipe
  • SRCREV is already pinned to specific commits, so branch names are unnecessary and can cause fetch failures if upstream rebases

Test plan

  • bitbake dstack-ovmf fetches and builds successfully
  • bitbake dstack-zfs fetches and builds successfully

Replace branch= with nobranch=1 in OVMF and ZFS recipes since
SRCREV is already pinned to specific commits. This makes fetching
more robust against upstream branch rebases or renames.
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR improves fetch robustness for git-based Yocto recipes by removing dependency on upstream branch names when SRCREV is already pinned to an exact commit.

Changes:

  • Update the ZFS recipe to use nobranch=1 instead of branch=zfs-2.2-release.
  • Update the OVMF/edk2 recipe to use nobranch=1 instead of branch=master.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
meta-dstack/recipes-core/dstack-zfs/dstack-zfs_2.2.5.bb Switch git fetch parameters to nobranch=1 while keeping SRCREV pinned for reproducible fetches.
meta-dstack/recipes-core/dstack-ovmf/dstack-ovmf_git.bb Switch gitsm fetch parameters to nobranch=1 to avoid failures if the upstream default branch changes/rebases.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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