You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by GitBox <gi...@apache.org> on 2019/12/20 17:14:37 UTC

[GitHub] [sling-org-apache-sling-installer-console] kwin opened a new pull request #1: SLING-8897 dedicated console for serializing configurations

kwin opened a new pull request #1: SLING-8897 dedicated console for serializing configurations
URL: https://github.com/apache/sling-org-apache-sling-installer-console/pull/1
 
 
   Update to parent 35

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [sling-org-apache-sling-installer-console] kwin commented on issue #1: SLING-8897 dedicated console for serializing configurations

Posted by GitBox <gi...@apache.org>.
kwin commented on issue #1: SLING-8897 dedicated console for serializing configurations
URL: https://github.com/apache/sling-org-apache-sling-installer-console/pull/1#issuecomment-568021000
 
 
   > We should avoid a dependency to Sling API as the installer (and therefore the console) is usable without Sling
   
   Got it, any preference how to escape the parameters then?

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [sling-org-apache-sling-installer-console] kwin merged pull request #1: SLING-8897 dedicated console for serializing configurations

Posted by GitBox <gi...@apache.org>.
kwin merged pull request #1: SLING-8897 dedicated console for serializing configurations
URL: https://github.com/apache/sling-org-apache-sling-installer-console/pull/1
 
 
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [sling-org-apache-sling-installer-console] cziegeler commented on issue #1: SLING-8897 dedicated console for serializing configurations

Posted by GitBox <gi...@apache.org>.
cziegeler commented on issue #1: SLING-8897 dedicated console for serializing configurations
URL: https://github.com/apache/sling-org-apache-sling-installer-console/pull/1#issuecomment-568168977
 
 
   No real preference, as long as  we don't create an additional dependency. As the encoding is pretty simple we might just copy the method we need into the class

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [sling-org-apache-sling-installer-console] kwin edited a comment on issue #1: SLING-8897 dedicated console for serializing configurations

Posted by GitBox <gi...@apache.org>.
kwin edited a comment on issue #1: SLING-8897 dedicated console for serializing configurations
URL: https://github.com/apache/sling-org-apache-sling-installer-console/pull/1#issuecomment-568021000
 
 
   > We should avoid a dependency to Sling API as the installer (and therefore the console) is usable without Sling
   
   Got it, any preference how to escape the parameters then? Is relying on commons-lang3 (https://commons.apache.org/proper/commons-lang/apidocs/org/apache/commons/lang3/StringEscapeUtils.html) ok for installer-console?

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services