[
https://issues.apache.org/jira/browse/SLING-10274?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17315483#comment-17315483
]
Carsten Ziegeler commented on SLING-10274:
------------------------------------------
[~angela] Just to be sure before SLING-10121 tests were failing on windows - I
personally consider \n platform independent as it is the same character on all
platforms
> Consistently use platform independent line separator
> ----------------------------------------------------
>
> Key: SLING-10274
> URL: https://issues.apache.org/jira/browse/SLING-10274
> Project: Sling
> Issue Type: Bug
> Components: Content-Package to Feature Model Converter
> Reporter: Angela Schreiber
> Priority: Minor
>
> [~kpauls], as discussed offline SLING-10268 will introduce platform
> independent line separators for all generated repo-init statements. However,
> the converter code base seems to have a difficult history with line
> separators:
> - originally they seemed to be mostly platform-independent but tests were not
> - test got fixed with
> https://github.com/apache/sling-org-apache-sling-feature-cpconverter/commit/6c2ef9929c3e12acf4b7b104871816a6e3e15f4e
> - then with SLING-10121 platform independent separators got replaced by \n
> https://github.com/apache/sling-org-apache-sling-feature-cpconverter/commit/86b412dffc2c9c5654745f5ab1946d758b11004a
> IMHO going with platform independent separators would be better.... so, i
> would suggest to revert tests back and fix any potential remaining usages of
> \n in the code base. wdyt?
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
