kwin commented on a change in pull request #7:
URL: 
https://github.com/apache/sling-org-apache-sling-starter/pull/7#discussion_r419602665



##########
File path: src/main/provisioning/repoinit.txt
##########
@@ -71,6 +71,13 @@
         allow  rep:write       on /apps/sling/install
     end
 
+    # content-package installer
+    create service user sling-package-install
+
+    set ACL for sling-package-install
+        allow  jcr:all     on  /

Review comment:
       After having a 2nd thought I think this should be fine as once we fixed 
https://issues.apache.org/jira/browse/SLING-9422 we definitely need to be 
install all packages (which might everywhere to the repo)




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to