On Thu, Aug 23, 2018 at 10:41:24AM +0800, Zorro Lang wrote: > When getfattr dumps values of all extended attributes (-d option), > it doesn't print empty extended attributes. e.g: user.name. But > from attr-2.4.48 this behavior is changed, new getfattr prints > user.name="". > > The {=""} will break the golden image, so filter the redundant ="" > at the end if it has. > > Signed-off-by: Zorro Lang <zlang@xxxxxxxxxx> > --- > > Hi, > > Sorry I took long time to write this V2, due to it changed many > cases, I have to test them one by one. V1 patch refer to: > https://patchwork.kernel.org/patch/10521875/ > > V2 did below changes: > 1) Bring in a common function named _getfattr() > 2) Replace all $GETFATTR_PROG with _getfattr Thanks a lot for the work! > > Thanks, > Zorro > > common/attr | 11 +++++++++++ > tests/ext4/022 | 2 +- > tests/ext4/026 | 4 ++-- > tests/generic/020 | 8 ++++---- > tests/generic/037 | 2 +- > tests/generic/062 | 4 ++-- > tests/generic/066 | 4 ++-- > tests/generic/093 | 2 +- > tests/generic/097 | 2 +- > tests/generic/337 | 2 +- > tests/generic/364 | 4 ++-- This test and .. > tests/generic/403 | 2 +- > tests/generic/454 | 6 +++--- > tests/generic/489 | 2 +- > tests/overlay/011 | 2 +- > tests/overlay/026 | 4 ++-- > tests/overlay/038 | 4 ++-- > tests/overlay/041 | 4 ++-- > tests/overlay/046 | 6 +++--- > tests/overlay/056 | 2 +- > tests/overlay/060 | 4 ++-- this one requires sourcing common/attr before using _getfattr. I've fixed them on commit. Thanks, Eryu