scripts/install.in: add riscv64 support to installer

[jdek]

Motivation

The artifacts are already built and hosted, the install script just needs to be taught about riscv64.

Context

Making this change manually to the install script the Nix installation was able to succeed.

Priorities and Process

Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

[cole-h]

I believe you'll also need to add riscv64 here:

nix/maintainers/upload-release.pl

Lines 161 to 165 in ef5c846

	downloadFile("binaryTarball.i686-linux", "1");
	downloadFile("binaryTarball.x86_64-linux", "1");
	downloadFile("binaryTarball.aarch64-linux", "1");
	downloadFile("binaryTarball.x86_64-darwin", "1");
	downloadFile("binaryTarball.aarch64-darwin", "1");

[jdek]

@cole-h wouldn't that require a native riscv64 builder? the cross-compiled one is already downloaded:

nix/maintainers/upload-release.pl

Lines 174 to 177 in ef5c846

	eval {
	downloadFile("binaryTarballCross.x86_64-linux.riscv64-unknown-linux-gnu", "1");
	};
	warn "$@" if $@;

Though this should probably be hooked up to the cross-compiled store path for riscv64 here too:

nix/maintainers/upload-release.pl

Lines 240 to 248 in ef5c846

	# Upload nix-fallback-paths.nix.
	write_file("$tmpDir/fallback-paths.nix",
	"{\n" .
	" x86_64-linux = \"" . getStorePath("build.x86_64-linux") . "\";\n" .
	" i686-linux = \"" . getStorePath("build.i686-linux") . "\";\n" .
	" aarch64-linux = \"" . getStorePath("build.aarch64-linux") . "\";\n" .
	" x86_64-darwin = \"" . getStorePath("build.x86_64-darwin") . "\";\n" .
	" aarch64-darwin = \"" . getStorePath("build.aarch64-darwin") . "\";\n" .
	"}\n");

[cole-h]

Oh oops you're right, I meant to point at the fallback-paths.nix writing. Not sure why I decided the place I pointed to was the right place 😅

[jdek]

@cole-h something like 73f9afd, I presume. I'm looking at https://hydra.nixos.org/build/260037794 for reference.

[roberth]

Looks to be complete.
Not sure about fallback paths, but I don't see harm in it, so 👍 from me.

[roberth]

The lack of armv7 here suggests that this is not strictly necessary, but it seems like adding cross builds here could be useful for those who need it, when they need it, which would be rare.

I don't know the history of this feature. wdyt @edolstra?

[edolstra]

Merging this with the caveat that it's best-effort (since we're not actually testing whether it works, only cross-building) and may be disabled if it breaks and blocks the release (since none of the maintainers are really capable of fixing riscv64 issues).

[jdek]

Cool, thanks. Feel free to @ me to test RISC-V related stuff.

[nixos-discourse]

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/2024-06-03-nix-team-meeting-minutes-149/46582/1

[Mic92]

I also got access to a bigger riscv64 machine.