* [PATCH] generic/062: filter redundant output by getfattr @ 2018-07-12 15:59 Zorro Lang 2018-07-13 6:39 ` Eryu Guan 0 siblings, 1 reply; 4+ messages in thread From: Zorro Lang @ 2018-07-12 15:59 UTC (permalink / raw) To: fstests 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@redhat.com> --- Hi, This's a bad news. Latest attr package attr-2.4.48 changed his behavior as above. And this's different with attr-2.4.47. With old attr: # setfattr -h -n user.name testfile # getfattr -d -n user.name testfile # file: testfile user.name With new attr: # setfattr -h -n user.name testfile # getfattr -d -n user.name testfile # file: testfile user.name="" Note: -d option is necessary This little difference will break golden image. So this's the problem. If you have better idea than this patch, please tell me. Thanks, Zorro tests/generic/062 | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/tests/generic/062 b/tests/generic/062 index df67960d..4fc2dc46 100755 --- a/tests/generic/062 +++ b/tests/generic/062 @@ -29,9 +29,13 @@ _cleanup() } trap "_cleanup; exit \$status" 0 1 2 3 15 +# When getfattr dump values of all extended attributes, it print empty attr +# as user.name before, but new getfattr print it as user.name="". For match +# the golden image, filter the redundant ="" at the end. getfattr() { - $GETFATTR_PROG --absolute-names -dh $@ 2>&1 | _filter_scratch + $GETFATTR_PROG --absolute-names -dh $@ 2>&1 | _filter_scratch | \ + sed -e 's/=\"\"//' } setfattr() -- 2.14.4 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] generic/062: filter redundant output by getfattr 2018-07-12 15:59 [PATCH] generic/062: filter redundant output by getfattr Zorro Lang @ 2018-07-13 6:39 ` Eryu Guan 2018-07-16 8:12 ` Zorro Lang 0 siblings, 1 reply; 4+ messages in thread From: Eryu Guan @ 2018-07-13 6:39 UTC (permalink / raw) To: Zorro Lang; +Cc: fstests On Thu, Jul 12, 2018 at 11:59:26PM +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@redhat.com> > --- > > Hi, > > This's a bad news. Latest attr package attr-2.4.48 changed his > behavior as above. And this's different with attr-2.4.47. > > With old attr: > # setfattr -h -n user.name testfile > # getfattr -d -n user.name testfile > # file: testfile > user.name > > With new attr: > # setfattr -h -n user.name testfile > # getfattr -d -n user.name testfile > # file: testfile > user.name="" > > Note: -d option is necessary > > This little difference will break golden image. So this's the problem. > If you have better idea than this patch, please tell me. > > Thanks, > Zorro > > > tests/generic/062 | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/tests/generic/062 b/tests/generic/062 > index df67960d..4fc2dc46 100755 > --- a/tests/generic/062 > +++ b/tests/generic/062 > @@ -29,9 +29,13 @@ _cleanup() > } > trap "_cleanup; exit \$status" 0 1 2 3 15 > > +# When getfattr dump values of all extended attributes, it print empty attr > +# as user.name before, but new getfattr print it as user.name="". For match > +# the golden image, filter the redundant ="" at the end. > getfattr() > { > - $GETFATTR_PROG --absolute-names -dh $@ 2>&1 | _filter_scratch > + $GETFATTR_PROG --absolute-names -dh $@ 2>&1 | _filter_scratch | \ > + sed -e 's/=\"\"//' > } This should really be done by a new helper function that hides all the details, so future tests could just take use of the helper. Thanks, Eryu > > setfattr() > -- > 2.14.4 > > -- > To unsubscribe from this list: send the line "unsubscribe fstests" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] generic/062: filter redundant output by getfattr 2018-07-13 6:39 ` Eryu Guan @ 2018-07-16 8:12 ` Zorro Lang 2018-07-17 3:02 ` Eryu Guan 0 siblings, 1 reply; 4+ messages in thread From: Zorro Lang @ 2018-07-16 8:12 UTC (permalink / raw) To: Eryu Guan; +Cc: fstests On Fri, Jul 13, 2018 at 02:39:25PM +0800, Eryu Guan wrote: > On Thu, Jul 12, 2018 at 11:59:26PM +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@redhat.com> > > --- > > > > Hi, > > > > This's a bad news. Latest attr package attr-2.4.48 changed his > > behavior as above. And this's different with attr-2.4.47. > > > > With old attr: > > # setfattr -h -n user.name testfile > > # getfattr -d -n user.name testfile > > # file: testfile > > user.name > > > > With new attr: > > # setfattr -h -n user.name testfile > > # getfattr -d -n user.name testfile > > # file: testfile > > user.name="" > > > > Note: -d option is necessary > > > > This little difference will break golden image. So this's the problem. > > If you have better idea than this patch, please tell me. > > > > Thanks, > > Zorro > > > > > > tests/generic/062 | 6 +++++- > > 1 file changed, 5 insertions(+), 1 deletion(-) > > > > diff --git a/tests/generic/062 b/tests/generic/062 > > index df67960d..4fc2dc46 100755 > > --- a/tests/generic/062 > > +++ b/tests/generic/062 > > @@ -29,9 +29,13 @@ _cleanup() > > } > > trap "_cleanup; exit \$status" 0 1 2 3 15 > > > > +# When getfattr dump values of all extended attributes, it print empty attr > > +# as user.name before, but new getfattr print it as user.name="". For match > > +# the golden image, filter the redundant ="" at the end. > > getfattr() > > { > > - $GETFATTR_PROG --absolute-names -dh $@ 2>&1 | _filter_scratch > > + $GETFATTR_PROG --absolute-names -dh $@ 2>&1 | _filter_scratch | \ > > + sed -e 's/=\"\"//' > > } > > This should really be done by a new helper function that hides all the > details, so future tests could just take use of the helper. OK. So how about a common function named _getfattr in common/attr, to instead of all $GETFATTR_PROG (or just g/062). Or a a common function named _filter_getfattr, to filter some $GETFATTR_PROG output if a case need it. _getfattr() { $GETFATTR_PROG $@ 2>&1 | _filter_testdir_and_scratch | \ sed -e 's/=\"\"//' } Or _filter_getfattr() { _filter_testdir_and_scratch | sed -e 's/=\"\"//' } Thanks, Zorro > > Thanks, > Eryu > > > > > setfattr() > > -- > > 2.14.4 > > > > -- > > To unsubscribe from this list: send the line "unsubscribe fstests" in > > the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- > To unsubscribe from this list: send the line "unsubscribe fstests" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] generic/062: filter redundant output by getfattr 2018-07-16 8:12 ` Zorro Lang @ 2018-07-17 3:02 ` Eryu Guan 0 siblings, 0 replies; 4+ messages in thread From: Eryu Guan @ 2018-07-17 3:02 UTC (permalink / raw) To: Zorro Lang; +Cc: fstests On Mon, Jul 16, 2018 at 04:12:43PM +0800, Zorro Lang wrote: > On Fri, Jul 13, 2018 at 02:39:25PM +0800, Eryu Guan wrote: > > On Thu, Jul 12, 2018 at 11:59:26PM +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@redhat.com> > > > --- > > > > > > Hi, > > > > > > This's a bad news. Latest attr package attr-2.4.48 changed his > > > behavior as above. And this's different with attr-2.4.47. > > > > > > With old attr: > > > # setfattr -h -n user.name testfile > > > # getfattr -d -n user.name testfile > > > # file: testfile > > > user.name > > > > > > With new attr: > > > # setfattr -h -n user.name testfile > > > # getfattr -d -n user.name testfile > > > # file: testfile > > > user.name="" > > > > > > Note: -d option is necessary > > > > > > This little difference will break golden image. So this's the problem. > > > If you have better idea than this patch, please tell me. > > > > > > Thanks, > > > Zorro > > > > > > > > > tests/generic/062 | 6 +++++- > > > 1 file changed, 5 insertions(+), 1 deletion(-) > > > > > > diff --git a/tests/generic/062 b/tests/generic/062 > > > index df67960d..4fc2dc46 100755 > > > --- a/tests/generic/062 > > > +++ b/tests/generic/062 > > > @@ -29,9 +29,13 @@ _cleanup() > > > } > > > trap "_cleanup; exit \$status" 0 1 2 3 15 > > > > > > +# When getfattr dump values of all extended attributes, it print empty attr > > > +# as user.name before, but new getfattr print it as user.name="". For match > > > +# the golden image, filter the redundant ="" at the end. > > > getfattr() > > > { > > > - $GETFATTR_PROG --absolute-names -dh $@ 2>&1 | _filter_scratch > > > + $GETFATTR_PROG --absolute-names -dh $@ 2>&1 | _filter_scratch | \ > > > + sed -e 's/=\"\"//' > > > } > > > > This should really be done by a new helper function that hides all the > > details, so future tests could just take use of the helper. > > OK. So how about a common function named _getfattr in common/attr, to instead > of all $GETFATTR_PROG (or just g/062). _getfattr() is fine, converting all existing bare getfattr calls would be good. Thanks! Eryu > > Or a a common function named _filter_getfattr, to filter some $GETFATTR_PROG > output if a case need it. > > _getfattr() > { > $GETFATTR_PROG $@ 2>&1 | _filter_testdir_and_scratch | \ > sed -e 's/=\"\"//' > } > > Or > > _filter_getfattr() > { > _filter_testdir_and_scratch | sed -e 's/=\"\"//' > } > > Thanks, > Zorro > > > > > Thanks, > > Eryu > > > > > > > > setfattr() > > > -- > > > 2.14.4 > > > > > > -- > > > To unsubscribe from this list: send the line "unsubscribe fstests" in > > > the body of a message to majordomo@vger.kernel.org > > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > -- > > To unsubscribe from this list: send the line "unsubscribe fstests" in > > the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-07-17 3:32 UTC | newest] Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-07-12 15:59 [PATCH] generic/062: filter redundant output by getfattr Zorro Lang 2018-07-13 6:39 ` Eryu Guan 2018-07-16 8:12 ` Zorro Lang 2018-07-17 3:02 ` Eryu Guan
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.