Skip to content

refactor: extract generic tar-walking logic into tarball pkg - #310

Merged
niemeyer merged 1 commit into
canonical:mainfrom
upils:bin-extract
Jul 10, 2026
Merged

niemeyer merged 1 commit into
canonical:mainfrom
upils:bin-extract

Conversation

@upils

@upils upils commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator
  • Have you signed the CLA?

This refactoring PR moves the package-format-agnostic extraction logic (glob matching, hard-link two-pass handling, parent-directory permission preservation) out of deb and into a new tarball package, leaving deb with only the deb-specific DataReader (the ar + data.tar.* opener).

This is a pure, behavior-preserving refactor, no logic was altered — only relocated. It prepares the ground for adding an XZ-strict data reader to enable bin package extraction.

@upils upils added Simple Nice for a quick look on a minute or two Priority Look at me first labels Jul 9, 2026

@niemeyer niemeyer left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you very much for isolating the move.

@niemeyer
niemeyer merged commit 3f42a28 into canonical:main Jul 10, 2026
18 checks passed
@ROCKsBot

Copy link
Copy Markdown
Command Mean [s] Min [s] Max [s] Relative
BASE 15.144 ± 0.053 15.023 15.193 1.01 ± 0.01
HEAD 15.064 ± 0.097 14.844 15.188 1.00

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Priority Look at me first Simple Nice for a quick look on a minute or two

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants