Skip to content

Conversation

@kdowney-lot49
Copy link
Contributor

Description

Replaced deprecated nanolayer package with just a plain shell script using curl following the recommended installation method at https://buf.build.

NOTE: due to a pre-commit hook issue I did not remove library_scripts.sh as part of this PR, though it is no longer needed. It seems like the toodaloo hook doesn't handle removed files properly.

Related Issue(s)

Fixes #51

How to test

Ran locally in devcontainer with no issues on Linux ARM.

Evidence:

vscode /workspaces/devcontainers (main) $ file /usr/local/bin/buf
/usr/local/bin/buf: ELF 64-bit LSB executable, ARM aarch64, version 1 (SYSV), statically linked, BuildID[sha1]=df7f31ef6d82c378dfcb103a6e6f0a7887387263, stripped
vscode /workspaces/devcontainers (main) $ uname -a
Linux 4fb284e7db4c 6.8.0-50-generic #51-Ubuntu SMP PREEMPT_DYNAMIC Sat Nov  9 18:03:35 UTC 2024 aarch64 aarch64 aarch64 GNU/Linux

Previously the installer downloaded the Linux RISC-V binary.

@kdowney-lot49
Copy link
Contributor Author

@mrsimonemms I also ran pre-commit autoupdate to upgrade your built-in pre-commit hooks, but feel free to exclude that from merge if you want the fix to be more limited.

@mrsimonemms mrsimonemms self-requested a review February 9, 2026 10:10
@mrsimonemms
Copy link
Owner

Can you rebase the commit with the signed off by line please?

Signed-off-by: Kyle Downey <8682922+kdowney-lot49@users.noreply.github.com>
@kdowney-lot49
Copy link
Contributor Author

@mrsimonemms it looks like merge might block due to lack of signature on the original commit. If that's the case I will have to take care of that in the evening because it's a more involved fix than the signoff-off rebase.

@mrsimonemms mrsimonemms enabled auto-merge (rebase) February 9, 2026 11:16
@mrsimonemms
Copy link
Owner

Thanks @kdowney-lot49

@mrsimonemms mrsimonemms disabled auto-merge February 9, 2026 11:22
@mrsimonemms mrsimonemms merged commit 4480a9f into mrsimonemms:main Feb 9, 2026
11 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.

Incorrectly installed RISC-V version of buf on Linux ARM

2 participants