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

Add the admin indexing permission #253

Open
neilb opened this issue Oct 21, 2017 · 3 comments

Comments

Projects
None yet
5 participants
@neilb
Copy link
Collaborator

commented Oct 21, 2017

Add the admin indexing permission, which will fall between first-come and co-maint:

If you have the admin permission:

  • Your uploads of that package will be considered for inclusion in the index.
  • You can grant co-maint to other users.
  • You cannot transfer your admin permission to other users.
  • You cannot grant admin to other users.
  • You can give up an admin permission, in which case you get/retain co-maint
  • It will appear in $CPAN/modules/06perms.txt, presumably with 'a' as the permission character.

The user with first-come on a package can grant admin to other users.

@charsbar

This comment has been minimized.

Copy link
Collaborator

commented Oct 22, 2017

IMO "admin" is rather confusing because we'll need to distinguish PAUSE admin and package admin in some situations.

And what's the real purpose of this addition? Inactive first-comes who don't respond to requests for co-maint from others are less likely to grant new admin permission to someone as well. Might it be better to discuss if we can allow an active co-maint to grant the same permission to someone?

@Grinnz

This comment has been minimized.

Copy link
Contributor

commented Nov 20, 2017

It's reasonably common for authors with first-come to want to designate someone else to manage comaints, but without giving up their own first-come permissions which carry essentially the master bit for that namespace. Changing comaints to be able to universally add and remove other comaints would be a big change to the current assumptions of the system, IMO.

@karenetheridge

This comment has been minimized.

Copy link
Contributor

commented Nov 21, 2017

@rjbs rjbs added the indexer label Apr 27, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.