Skip to content

Conversation

@simonbeaudoin0935
Copy link
Contributor

@simonbeaudoin0935 simonbeaudoin0935 commented Oct 20, 2025

Switched the input parameter ubuntu-codename to distro-codename to better align with what it represents.
The sbuild --distro parameter is what this input represents, so having distro- in the name is better.

docker_build_deb.py :

  • Verify Docker CLI/daemon and socket permissions with actionable errors.
  • Ensure builder image (qualcomm-linux/pkg-build:{arch}-latest); build from ../docker/Dockerfile.{arch} if missing.
  • Run gbp buildpackage inside the container (mounts src/output, preserves uid:gid, sets git safe.directory).
  • Stream build output live; create output dir and print last 500 lines of .build on failure.

@simonbeaudoin0935 simonbeaudoin0935 force-pushed the development branch 2 times, most recently from cab1d22 to 491ed45 Compare October 20, 2025 18:18
@simonbeaudoin0935 simonbeaudoin0935 changed the title docker_deb_build: verify docker, build image, run gbp in container Switch ubuntu-codename to distro-codename and new deocker_build_deb.py script Oct 20, 2025
@simonbeaudoin0935 simonbeaudoin0935 changed the title Switch ubuntu-codename to distro-codename and new deocker_build_deb.py script Switch ubuntu-codename to distro-codename and new docker_build_deb.py script Oct 20, 2025
Builds arm64 packages nativaly on arm64, or cross compiles on amd64

- Choice of ditribution codename between 'noble' and 'questing' since
  those are the two chroots initially created in the Dockerfile.{arch}.
- Verify Docker CLI/daemon and socket permissions with actionable errors.
- Ensure builder image (qualcomm-linux/pkg-build:{arch}-latest); build from ../docker/Dockerfile.{arch} if missing.
- Run gbp buildpackage inside the container (mounts src/output, preserves uid:gid, sets git safe.directory).
- Stream build output live; create output dir and print last 500 lines of .build on failure.

Signed-off-by: Simon Beaudoin <[email protected]>
Copy link
Contributor

@bjordiscollaku bjordiscollaku left a comment

Choose a reason for hiding this comment

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

Looks good.

@simonbeaudoin0935 simonbeaudoin0935 merged commit be8949a into main Oct 20, 2025
6 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.

3 participants