[SSSD] [PATCH] BUILD: Fix cmocka

Stephen Gallagher sgallagh at redhat.com
Mon Mar 11 17:08:50 UTC 2013


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Mon 11 Mar 2013 01:03:52 PM EDT, Lukas Slebodnik wrote:
> On (11/03/13 12:38), Stephen Gallagher wrote:
>> -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
>> 
>> We were not properly detecting that cmocka was unavailable. It
>> was expecting an empty value and getting "no" instead. This
>> patch corrects the expectation, so we will now skip building and
>> running cmocka tests on platforms that do not have it available.
>> 
>> Also, the common_cmocka.h header was missing from the
>> distribution tarball, causing 'make distcheck' to fail.
>> 
>> 
>> 
>> 
>> 
>> Note: the nss cmocka test is consistently segfaulting for me
>> during 'make distcheck' on F18 x86_64. I haven't had time to look
>> into why, yet. It works fine in normal 'make check', so I suspect
>> there's likely something going on with permissions (since 'make
>> check' sets $libdir as read-only). But that's just a guess.
> 
> Another solution to proper detection of libcmocka is attached in
> my patch. I am not very familiar with autotools, so I am not sure
> which solution is more suitable.
> 


Both are probably fine. I'm reattaching my patch though, as I
accidentally sent the earlier one I had made that didn't have the
Makefile.am fixes as well.

I'm personally slightly in favor of setting the explicit default and
testing for it (the way my patch does it) just because explicit values
are easier to spot later if we're trying to track down misbehavior.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.13 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iEYEARECAAYFAlE+D6IACgkQeiVVYja6o6PARwCePMq8NGvGmk1N286WyO3YDA5M
lc4AnjFk/U7BtpK4LdjxHgPBtBIF+szX
=Iyau
-----END PGP SIGNATURE-----
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-BUILD-Fix-cmocka-detection.patch
Type: text/x-patch
Size: 1756 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20130311/2156d2d6/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-BUILD-Fix-cmocka-detection.patch.sig
Type: application/pgp-signature
Size: 72 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20130311/2156d2d6/attachment.sig>


More information about the sssd-devel mailing list