[GitHub] flink pull request #6102: [FLINK-9091][build] Dependency convergence run aga...

[twalthr <gi...@git.apache.org>]

Github user twalthr commented on a diff in the pull request:

    https://github.com/apache/flink/pull/6102#discussion_r199125903
  
    --- Diff: tools/check_dependency_convergence.sh ---
    @@ -0,0 +1,67 @@
    +#!/usr/bin/env bash
    +################################################################################
    +#  Licensed to the Apache Software Foundation (ASF) under one
    +#  or more contributor license agreements.  See the NOTICE file
    +#  distributed with this work for additional information
    +#  regarding copyright ownership.  The ASF licenses this file
    +#  to you under the Apache License, Version 2.0 (the
    +#  "License"); you may not use this file except in compliance
    +#  with the License.  You may obtain a copy of the License at
    +#
    +#      http://www.apache.org/licenses/LICENSE-2.0
    +#
    +#  Unless required by applicable law or agreed to in writing, software
    +#  distributed under the License is distributed on an "AS IS" BASIS,
    +#  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
    +#  See the License for the specific language governing permissions and
    +# limitations under the License.
    +################################################################################
    +
    +HERE="`dirname \"$0\"`"				# relative
    +HERE="`( cd \"$HERE\" && pwd )`" 	# absolutized and normalized
    +if [ -z "$HERE" ] ; then
    +	# error; for some reason, the path is not accessible
    +	# to the script (e.g. permissions re-evaled after suid)
    +	exit 1  # fail
    +fi
    +
    +FLINK_DIR=HERE
    +
    +if [[ $(basename ${HERE}) == "tools" ]] ; then
    +  FLINK_DIR="${HERE}/.."
    +fi
    +
    +FLINK_DIR="`( cd \"${FLINK_DIR}\" && pwd )`" 
    +
    +echo ${FLINK_DIR}
    +
    +# get list of all flink modules
    +# searches for directories containing a pom.xml file
    +# sorts the list alphabetically
    +# only accepts directories starting with "flink" to filter force-shading
    +modules=$(find -maxdepth 3 -name 'pom.xml' -printf '%h\n' | sort -u | grep "flink")
    --- End diff --
    
    Specify the path otherwise is not runnable in every environment.
    
    `printf` is also not available in MacOs:
    https://stackoverflow.com/questions/752818/find-lacks-the-option-printf-now-what


---