You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by bd...@apache.org on 2021/02/16 15:13:04 UTC

[sling-site] branch master updated: Remove * is now supported

This is an automated email from the ASF dual-hosted git repository.

bdelacretaz pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/sling-site.git


The following commit(s) were added to refs/heads/master by this push:
     new f12ac75  Remove * is now supported
f12ac75 is described below

commit f12ac75a3ae92606f03d091737ea4b0149e307f3
Author: Bertrand Delacretaz <bd...@apache.org>
AuthorDate: Tue Feb 16 16:12:35 2021 +0100

    Remove * is now supported
---
 .../documentation/bundles/repository-initialization.md      | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/src/main/jbake/content/documentation/bundles/repository-initialization.md b/src/main/jbake/content/documentation/bundles/repository-initialization.md
index c000a83..0a0c9ae 100644
--- a/src/main/jbake/content/documentation/bundles/repository-initialization.md
+++ b/src/main/jbake/content/documentation/bundles/repository-initialization.md
@@ -50,7 +50,7 @@ The language is self-explaining but please refer to the actual test cases for de
         deny jcr:write for u-ser_2
         deny jcr:lockManagement for user1
 
-        # remove does not work so far, SLING-9090
+        # See note [1] below
         remove jcr:understand,some:other for u3
     end
 
@@ -76,7 +76,7 @@ The language is self-explaining but please refer to the actual test cases for de
     create path (nt:unstructured) /var
 
     set ACL for alice, bob,fred
-        # remove does not work so far, SLING-9090
+        # See note [1] below
         remove * on / 
         allow jcr:read on /content,/var
         deny jcr:write on /content/example.com
@@ -118,7 +118,7 @@ The language is self-explaining but please refer to the actual test cases for de
     # repository needs to support 'o.a.j.api.security.authorization.PrincipalAccessControlList'
     set principal ACL for alice,bob
 
-        # remove does not work so far, SLING-9090
+        # See note [1] below
         remove * on /libs,/apps
 
         allow jcr:read on /content,/var
@@ -196,8 +196,6 @@ The language is self-explaining but please refer to the actual test cases for de
     # o.a.s.repoinit.parser 1.5.2
     # and o.a.s.jcr.repoinit 1.1.22
     add user1,user2 to group grpA
-
-    # remove does not work so far, SLING-9090
     remove user3,user5 from group grpB
     
     # ACLs on user homes, requires
@@ -236,6 +234,11 @@ The language is self-explaining but please refer to the actual test cases for de
     end
 </pre>
 
+### Notes on the above examples
+
+* [1] Although the repoinit language includes a `remove` statement, it is **not** generally supported by the current version of the `o.a.s.jcr.repoinit` module. Only the `remove *`
+variant is supported starting with `o.a.s.jcr.repoinit V1.1.34`.
+
 ### Notes on Repository Initializer Config Files
 
 If the Repository Initializer is defined inside a **.config** file then according to the .config file