README checks devel package dependency

Bug #1348714 reported by David Bennett
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Galera
Opinion
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.

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  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.