On Sat, Aug 22, 2015 at 2:44 AM, Andre McCurdy <armccu...@gmail.com> wrote:
> On Fri, Aug 21, 2015 at 4:28 PM, Andrea Adami <andrea.ad...@gmail.com>
> wrote:
> > After commit 24fde4d do_compile fails:
> >
> > | mkfs.jffs2.c:70:21: fatal error: sys/acl.h: No such file or directory
> > | #include <sys/acl.h>
> >
> > Adding acl to the list of dependencies fixes the build.
>
> Unconditionally enabling xattr and acl support is OK for the native
> build, but for the target both should probably be conditional on their
> respective DISTRO_FEATURES.
>
I can agree, acl is one distro feature but atm it is not used by nodistro
and poky.
So the patch unconditionally -DWITHOUT_XATTR seems wrong...this should be
done only if acl is used.
Andrea
>
>
> > Signed-off-by: Andrea Adami <andrea.ad...@gmail.com>
> > ---
> > meta/recipes-devtools/mtd/mtd-utils_git.bb | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/meta/recipes-devtools/mtd/mtd-utils_git.bb
> b/meta/recipes-devtools/mtd/mtd-utils_git.bb
> > index 8d4892a..72ce9ed 100644
> > --- a/meta/recipes-devtools/mtd/mtd-utils_git.bb
> > +++ b/meta/recipes-devtools/mtd/mtd-utils_git.bb
> > @@ -5,7 +5,7 @@ LICENSE = "GPLv2+"
> > LIC_FILES_CHKSUM = "file://COPYING;md5=0636e73ff0215e8d672dc4c32c317bb3
> \
> >
> file://include/common.h;beginline=1;endline=17;md5=ba05b07912a44ea2bf81ce409380049c"
> >
> > -DEPENDS = "zlib lzo e2fsprogs util-linux"
> > +DEPENDS = "zlib lzo e2fsprogs util-linux acl"
> >
> > PV = "1.5.1+git${SRCPV}"
> >
> > --
> > 1.9.1
> >
> > --
> > _______________________________________________
> > Openembedded-core mailing list
> > Openembedded-core@lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
--
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core
