Re: [PATCH 1/1] mtd-utils: add xattr PACKAGECONFIG and fix acl dependency

[Patrick Ohly <patrick.ohly@intel.com>]

On Wed, 2015-08-26 at 12:02 -0700, Andre McCurdy wrote:
> On Wed, Aug 26, 2015 at 9:10 AM, Patrick Ohly <patrick.ohly@intel.com> wrote:
> > The unconditional removal of -DWITHOUT_XATTR accidentally introduced a
> > compile-time dependency on "acl", because "sys/acl.h" gets
> > included. This caused random compile failures.
> >
> > To fix this, we introduce a proper PACKAGECONFIG for the "xattr"
> > support, with the distro's "xattr" feature determining the
> > default. Setting the define must be done with custom code because
> > PACKAGECONFIG can only influence configure parameters.
> >
> > The "acl" distro feature is not checked because although enabling
> > "xattr" support now triggers a build of acl, nothing from it will not
> > get included in the resulting target image.
> 
> Typo?

Yes, the "not" needs to be removed. Can whoever merges the patch fix
that or should I send a revised version?

-- 
Best Regards, Patrick Ohly

The content of this message is my personal opinion only and although
I am an employee of Intel, the statements I make here in no way
represent Intel's position on the issue, nor am I authorized to speak
on behalf of Intel on this matter.