I have a better fix in mng-6195 that will work on solaris 10 too

On 29 March 2017 at 21:40, <[email protected]> wrote:

> [MNG-6191] mvn -f complains about illegal readlink option under macOS
>
>
> Project: http://git-wip-us.apache.org/repos/asf/maven/repo
> Commit: http://git-wip-us.apache.org/repos/asf/maven/commit/21373bd7
> Tree: http://git-wip-us.apache.org/repos/asf/maven/tree/21373bd7
> Diff: http://git-wip-us.apache.org/repos/asf/maven/diff/21373bd7
>
> Branch: refs/heads/MNG-6191
> Commit: 21373bd7e13e1e4244b5f062df3e2a75d52caf5a
> Parents: 00c82b7
> Author: Karl Heinz Marbaise <[email protected]>
> Authored: Sat Mar 25 11:31:29 2017 +0100
> Committer: Karl Heinz Marbaise <[email protected]>
> Committed: Wed Mar 29 22:39:59 2017 +0200
>
> ----------------------------------------------------------------------
>  apache-maven/src/bin/mvn | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> ----------------------------------------------------------------------
>
>
> http://git-wip-us.apache.org/repos/asf/maven/blob/21373bd7/
> apache-maven/src/bin/mvn
> ----------------------------------------------------------------------
> diff --git a/apache-maven/src/bin/mvn b/apache-maven/src/bin/mvn
> index df62f68..a447c4b 100755
> --- a/apache-maven/src/bin/mvn
> +++ b/apache-maven/src/bin/mvn
> @@ -142,7 +142,7 @@ find_file_argument_basedir() {
>    for arg in "$@"; do
>      if [ ${found_file_switch} -eq 1 ]; then
>        if [ -f "${arg}" ]; then
> -        basedir=$(dirname "$(readlink -f "${arg}")")
> +        basedir="$(cd "$(dirname "${arg}")" && pwd -P)"
>          if [ ! -d "${basedir}" ]; then
>            echo "Directory ${basedir} extracted from the -f/--file
> command-line argument ${arg} does not exist" >&2
>            exit 1
>
>

Reply via email to