We probably don't need the comment since it's made clear by the DEPENDS but other than that this LGTM.
Best, Philip On 08/18/2015 10:22 PM, jackie.hu...@windriver.com wrote: > From: Joe Slater <jsla...@windriver.com> > > Patches added by this bbappend make us depend on the attr > package. > > Signed-off-by: Joe Slater <jsla...@windriver.com> > Signed-off-by: Jackie Huang <jackie.hu...@windriver.com> > --- > recipes-devtools/e2fsprogs/e2fsprogs_1.42.9.bbappend | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/recipes-devtools/e2fsprogs/e2fsprogs_1.42.9.bbappend > b/recipes-devtools/e2fsprogs/e2fsprogs_1.42.9.bbappend > index d8de9da..e65df12 100644 > --- a/recipes-devtools/e2fsprogs/e2fsprogs_1.42.9.bbappend > +++ b/recipes-devtools/e2fsprogs/e2fsprogs_1.42.9.bbappend > @@ -1,5 +1,8 @@ > FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:" > > +# These patches make us depend on attr. > +DEPENDS += "attr" > + > SRC_URI += " \ > file://misc-xattr-add-xattr-module-stub.patch \ > file://mke2fs.c-create_inode.c-copy-xattrs.patch \ > -- _______________________________________________ yocto mailing list yocto@yoctoproject.org https://lists.yoctoproject.org/listinfo/yocto