[ 
https://issues.apache.org/jira/browse/MWRAPPER-88?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17655084#comment-17655084
 ] 

ASF GitHub Bot commented on MWRAPPER-88:
----------------------------------------

bmarwell commented on PR #79:
URL: https://github.com/apache/maven-wrapper/pull/79#issuecomment-1372564795

   Needs a slight revert:
   https://stackoverflow.com/a/53116875/1549977
   




> mvnw is not POSIX compatible
> ----------------------------
>
>                 Key: MWRAPPER-88
>                 URL: https://issues.apache.org/jira/browse/MWRAPPER-88
>             Project: Maven Wrapper
>          Issue Type: Bug
>          Components: Maven Wrapper Scripts
>    Affects Versions: 3.1.1
>            Reporter: Benjamin Marwell
>            Assignee: Benjamin Marwell
>            Priority: Major
>              Labels: posix
>
> * POSIX scripts should not use bash-specific features
>  * the location of sh is NOT /bin/sh. Instead use /usr/bin/env sh.
>  * Fix suggestions from shellcheck 
> (https://github.com/apache/maven-wrapper/pull/74)



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to