5 Added variables:

    'const uint8_t qb_ver_major'    {qb_ver_major}
    'const uint8_t qb_ver_micro'    {qb_ver_micro}
    'const uint8_t qb_ver_minor'    {qb_ver_minor}
    'const char* const qb_ver_rest'    {qb_ver_rest}
    'const char* const qb_ver_str'    {qb_ver_str}

  1 Removed variable symbol not referenced by debug info:

    facilitynames

Removed item was never in the header files/meant to be public API. It's expected nothing in the client space uses it.

Works

Sorry, found out that the fix changes a precondition for a hardened run-time sanity check that is also included in the build (part of the same PR). Will need to do a respin.

(also forgot to mention the impact ... pacemaker won't compile with such libqb, but it should we the upcoming respin)

Regarding two previous comments, the reasoning was wrong, see https://bugzilla.redhat.com/show_bug.cgi?id=1487787#c12 that justifies the corosync build I've just added.

Re first comment, after some rethinking, it eventually becomes:

  2 Added variables:

    'const qb_version qb_ver'    {qb_ver}
    'const char* const qb_ver_str'    {qb_ver_str}

  1 Removed variable symbol not referenced by debug info:

    facilitynames

works for me

Works great! LGTM! =)