frankosterfeld / qtkeychain

Platform-independent Qt API for storing passwords securely.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Use BUILD_SHARED_LIBS instead of QTKEYCHAIN_STATIC?

KitsuneRal opened this issue · comments

Is there any particular reason why QTKEYCHAIN_STATIC is used in order to switch between static/shared library instead of BUILD_SHARED_LIBS? Is it just to build a shared library by default?

That seems to be the cleaner way indeed. One might keep QTKEYCHAIN_STATIC around as "alias" to not break people's builds although that somehow defeats the purpose of the cleanup...

I think both can be supported for some transition perioud, with QTKEYCHAIN_STATIC only respected if (!DEFINED BUILD_SHARED_LIBS) (i.e., BUILD_SHARED_LIBS is literally unset, not just 0 or false).

commented

Fixed in #204