Allow Debian packaging using builder's native architecture#288
Conversation
|
|
There was a problem hiding this comment.
Pull request overview
This PR updates the Debian 10 and 11 builder scripts so that Debian packages are built for the builder’s native architecture instead of being hardcoded to amd64.
Changes:
- Introduces an
ARCHvariable derived fromdpkg --print-architectureand uses it in the generated.debfilename. - Updates dependency extraction to attempt to strip the architecture suffix dynamically instead of assuming
:amd64. - Cleans up and corrects some
apt-getinstall lines and comments in the builder scripts.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| pkg/debian/builder-debian11/builder.sh | Uses the builder’s detected architecture for dependency normalization and output package naming; adjusts package installation commands. |
| pkg/debian/builder-debian10/builder.sh | Mirrors the architecture-aware dependency handling and output naming for Debian 10, plus minor cleanup of install/comment lines. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Hello @jwarnier, Thanks for the PR. Please check the two Copilot suggestions. |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Allow building the Debian packages for the builder's architecture (i.e. 'amd64' no longer hardcoded).
Works on ARM64.