diff options
| author | Aaron Gable <agable@chromium.org> | 2016-09-20 15:25:00 -0700 |
|---|---|---|
| committer | David Pursehouse <dpursehouse@collab.net> | 2016-09-21 05:41:09 +0000 |
| commit | 1a1b471a1ca713eb64d1f94909ca192043ea7822 (patch) | |
| tree | 4a492b9b204f15840f2595ec4a4ce5cae108046c | |
| parent | 4734437557d9915c3ed8f7f9ef9e3f3c090bf95c (diff) | |
Improve 'Add Patch Set' ACL documentation.
I found the previous documentation to be confusing: it basically
said "The absence of this permission does X; by the way, by default
this permission isn't absent even if it looks like it is." This
commit improves the documentation to note the default state first,
and to explain how to explicitly revoke the permission.
Change-Id: Id62d81f74042ecdb841c3432b563923abb1a3f46
(cherry picked from commit bf7f41a09a91b876bc17ada74d3fc4ca32a39d77)
| -rw-r--r-- | Documentation/access-control.txt | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/Documentation/access-control.txt b/Documentation/access-control.txt index eeb1b30c5f..2cc8c059c6 100644 --- a/Documentation/access-control.txt +++ b/Documentation/access-control.txt @@ -619,10 +619,10 @@ existing changes. Irrespective of this permission, change owners are always allowed to upload new patch sets for their changes. This permission needs to be set on `refs/for/*`. -The absence of this permission will prevent users from uploading a -patch set to a change they do not own. By default, this permission is granted to -`Registered Users` on `refs/for/*` allowing all registered users to upload a new -patch set to any change on that ref. +By default, this permission is granted to `Registered Users` on `refs/for/*`, +allowing all registered users to upload a new patch set to any change. Revoking +this permission (by granting it to no groups and setting the "Exclusive" flag) +will prevent users from uploading a patch set to a change they do not own. [[category_push_merge]] |