Skip to content

Feature/source package rf#6

Merged
embtom merged 10 commits into
mainfrom
feature/source_package_rf
Jul 3, 2026
Merged

Feature/source package rf#6
embtom merged 10 commits into
mainfrom
feature/source_package_rf

Conversation

@embtom

@embtom embtom commented Jun 30, 2026

Copy link
Copy Markdown
Member
  • deb_source: cache parsed changelog; invalidate after git checkout in reset_source_tree
  • debuild: pass -sa flag to always include orig tarball in .changes for source/full builds
  • apt: improve binary and source package existence detection with per-file [EXISTS]/[MISSING] logging
  • AptManager: reorganise class into logical sections (repository, installation, existence checks, private helpers)
  • log: add error logging for failed package upload
  • Source package upload no longer fails when .orig.tar.* is absent from .changes on non-first revisions

@github-actions

github-actions Bot commented Jun 30, 2026

Copy link
Copy Markdown

☂️ Code Coverage

current status: ✅

Overall Coverage

Statements Covered Coverage Threshold Status
1697 1371 81% 0% 🟢

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: 2d49bea by action🐍

@embtom embtom force-pushed the feature/source_package_rf branch from f11847f to 34a56df Compare July 1, 2026 04:00
@embtom embtom force-pushed the feature/source_package_rf branch from 34a56df to 366a27e Compare July 1, 2026 04:03
@embtom embtom requested a review from Copilot July 1, 2026 04:51

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR enhances the packtly-builder pipeline around source package handling (notably pristine-tar workflows), while also improving developer/container ergonomics (fixed-path venv, optional custom CA certs) and updating documentation to match the expanded workflow.

Changes:

  • Added/updated Debian source build logic (GitPython-backed repo detection, pristine-tar branch handling) and introduced a focused test suite for this behavior.
  • Standardized Poetry/venv behavior in containers and dev tooling (fixed /opt/venv, Poetry configured not to create per-project venvs).
  • Improved operational/dev UX: optional corporate CA injection in the base image, VS Code task tweaks, and a significantly expanded README.

Reviewed changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
Readme.md Expanded usage docs, pipeline explanation, and CLI options/modes.
packtly-builder/tooling/pyproject.toml Adds GitPython dependency used by updated source build logic.
packtly-builder/tooling/poetry.toml Forces Poetry to install into an existing venv (no auto-created venv).
packtly-builder/tooling/packtly_builder_tooling/tests/test_deb_source.py New tests covering quilt/native logic and git/pristine-tar handling.
packtly-builder/tooling/packtly_builder_tooling/parts/deb_source.py Refactors git operations to GitPython and adds repo caching/helpers.
packtly-builder/tooling/packtly_builder_tooling/parts/aptly.py Minor formatting-only change to connection error raising.
packtly-builder/tooling/packtly_builder_tooling/parts/apt.py Minor formatting-only change in DSC parsing condition.
packtly-builder/tooling/justfile Adjusts prepare flow and removes local Poetry venv config step.
packtly-builder/containers/Containerfile Adds python venv support, fixed-path venv, and optional custom CA cert installation.
packtly-builder/containers/conf/ca-certificates/.gitkeep Documents how to add custom CA certs for image builds.
packtly-builder/containers/conf/ca-certificates/.gitignore Prevents committing actual .crt files.
.vscode/tasks.json Switches to a shell-based just task with explicit working directory.
.vscode/settings.json Updates Python interpreter path to the new fixed venv location.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packtly-builder/tooling/packtly_builder_tooling/parts/deb_source.py
Comment thread packtly-builder/tooling/packtly_builder_tooling/parts/deb_source.py Outdated
Comment thread packtly-builder/tooling/packtly_builder_tooling/parts/deb_source.py
Comment thread packtly-builder/tooling/justfile
@embtom embtom force-pushed the feature/source_package_rf branch from f5d3dc7 to 2d49bea Compare July 3, 2026 22:38
@embtom embtom merged commit dd4545c into main Jul 3, 2026
8 checks passed
@embtom embtom deleted the feature/source_package_rf branch July 3, 2026 23:02
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.

2 participants