You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "ASF GitHub Bot (Jira)" <ji...@apache.org> on 2023/01/05 11:34:00 UTC
[jira] [Commented] (MWRAPPER-81) mvnw does not remove return chars from jvm.config
[ https://issues.apache.org/jira/browse/MWRAPPER-81?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17654923#comment-17654923 ]
ASF GitHub Bot commented on MWRAPPER-81:
----------------------------------------
bmarwell commented on code in PR #67:
URL: https://github.com/apache/maven-wrapper/pull/67#discussion_r1062378797
##########
maven-wrapper-distribution/src/resources/mvnw:
##########
@@ -160,7 +160,7 @@ find_maven_basedir() {
# concatenates all lines of a file
concat_lines() {
if [ -f "$1" ]; then
- echo "$(tr -s '\n' ' ' < "$1")"
+ echo "$(tr -s '\r\n' ' ' < "$1")"
Review Comment:
Needs to be rebased on top of https://github.com/apache/maven-wrapper/pull/79 later (removed that unnecessary echo call).
> mvnw does not remove return chars from jvm.config
> -------------------------------------------------
>
> Key: MWRAPPER-81
> URL: https://issues.apache.org/jira/browse/MWRAPPER-81
> Project: Maven Wrapper
> Issue Type: Bug
> Components: Maven Wrapper Scripts
> Affects Versions: 3.1.1
> Reporter: Arlo Louis O'Keeffe
> Priority: Minor
>
> When the jvm.config contains return characters (\r) in addition to newlines (\n) e.g. on Windows the mvnw script does not remove those.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)