You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Devin Avery (JIRA)" <ji...@apache.org> on 2017/03/07 12:41:38 UTC

[jira] [Created] (THRIFT-4109) Configure Script uses string comparison for versions

Devin Avery created THRIFT-4109:
-----------------------------------

             Summary: Configure Script uses string comparison for versions
                 Key: THRIFT-4109
                 URL: https://issues.apache.org/jira/browse/THRIFT-4109
             Project: Thrift
          Issue Type: Bug
            Reporter: Devin Avery
             Fix For: 0.10.0, 0.9.3


I am running the configure script as outlined here for MAC OSx -
http://thrift.apache.org/docs/install/os_x

I found that the configure script is doing string based comparison for
performing version verification which breaks as soon as a version changes from one digit to two. For example, homebrew installs ant version 1.10.x now (as of March 7th, 2017) which is clearly greater than 1.7 which thrift requires. However the thrift configure script fails to detect ant because it thinks the version is wrong.

This is because this line in the configure script (around line 18844 in
version 0.9.3) - 

{code}
ANT_VALID=expr $($ANT -version 2>/dev/null | sed -n 's/.*version \([0-9\.]*\).*/\1/p') \>= 1.7`
{code}
Notice how you are using the >= which is STRING comparison, not version
comparison. When 1.10.1 compares to 1.7 it is considered LESS than instead
of greater than.




--
This message was sent by Atlassian JIRA
(v6.3.15#6346)