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

Web interface reports different results from 06perms.txt for package Alien #187

Closed
neilb opened this issue Sep 14, 2015 · 2 comments
Closed
Labels

Comments

@neilb
Copy link
Collaborator

neilb commented Sep 14, 2015

Last week I asked you (ANDK) to change Alien's permission for ABERGMAN from m to f, which you did. I then transferred ownership to PLICEASE. When I looked at 06perms.txt there was no entry for ABERGMAN, as he asked me to drop his co-maint as well, which I had done.

But then PLICEASE mentioned that the web interface was telling him that ABERGMAN still had module-list permission ('m'). Here's what the web interface tells me right now:

screen shot 2015-09-14 at 22 53 45

Which is slightly odd, since it's saying ABERGMAN has modulelist, but on the other rows it's saying PLICEASE has ownership!

I just grabbed the latest 06perms.txt, and ran:

% grep '^Alien,' 06perms.txt
Alien,ABERGMAN,c
Alien,ETJ,c
Alien,JBERGER,c
Alien,NEILB,c
Alien,PLICEASE,f
Alien,ZMUGHAL,c

(I asked PLICEASE to try giving me co-maint, to test if he had ownership, which is why you see me having co-maint).

So 06perms is saying that ABERGMAN has co-maint but the web interface is saying he has 'm', and earlier today 06perms wasn't listing ABERGMAN at all for Alien, which is what I expected.

Cheers,
Neil

@andk
Copy link
Owner

andk commented Apr 25, 2016

Confirmed, the web interface is wrong, 06perms is right

@andk
Copy link
Owner

andk commented May 14, 2017

Thanks, charsbar! Deployed.

andk pushed a commit that referenced this issue Apr 21, 2018
This should fix a similar issue as #187. In this case, the current code removes modules someone who still has "deleted" entries in "mod" table has comaintainership for.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants