Skip to content

Conversation

@jgarzik
Copy link
Contributor

@jgarzik jgarzik commented Nov 29, 2025

No description provided.

@jgarzik jgarzik requested a review from Copilot November 29, 2025 00:04
@jgarzik jgarzik self-assigned this Nov 29, 2025
@jgarzik jgarzik added the enhancement New feature or request label Nov 29, 2025
Copilot finished reviewing on behalf of jgarzik November 29, 2025 00:05
Copy link

Copilot AI left a comment

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 adds a comprehensive implementation of the pax utility (POSIX archiver) to the posixutils-rs project, including support for multiple archive formats (ustar, cpio, pax), multi-volume archives, path substitutions, and interactive mode operations.

Key Changes:

  • Implements pax archiver with read, write, list, copy, and append modes
  • Adds support for multi-volume archives, substitution patterns, and privilege preservation
  • Includes comprehensive test coverage for all major features

Reviewed changes

Copilot reviewed 33 out of 34 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pax/Cargo.toml Package configuration for posixutils-pax with test features
Cargo.toml Adds pax to workspace members
pax/archive.rs Core archive entry types and traits
pax/error.rs Error type definitions
pax/formats/mod.rs Archive format module organization
pax/interactive.rs Interactive rename functionality
pax/modes/*.rs Implementation of different pax operation modes
pax/multivolume.rs Multi-volume archive support
pax/options.rs Format options parsing and handling
pax/pattern.rs POSIX-style pattern matching
pax/subst.rs Path substitution support using POSIX regex
pax/tests/**/*.rs Comprehensive test suites for all features

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

@jgarzik jgarzik merged commit b8f07d5 into main Nov 29, 2025
4 checks passed
@jgarzik jgarzik deleted the merge-pax branch November 29, 2025 01:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants