Skip to content

Fix duplicate maintainers on package pages (#427)#661

Open
m1rm wants to merge 1 commit into
archlinux:masterfrom
m1rm:fix/427-maintainer-doubled-on-specific-page
Open

Fix duplicate maintainers on package pages (#427)#661
m1rm wants to merge 1 commit into
archlinux:masterfrom
m1rm:fix/427-maintainer-doubled-on-specific-page

Conversation

@m1rm

@m1rm m1rm commented Apr 11, 2026

Copy link
Copy Markdown
Contributor

Fixes: #427

This pull requests appends .distinct() to the User queryset in Package.maintainers so each user is returned at most once, regardless of duplicate relation rows.

@christian-heusel

Copy link
Copy Markdown
Member

Whats an example package where this bug occurs? 😊

@m1rm

m1rm commented Apr 12, 2026

Copy link
Copy Markdown
Contributor Author

Whats an example package where this bug occurs? 😊

The bug still appears here: https://archlinux.org/packages/extra/any/python-qt.py.

I also tried reproducing it locally using the commands from the current README.md step no. 7 to load real prod data locally, but python-qt then was not found when running locally. So I assume it might be worth a thought providing some updated dumps (which would be a separate issue/todo ofc).

I also think the distinct is rather a "workaround" then a real fix. I assume we would have to check prod tables and maybe do an according migration to clean up data actually causing the bug in the first place (out of my reach however).

@m1rm

m1rm commented Jul 6, 2026

Copy link
Copy Markdown
Contributor Author

after I stopped overseeing that one sentence in the README, that I can import other data easily, I tried:

wget http://mirrors.kernel.org/archlinux/extra/os/any/extra.files.tar.gz && uv run ./manage.py reporead --filesonly any extra.files.tar.gz -> 404

wget http://mirrors.kernel.org/archlinux/extra/os/x86_64/extra.files.tar.gz && uv run ./manage.py reporead --filesonly x86_64 extra.files.tar.gz works but when I - for example - search for 7zip locally, it is not found.

sorry for closing and opening - I am switching platforms to much and had a "ctrl+ Enter" for newline, which is not needed in Github.

@m1rm m1rm closed this Jul 6, 2026
@m1rm m1rm reopened this Jul 6, 2026
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.

Maintainer doubled on specific package

2 participants