[ 
https://issues.apache.org/jira/browse/SLING-9084?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17054985#comment-17054985
 ] 

Bertrand Delacretaz commented on SLING-9084:
--------------------------------------------

Thank you very much for this contribution!

I have merged the parser PR and made [a few 
tweaks|https://github.com/apache/sling-org-apache-sling-repoinit-parser/commit/30c5e5cd1509737d4b4f6321f74eb1208cad7dbb]
 to simplify the syntax.

I think the "members" keyword is not really useful, with my changes the syntax 
is now

{code}
add user1,user2 to group grpA
remove user3,user5 from group grpB
{code}

Does that work for you [~nitigupt] or was there a specific reason to say "add 
members user1..." instead of just "add user1..." ?

As for the jcr-repoinit PR I'd like to have @angela review it, asked for that 
there.

> Add members to groups during repoinit
> -------------------------------------
>
>                 Key: SLING-9084
>                 URL: https://issues.apache.org/jira/browse/SLING-9084
>             Project: Sling
>          Issue Type: Improvement
>          Components: Repoinit
>    Affects Versions: Repoinit JCR 1.1.16
>            Reporter: Syama Varma 
>            Priority: Major
>          Time Spent: 50m
>  Remaining Estimate: 0h
>
> For a new functionality in AEM, I need to create a node under /conf on which 
> I need to give ACLs to certain groups. 
>  
> To give ACLs to the 2 groups mentioned above, during repoinit, the groups 
> need to be present during repoinit. Currently, these groups are created by 
> content package and group memberships are enforced using content package as 
> well. If we start to create the groups as well as set ACLs at repoinit, we 
> need to set group memberships among the new groups at repoinit as well.
>  
> This requires extending the repoinit grammar to add users/groups to other 
> groups.
>  



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to