OpenIndiana/ips
Watch
2
0
Fork
You've already forked ips
 0
Code Issues 29 Pull requests 2 Projects Releases 4 Packages Wiki Activity Actions

v1 repository tools: remove misleading commands and guard receives #49

Closed
forge wants to merge 0 commits from v1-repo-cli-20260923 into v1-integration-20260923
pull from: v1-repo-cli-20260923
merge into: OpenIndiana:v1-integration-20260923
Conversation 1 Commits - Files changed -
forge commented 2026-09-23 09:36:49 +00:00
First-time contributor
Copy link

pkg6repo previously accepted URI/TLS and filter options it ignored, let several no-op or destructive commands report success, and exposed a purge of historical obsoleted entries. This change rejects unsupported usage before mutation, removes the purge/archive overloads, preserves genuine pkg5 pkg.obsolete catalog behavior, and documents the surviving local commands in an illumos-style pkg6repo(1) page. pkg6recv now preflights requested packages, fails on empty or ambiguous selections, and says clearly when recursive receive can leave partial data. The legacy export uses atomic no-clobber output.

Validation: pkg6repo 28/28, pkg6recv 9/9 and focused libips export tests passed, fmt clean. Independent Opus review found no code blocker; it identified package manpage staging and release test-inventory updates, which will land on the integration branch before the RC tag. This draft targets the v1 integration line.

`pkg6repo` previously accepted URI/TLS and filter options it ignored, let several no-op or destructive commands report success, and exposed a purge of historical obsoleted entries. This change rejects unsupported usage before mutation, removes the purge/archive overloads, preserves genuine pkg5 `pkg.obsolete` catalog behavior, and documents the surviving local commands in an illumos-style `pkg6repo(1)` page. `pkg6recv` now preflights requested packages, fails on empty or ambiguous selections, and says clearly when recursive receive can leave partial data. The legacy export uses atomic no-clobber output. Validation: `pkg6repo` 28/28, `pkg6recv` 9/9 and focused libips export tests passed, fmt clean. Independent Opus review found no code blocker; it identified package manpage staging and release test-inventory updates, which will land on the integration branch before the RC tag. This draft targets the v1 integration line.
cli: make repository and receive commands fail explicitly
All checks were successful
CI / Linux (fmt / clippy / nextest / e2e / docs) (pull_request) Successful in 6m6s
Details
CI / illumos (OpenIndiana) build + nextest (pull_request) Successful in 7m44s
Details
f0ec530ead
toasterson commented 2026-09-27 13:13:03 +00:00
Owner
Copy link

Superseded by #54, which brings the whole RC13 line (feat/historical-import, a strict superset of v1-integration-20260923) to main in one pull request, as decided in ADR-015 item 6. Nothing from this branch is lost; closing to keep one integration path.

Superseded by #54, which brings the whole RC13 line (`feat/historical-import`, a strict superset of `v1-integration-20260923`) to `main` in one pull request, as decided in ADR-015 item 6. Nothing from this branch is lost; closing to keep one integration path.
toasterson closed this pull request 2026-09-27 13:13:06 +00:00
All checks were successful
CI / Linux (fmt / clippy / nextest / e2e / docs) (pull_request) Successful in 6m6s
Details
CI / illumos (OpenIndiana) build + nextest (pull_request) Successful in 7m44s
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
Compat/Breaking
Breaking change that won't be backward compatible

  
Crazy Idea
Something that went through my mind which is not part of the core use case but fun

  
Kind/Bug
Something is not working

  
Kind/Documentation
Documentation changes

  
Kind/Enhancement
Improve existing functionality

  
Kind/Feature
New functionality

  
Kind/Security
This is security issue

  
Kind/Testing
Issue or pull request related to testing

  
Priority/Critical
The priority is critical

  
Priority/High
The priority is high

  
Priority/Low
The priority is low

  
Priority/Medium
The priority is medium

  
Reviewed/Confirmed
Issue has been confirmed

  
Reviewed/Duplicate
This issue or pull request already exists

  
Reviewed/Invalid
Invalid issue

  
Reviewed/Won't Fix
This issue won't be fixed

  
Status/Abandoned
Somebody has started to work on this but abandoned work

  
Status/Blocked
Something is blocking this issue or pull request

  
Status/Need More Info
Feedback is required to reproduce issue or to continue work

  
needs-info
Waiting on reporter for more information

  
needs-triage
Maintainer needs to evaluate this issue

  
ready-for-agent
Fully specified, ready for an AFK agent

  
ready-for-human
Requires human implementation

  
wayfinder:grilling
Wayfinder decision ticket: grilling

  
wayfinder:map
Wayfinder map issue (parent of decision tickets)

  
wayfinder:prototype
Wayfinder decision ticket: prototype

  
wayfinder:research
Wayfinder decision ticket: research

  
wayfinder:task
Wayfinder ticket: task that unblocks a decision

  
wontfix
Will not be actioned

No labels
Compat/Breaking
Crazy Idea
Kind/Bug
Kind/Documentation
Kind/Enhancement
Kind/Feature
Kind/Security
Kind/Testing
Priority/Critical
Priority/High
Priority/Low
Priority/Medium
Reviewed/Confirmed
Reviewed/Duplicate
Reviewed/Invalid
Reviewed/Won't Fix
Status/Abandoned
Status/Blocked
Status/Need More Info
needs-info
needs-triage
ready-for-agent
ready-for-human
wayfinder:grilling
wayfinder:map
wayfinder:prototype
wayfinder:research
wayfinder:task
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
OpenIndiana/ips!49
No description provided.