configure logic is incorrect for database detection

[kburtch]

configure logic is incorrect for database detection

[kburtch]

Yes, there is a bug in the current logic. I turning off PostgreSQL worked but turning off MySQL did not. "-z" is working correctly as far as I know...I don't use it to check for 1 or 0. I will check into this more in a few days when I have more time.

[qunying]

You current testing actually make HAS_MYSQL and HAS_POSTGRES have the right value when at the stage of configuring APQ.

At your checking in line 928 of configure, only when NO_MYSQL is not set will it try to detect HAS_MYSQL, otherwise it is always empty regardless the system has it or not. The same goes to PosgreSQL, so it is more logical to to use HAS_MYSQL or HAS_POSTGRES in the APQ configure. The logic is more clear IMHO.

[kburtch]

Checked and merged. Replaced some ! -z with -n. Added comments.

[kburtch]

I've added a new set of tests to verify the configure flags work correctly for the databases. Let me know if this issue can be closed or if more work is needed.

[kburtch]

closing issue after 10 days. no response from reporter.