On Mon, Mar 11, 2013 at 05:13:27PM +0100, Michal Privoznik wrote: > Currently, only three wrappers are being implemented: > virFileSetAttr for setting attributes > virFileGetAttr for querying attributes (note we need to call it twice, > first time to get length of attribute value, second to get actual value) > virFileRemoveAttr for removing attributes > --- > > diff to v2: > -drop multiple check for libattr > > src/libvirt_private.syms | 3 ++ > src/util/virfile.c | 106 +++++++++++++++++++++++++++++++++++++++++++++++ > src/util/virfile.h | 12 ++++++ > 3 files changed, 121 insertions(+) > > diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms > index 599b71e..f787a12 100644 > --- a/src/libvirt_private.syms > +++ b/src/libvirt_private.syms > @@ -1273,8 +1273,11 @@ virFileClose; > virFileDirectFdFlag; > virFileFclose; > virFileFdopen; > +virFileGetAttr; > virFileLoopDeviceAssociate; > +virFileRemoveAttr; > virFileRewrite; > +virFileSetAttr; > virFileTouch; > virFileUpdatePerm; > virFileWrapperFdClose; > diff --git a/src/util/virfile.c b/src/util/virfile.c > index 4a9fa81..aae7101 100644 > --- a/src/util/virfile.c > +++ b/src/util/virfile.c > @@ -37,6 +37,11 @@ > # include <sys/ioctl.h> > #endif > > +#ifdef HAVE_LIBATTR There is no such conditional - use WITH_ATTR > +# include <attr/xattr.h> > +# include <sys/types.h> > +#endif > + > #include "vircommand.h" > #include "configmake.h" > #include "viralloc.h" > @@ -644,3 +649,104 @@ int virFileLoopDeviceAssociate(const char *file, > } > > #endif /* __linux__ */ > + > +#ifdef HAVE_LIBATTR Again use WITH_ATTR Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :| -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list