rm5248 commented on pull request #81:
URL: https://github.com/apache/logging-log4cxx/pull/81#issuecomment-991703645


   > I thought that the purpose of the testcase is to test both possiblities of 
line continuations (on Win and *nix), as the properties are named accordingly 
(the extra crlf in the properties name)
   
   There's one line in the properties file per test-case.  So it does test 
`\r\n` on *nix.  There's one line in the properties file per test-case defined.
   
   My proposal is to simply create a new 
file(propertiestestcase-crlf.properties or something like that) that only has 
one line in it(apart from the standard Apache header):
   ```
   propertiestestcase.crlf1=continued\^M
    value
   ```
   That way, we can set the .gitattributes file to set 
propertiestestcase-crlf.properties to be `eol=crlf`:
   ```
   src/main/resources/propertiestestcase-crlf.properties eol=crlf
   ```


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to