Skip to content

ci: Add test env for riscv#17089

Closed
maflcko wants to merge 1 commit intobitcoin:masterfrom
maflcko:1909-ciRiscV
Closed

ci: Add test env for riscv#17089
maflcko wants to merge 1 commit intobitcoin:masterfrom
maflcko:1909-ciRiscV

Conversation

@maflcko
Copy link
Member

@maflcko maflcko commented Oct 9, 2019

No description provided.

@fanquake fanquake added the Tests label Oct 9, 2019
@laanwj
Copy link
Member

laanwj commented Oct 10, 2019

Concept ACK, of course, though to be honest I'm not sure there is much value to spending the time to run this for every PR and merge. RISC-V is not sufficiently different from the other architectures, and we don't have any RISC-V specific assembly code.

If we're going to test on wildly different architectures I'd prefer a big-endian MIPS run or such (see #6466).

@maflcko
Copy link
Member Author

maflcko commented Oct 10, 2019

Makes sense, and the qemu-riscv64 is only available on Ubuntu Cosmic, not Bionic (https://packages.ubuntu.com/bionic/qemu-user).

@maflcko maflcko closed this Oct 10, 2019
@maflcko maflcko deleted the 1909-ciRiscV branch October 10, 2019 14:19
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Dec 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants