Re: [PATCH] btrfs-progs: cmds: Fix build for using NAME_MAX

[Qu Wenruo <quwenruo.btrfs@gmx.com>]

On 2021/7/24 下午4:23, Sidong Yang wrote:
> On Sat, Jul 24, 2021 at 03:50:25PM +0800, Qu Wenruo wrote:
>>
>>
>> On 2021/7/24 下午3:46, Sidong Yang wrote:
>>> There is some code that using NAME_MAX but it doesn't include header
>>> that is defined. This patch adds a line that includes linux/limits.h
>>> which defines NAME_MAX.
>>
>> I guess it's related to this issue?
>>
>> https://github.com/kdave/btrfs-progs/issues/386
>
> Yeah, It seems that there is no patch for this yet. So I sent this
> patch. Is this too minor patch?

No, I just want to mention you could add tag like the below, so David
can be more aware of the fix.
As when it merged into devel branch, github will automatically mention
it in the issue.

Issue: #386

And it looks good to me, so:

Reviewed-by: Qu Wenruo <wqu@suse.com>

Thanks,
Qu
>
> Thanks,
> Sidong
>
>>
>> Thanks,
>> Qu
>>
>>>
>>> Signed-off-by: Sidong Yang <realwakka@gmail.com>
>>> ---
>>>    cmds/filesystem-usage.c | 1 +
>>>    1 file changed, 1 insertion(+)
>>>
>>> diff --git a/cmds/filesystem-usage.c b/cmds/filesystem-usage.c
>>> index 50d8995e..2a76e29c 100644
>>> --- a/cmds/filesystem-usage.c
>>> +++ b/cmds/filesystem-usage.c
>>> @@ -24,6 +24,7 @@
>>>    #include <stdarg.h>
>>>    #include <getopt.h>
>>>    #include <fcntl.h>
>>> +#include <linux/limits.h>
>>>
>>>    #include "common/utils.h"
>>>    #include "kerncompat.h"
>>>