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/c8ce2bdd4045782c243605a1b999098...
>
> 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