Re: [PATCH 3/5] Add _require_chown as needed

[Eric Sandeen <sandeen@redhat.com>]

On 8/27/26 10:14 AM, Zorro Lang wrote:
> On Wed, Aug 26, 2026 at 02:56:42PM -0500, Eric Sandeen wrote:
>> On 8/26/26 12:52 PM, Zorro Lang wrote:
>>> On Mon, Aug 24, 2026 at 07:11:10PM -0500, Eric Sandeen wrote:
>>>> Signed-off-by: Eric Sandeen <sandeen@redhat.com>
>>>> ---
>>> I doubt there's any filesystem that supports `chmod` but not `chown`.
>>>
>>> Reviewed-by: Zorro Lang <zlang@kernel.org>
>>
>> Yep, I'm not sure why we have both helpers, tbh ...
>>
>> hey, you asked for ful _requirel annotation so .... ;)
> 
> Haha, I just wanted to add the same _require_ to patches that were
> copy-pasted from the same template. I wasn't planning on updating all
> cases across the fstests. Generally, for test cases that are already
> merged, we usually don't add extra _require checks unless there's an
> obvious failure. Anyway, since you've already gone through and updated
> everything, the current changes look good to me:)
Ok, I will leave it up to you whether you want this series, or the
trivial targeted patch I sent first, which is the only one addressing
an observed problem. :)

-Eric