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

Use full output of git show-ref --tags to get tags for PushUpdateAddTag (#19235) #19236

Merged
merged 2 commits into from
Mar 29, 2022

Conversation

zeripath
Copy link
Contributor

Backport #19235

Strangely #19038 appears to relate to an issue whereby a tag appears to
be listed in git show-ref --tags but then does not appear when git show-ref --tags -- short_name is called.

As a solution though I propose to stop the second call as it is
unnecessary and only likely to cause problems.

Fix #19038

Signed-off-by: Andrew Thornton art27@cantab.net

…ag (go-gitea#19235)

Strangely go-gitea#19038 appears to relate to an issue whereby a tag appears to
be listed in `git show-ref --tags` but then does not appear when `git
show-ref --tags -- short_name` is called.

As a solution though I propose to stop the second call as it is
unnecessary and only likely to cause problems.

I've also noticed that the tags calls are wildly inefficient and aren't using the common cat-files - so these have been added.

I've also noticed that the git commit-graph is not being written on mirroring - so I've also added writing this to the migration which should improve mirror rendering somewhat.

Fix go-gitea#19038

Signed-off-by: Andrew Thornton <art27@cantab.net>
Co-authored-by: 6543 <6543@obermui.de>
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Mar 29, 2022
@6543
Copy link
Member

6543 commented Mar 29, 2022

@zeripath cherry-picked the latest state and force pushed ...

@6543 6543 requested a review from Gusted March 29, 2022 17:36
@zeripath
Copy link
Contributor Author

Thanks @6543

@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Mar 29, 2022
@lafriks lafriks merged commit db43f63 into go-gitea:release/v1.16 Mar 29, 2022
@zeripath zeripath deleted the backport-19235-v1.16 branch March 29, 2022 21:02
@go-gitea go-gitea locked and limited conversation to collaborators Apr 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. topic/repo-migration Migrate repos from other platforms to Gitea, or from Gitea to them type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants