Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stray tag 'rm' #654

Closed
jwnimmer-tri opened this issue Dec 17, 2024 · 1 comment
Closed

Stray tag 'rm' #654

jwnimmer-tri opened this issue Dec 17, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@jwnimmer-tri
Copy link
Contributor

Description

It seems like gz-math has a mistaken tag pushed, named rm:

I imagine that is unwanted, so I thought I'd mention it here in case you wanted to delete it.


Also BTW it's impossible for me to tell for sure, but it seems like the gz-math8_8.1.0 tag was initially pushed to point to 74416aa but then later was overwritten with 2179284?

If that's what happened, then please note that overwriting release tags is extremely bad juju and causes all kinds of headaches for downstream projects. Luckily Drake mirrors all of our externals so the change to source code integrity only generated a warning instead of a build error, but other projects might not be so lucky. If mistake happens during the release process that's understandable, but in that case please consider doing another patch release right away (e.g., 8.1.1), instead of overwriting an existing one.

Or if that's not what happened and instead it was some mistake in Drake's mirroring, I'm sorry and please excuse the interruption.

@jwnimmer-tri jwnimmer-tri added the bug Something isn't working label Dec 17, 2024
@azeey
Copy link
Contributor

azeey commented Dec 17, 2024

Yeah, I think the rm tag was a mistake. I'll remove it shortly.

Also BTW it's impossible for me to tell for sure, but it seems like the gz-math8_8.1.0 tag was initially pushed to point to 74416aa but then later was overwritten with 2179284?

You are correct. There was an error in the initial release and the 8.1.0 tag was attached to the wrong commit. I overwrote the tag when fixing the release thinking overwriting was better than having the tag point to the wrong commit. In hindsight, it was the wrong thing to do. My apologies for the headache.

@azeey azeey closed this as completed Dec 17, 2024
@github-project-automation github-project-automation bot moved this from Inbox to Done in Core development Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Done
Development

No branches or pull requests

2 participants