anchela commented on a change in pull request #69:
URL: 
https://github.com/apache/sling-org-apache-sling-feature-cpconverter/pull/69#discussion_r608593275



##########
File path: 
src/test/resources/org/apache/sling/feature/cpconverter/handlers/META-INF/vault/privileges.xml
##########
@@ -17,5 +17,5 @@
 -->
 <privileges xmlns:sling="http://sling.apache.org/jcr/sling/1.0"; 
xmlns:jcr="http://www.jcp.org/jcr/1.0";>
    <privilege name="sling:replicate"/>
-   <privilege name="sling:test" abstract="true"><contains 
name="sling:replicate"/><contains name="jcr:read"/></privilege>
+   <privilege name="sling:test" abstract="true"><contains 
name="sling.replicate"/><contains name="jcr.read"/></privilege>

Review comment:
       @karlpauls , altered the test-42.txt (and the corresponding output) to 
have one aggregated privilege replaced by jcr:read => 
   
   {code}
   org.apache.sling.repoinit.parser.RepoInitParsingException: 
   Encountered "" at line 4, column 59.
   Was expecting one of:
       
        at 
org.apache.sling.repoinit.parser.impl.RepoInitParserService.parse(RepoInitParserService.java:48)
   [...]
   Caused by: org.apache.sling.repoinit.parser.impl.ParseException: Encountered 
"" at line 4, column 59.
   Was expecting one of:
       
        at 
org.apache.sling.repoinit.parser.impl.RepoInitParserImpl.generateParseException(RepoInitParserImpl.java:3843)
   {code}




-- 
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:
[email protected]


Reply via email to