andrewd18 / df-lnp-installer

An installation script for the Dwarf Fortress Lazy Newb Pack, for Linux.
Other
93 stars 35 forks source link

Fix comparison test when qmake returns "**Unknown**" #29

Closed hawson closed 11 years ago

hawson commented 11 years ago

When running df-lnp-installer.sh for the first time, I get this error:

[~/df-lnp-installer]$ ./df-lnp-installer.sh Checking for dependencies... ./df-lnp-installer.sh: line 107: [: Unknown: integer expression expected

Where should Dwarf Fortress be installed? [/home/jbecker/bin/Dwarf Fortress]: ^C [...]

This comes from the output of "qmake -query QT_VERSION", which can return the string "Unknown". This isn't valid for comparing with "-eq", since that requires numbers, and not strings.

andrewd18 commented 11 years ago

Change made as part of 0.4.0-dev branch. See 9a46dfad22048c8b5cba8ef4265cfe41ecb389a8