[GitHub] maven pull request #127: Fix concat_lines in mvn to deal with CRLF

2017-07-10 Thread grkvlt
GitHub user grkvlt opened a pull request: https://github.com/apache/maven/pull/127 Fix concat_lines in mvn to deal with CRLF Change from `tr` to `paste` to deal with CRLF line endings properly You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] maven issue #127: Fix concat_lines in mvn to deal with CRLF

2017-07-12 Thread grkvlt
Github user grkvlt commented on the issue: https://github.com/apache/maven/pull/127 Good point, I reverted to the original `tr` command, but stripping out both *CR* and *LF* now, which will have the desired effect on files with any set of line endings. --- If your project is set up

[GitHub] maven-integration-testing pull request #24: Add test for MNG-6255 jvm.config...

2017-07-18 Thread grkvlt
GitHub user grkvlt opened a pull request: https://github.com/apache/maven-integration-testing/pull/24 Add test for MNG-6255 jvm.config line endings Integration test for fix made in https://github.com/apache/maven/pull/127 You can merge this pull request into a Git repository

[GitHub] maven-integration-testing issue #24: Add test for MNG-6255 jvm.config line e...

2017-07-18 Thread grkvlt
Github user grkvlt commented on the issue: https://github.com/apache/maven-integration-testing/pull/24 Tested on OSX; passes when change from https://github.com/apache/maven/pull/127 are made to the `mvn` script. --- If your project is set up for it, you can reply to this email

[GitHub] maven-integration-testing issue #24: Add test for MNG-6255 jvm.config line e...

2017-07-26 Thread grkvlt
Github user grkvlt commented on the issue: https://github.com/apache/maven-integration-testing/pull/24 Removed *CR* test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] maven issue #127: [MNG-6255] Fix concat_lines in mvn to deal with CRLF

2017-07-19 Thread grkvlt
Github user grkvlt commented on the issue: https://github.com/apache/maven/pull/127 @Tunaki I will disable the *CR* test, then --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] maven issue #127: [MNG-6255] Fix concat_lines in mvn to deal with CRLF

2017-08-23 Thread grkvlt
Github user grkvlt commented on the issue: https://github.com/apache/maven/pull/127 ping? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so