On Fri, 2019-01-18 at 14:03 +0000, Ulf Hermann wrote:
> I think you should also adapt tests/Makefile.am to use our own elf.h
> in 
> this case. See https://codereview.qt-project.org/#/c/187812/25 for
> my 
> solution to this.

Yes, it should indeed.
I used a slightly different solution though.
It relies on the default include flags already including the srcdirs.
Does that work for your use case too? (See revised patch attached.)

I looked at the features.h tweak also in your patch.
But I am not comfortable including something like that.
It feels like it needs a different test (whether we are actually on a
POSIX system or not). And I am not sure it really should define uid_t,
gid_t, mode_t, and pid_t. Those normally don't come from features.h
(they would come from sys/types.h).

Cheers,

Mark