[SSSD] [PATCH v3 3/4] NFSv4 client: add to build system

Nikolai Kondrashov Nikolai.Kondrashov at redhat.com
Tue Sep 2 12:21:51 UTC 2014


On 09/02/2014 02:57 PM, Lukas Slebodnik wrote:
> On (02/09/14 14:50), Nikolai Kondrashov wrote:
>> On 07/14/2014 07:30 PM, Jakub Hrozek wrote:
>>> On Fri, Jun 27, 2014 at 09:44:37AM +0300, Noam Meltzer wrote:
>>>> ---
>>>>   Makefile.am                 | 24 ++++++++++++++++++++++++
>>>>   configure.ac                | 10 ++++++++++
>>>>   src/conf_macros.m4          | 30 ++++++++++++++++++++++++++++++
>>>>   src/external/libnfsidmap.m4 | 17 +++++++++++++++++
>>>>   4 files changed, 81 insertions(+)
>>>>   create mode 100644 src/external/libnfsidmap.m4
>>>
>>> ACK
>>
>> This commit fails CI due to missing nfsidmap.h header:
>> http://sssd-ci.duckdns.org/logs/commit/b9/c8ce2bdd4045782c243605a1b999098bedcffc/23/summary.html
>>
>> A dependency should be added perhaps.
>>
> It is added in next patch
> "[SSSD] [PATCH v3 4/4] NFSv4 client: add to RPM spec"

Ah, yes, was too hasty assuming if it didn't build on Debian, then it was
still broken. Will send a Debian CI patch in a moment.

> I don't like it, but it was external contributor.
> I didn't want to be picky.

Sure.

Nick



More information about the sssd-devel mailing list