[Mesa-dev] [PATCH v4 18/18] configure.ac: Add required LLVM versions to the top

Emil Velikov emil.l.velikov at gmail.com
Wed Oct 19 13:28:37 UTC 2016


On 16 October 2016 at 01:20, Tobias Droste <tdroste at gmx.de> wrote:

> -        if test -n "${LLVM_VERSION_MAJOR}"; then
> +        if test "$LLVM_VERSION_MINOR" -lt 10; then
>              LLVM_VERSION_INT="${LLVM_VERSION_MAJOR}0${LLVM_VERSION_MINOR}"
>          else
> -            LLVM_VERSION_INT=`echo $LLVM_VERSION | sed -e 's/\([[0-9]]\)\.\([[0-9]]\)/\10\2/g'`
> +            LLVM_VERSION_INT="${LLVM_VERSION_MAJOR}${LLVM_VERSION_MINOR}"
Since this hunk does two distinctly separate things from the rest of
the patch, can you please split it out to a patch on top.

-Emil


More information about the mesa-dev mailing list