README checks devel package dependency

Bug #1348714 reported by David Bennett on 2014-07-25
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Galera
Undecided
Unassigned

Bug Description

The README file mentions:

  * Check unit test library http://check.sourceforge.net/

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 http://check.sourceforge.net/

Similar to the Boost devel package build requirement.

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
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