Skip to content

Conversation

@CorySanin
Copy link
Contributor

Current template uses the owner followed by the instance URL as the repo name. Technically this can work if the repo happens to be named the exact same way. But if, for example, you follow the docs, you'll end up with a package in core while the pacman conf refers to a non-existent repo testuser.gitea.example.com. Whatever is in the square brackets get substituted in for $repo, so we do not want anything except the exact repo name there.

And since it's now referring to the repo and not the owner, I've updated the pacman conf to show all repositories.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Nov 2, 2025
@github-actions github-actions bot added the modifies/templates This PR modifies the template files label Nov 2, 2025
@CorySanin
Copy link
Contributor Author

image

@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Nov 3, 2025
@silverwind
Copy link
Member

silverwind commented Nov 3, 2025

I think this will need a docs update for the pacman.conf section:

https://docs.gitea.com/usage/packages/arch/#configuring-the-package-registry

Replace [{owner}.gitea.example.com] with [custom], whatever repository was specified during publish.

@silverwind silverwind added the docs-update-needed The document needs to be updated synchronously label Nov 3, 2025
@CorySanin
Copy link
Contributor Author

@silverwind agree, will do

@CorySanin
Copy link
Contributor Author

@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 Nov 3, 2025
@lunny lunny added the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Nov 4, 2025
@github-actions github-actions bot removed the docs-update-needed The document needs to be updated synchronously label Nov 4, 2025
@silverwind silverwind merged commit 851db77 into go-gitea:main Nov 4, 2025
25 checks passed
@GiteaBot GiteaBot added this to the 1.26.0 milestone Nov 4, 2025
@GiteaBot GiteaBot removed the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Nov 4, 2025
@CorySanin CorySanin deleted the arch-repo-fixes branch November 4, 2025 17:20
zjjhot added a commit to zjjhot/gitea that referenced this pull request Nov 6, 2025
* giteaofficial/main:
  Remove padding override on `.ui .sha.label` (go-gitea#35864)
  fix(api/repo/contents): set the dates to now when not specified by the caller (go-gitea#35861)
  Remove `fix` Make targets (go-gitea#35868)
  Refactor ls-tree and git path related problems (go-gitea#35858)
  Fix pull description code label background (go-gitea#35865)
  Make ACME email optional (go-gitea#35849)
  Remove wrong code (go-gitea#35846)
  Fix Arch repo pacman.conf snippet (go-gitea#35825)
  Port away from `flake-utils` (go-gitea#35675)
  Update golangci-lint to v2.6.0 (go-gitea#35801)
  Add a doctor command to fix inconsistent run status (go-gitea#35840)
  Fix viewed files number is not right if not all files loaded (go-gitea#35821)
  Fix incorrect pull request counter (go-gitea#35819)
  Fix a number of `strictNullChecks`-related issues (go-gitea#35795)
  ignore .worktrees as a "special folder" (go-gitea#35835)
  upgrade go mail to 0.7.2 and fix the bug (go-gitea#35833)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. modifies/templates This PR modifies the template files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants