README checks devel package dependency

Bug #1348714 reported by David Bennett
This bug affects 1 person
Affects Status Importance Assigned to Milestone

Bug Description

The README file mentions:

  * Check unit test library

As a build requirement, however the check.h header file is required for compilation:

  Checking for C library crypto... yes
  Checking for C header file check.h... no
  Error: check header file not found or not usable

Recommend changing the README line to:

  * Check unit test devel package

Similar to the Boost devel package build requirement.

Revision history for this message
Alex Yurchenko (ayurchen) wrote :

Not every distribution has check devel package (e.g. Ubuntu) as there is absolutely no point in a non-devel one. Hence stating "devel package" may be confusing to non-RHEL users. The whole point of check library is that check.h header file, cause it (the library) is not used for anything but compilation of other software projects. It is more a of a bug in RHEL that they provide two packages instead of one.

Changed in galera:
status: New → Opinion
Revision history for this message
David Bennett (dbpercona) wrote :

Is this not the case for Boost and openssl? If so, perhaps the check build requirement should be changed to:

Check unit testing library and headers...

In any case, I had the library installed per the build requirements and the build failed without the check.h header.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers