[PATCH] tests: Include linux/stat.h
raj.khem at gmail.com
Sun Aug 5 19:51:32 UTC 2018
On 8/5/18 8:37 AM, Dmitry V. Levin wrote:
> On Sun, Aug 05, 2018 at 08:14:34AM -0700, Khem Raj wrote:
>> On Sat, Aug 4, 2018 at 1:33 PM Dmitry V. Levin wrote:
>>> On Sat, Aug 04, 2018 at 01:20:10PM -0700, Khem Raj wrote:
>>>> Starting with glibc 2.28, sys/stat.h header will have the
>>> s/will have/has/ as glibc 2.28 has already been released.
>>> In fact, starting with glibc commit glibc-2.28~105.
>> thats right, I will change the wording.
>>>> definition for struct statx, in which case including linux/stat.h should be
>>>> avoided, in order to prevent a duplicate definition.
>>> Why linux/stat.h should be avoided in the test?
>> it complains otherwise of duplicate definitions
>>> Have you tried the latest HEAD from strace.git?
>> I have been using 4.23 release not latest master
> While it's fine to report against the latest release,
> it's advised to check the latest HEAD from strace.git.
>>> In particular, what do you think about commit v4.23-37-g3dbffb7?
>> Can you point to exact commit on master?
This patch when backported on top of latest release fixes the issue,
please ignore this patch
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 195 bytes
Desc: OpenPGP digital signature
More information about the Strace-devel