Skip to content

boot-utils: Add RISC-V to use '--use-cbl-qemu'#33

Merged
nathanchance merged 1 commit into
ClangBuiltLinux:mainfrom
nathanchance:riscv-use-cbl-qemu
Feb 5, 2021
Merged

boot-utils: Add RISC-V to use '--use-cbl-qemu'#33
nathanchance merged 1 commit into
ClangBuiltLinux:mainfrom
nathanchance:riscv-use-cbl-qemu

Conversation

@nathanchance

Copy link
Copy Markdown
Member

Signed-off-by: Nathan Chancellor <nathan@kernel.org>

@nickdesaulniers nickdesaulniers left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM thanks 🖲️ 🦠

Comment thread boot-qemu.sh
Comment on lines +233 to +234
[[ -d ${QEMU_BINARIES} ]] || git clone https://github.com/ClangBuiltLinux/qemu-binaries "${QEMU_BINARIES}"
git -C "${QEMU_BINARIES}" pull --rebase

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

consider shallow clone and git reset --hard FETCH_HEAD? Though probably overkill as we only have like 2 commits in that repo...it is of binaries though.

@nathanchance

Copy link
Copy Markdown
Member Author

Yes, I can double back to shallow cloning in the future. Thanks for the review!

@nathanchance nathanchance merged commit fde0366 into ClangBuiltLinux:main Feb 5, 2021
@nathanchance nathanchance deleted the riscv-use-cbl-qemu branch February 5, 2021 01:25
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