Skip to content

Comments

Rename package specs & retract v1.3.0 tag#1

Merged
thaJeztah merged 1 commit intomoby:mainfrom
corhere:fix-package-name
Feb 9, 2024
Merged

Rename package specs & retract v1.3.0 tag#1
thaJeztah merged 1 commit intomoby:mainfrom
corhere:fix-package-name

Conversation

@corhere
Copy link
Contributor

@corhere corhere commented Feb 9, 2024

The package at package path github.com/moby/docker-image-spec/specs-go was assigned the package name v1. It is supposed to have the package name specs to mirror the name of the
github.com/opencontainers/image-spec/specs-go package, but this mistake was not caught until after stable module versions were tagged. Fix the package name and retract all the existing module versions.

@corhere corhere requested review from thaJeztah and vvoland February 9, 2024 15:29
@corhere
Copy link
Contributor Author

corhere commented Feb 9, 2024

For reference, retract directives only need to be present in the highest numbered version of the module. Tags v1.0.1, v1.1.1 and v1.2.1 only need to be pushed with the package line fix backported; no need to touch their go.mod files.

@thaJeztah
Copy link
Member

v1.3.0 is the only tag that has code in the repo; older tags only have the specification

@corhere
Copy link
Contributor Author

corhere commented Feb 9, 2024

Well that makes things easier! I'll remove the unnecessary retractions, one sec

The package at package path github.com/moby/docker-image-spec/specs-go
was assigned the package name 'v1'. It is supposed to have the package
name 'specs' to mirror the name of the
github.com/opencontainers/image-spec/specs-go package, but this mistake
was not caught until after stable module versions were tagged. Fix the
package name and retract the v1.3.0 tag.

Signed-off-by: Cory Snider <[email protected]>
Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

@corhere corhere changed the title Rename package specs & retract all tags Rename package specs & retract v1.3.0 tag Feb 9, 2024
@thaJeztah thaJeztah merged commit f1d00eb into moby:main Feb 9, 2024
@thaJeztah
Copy link
Member

Not near a computer; are you able to create a tag, @corhere ?

@corhere corhere deleted the fix-package-name branch February 9, 2024 17:27
@corhere
Copy link
Contributor Author

corhere commented Feb 9, 2024

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