Btrfs devel - Nov 2011 - [PATCH] Btrfs-progs: mention libattr dependency in INSTALL file

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
---
 INSTALL |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/INSTALL b/INSTALL
index 6afbd90..11c388b 100644
--- a/INSTALL
+++ b/INSTALL
@@ -19,7 +19,8 @@ insmod btrfs.ko
 
 The Btrfs utility programs require libuuid to build.  This can be found
 in the e2fsprogs sources, and is usually available as libuuid or
-e2fsprogs-devel from various distros.
+e2fsprogs-devel from various distros.  The other dependency is libattr
+(libattr1-dev in Debian-based distros).
 
 Building the utilities is just make ; make install.  The programs go
 into /usr/local/bin.  The mains commands available are:
-- 
1.7.6.3

--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs"
in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Hallo, Ilya,

Du meintest am 02.11.11:
>  The Btrfs utility programs require libuuid to build.  This can be
> found  in the e2fsprogs sources, and is usually available as libuuid
> or e2fsprogs-devel from various distros.  The other dependency is
> libattr +(libattr1-dev in Debian-based distros).
Just for information: my btrfs-progs packet (integration-20111030) shows  
(via "ldd")

	libcom_err.so.2 => /lib/libcom_err.so.2 (0x401cc000)
	libcom_err.so.3 => /usr/X11R6/lib/libcom_err.so.3 (0x40061000)
	libc.so.6 => /lib/libc.so.6 (0x4003b000)
	libext2fs.so.2 => /lib/libext2fs.so.2 (0x40037000)
	libkrb5support.so.0 => /usr/lib/libkrb5support.so.0 (0x401cf000)
	/lib/ld-linux.so.2 (0x40000000)
	libpthread.so.0 => /lib/libpthread.so.0 (0x40037000)
	libresolv.so.2 => /lib/libresolv.so.2 (0x401d2000)
	libuuid.so.1 => /lib/libuuid.so.1 (0x40037000)

No "libattr". No special compiler option.

Slackware 13.37

Viele Gruesse!
Helmut
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs"
in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

On Wed, Nov 02, 2011 at 05:14:33PM +0200, Ilya Dryomov
wrote:
> Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
> ---
>  INSTALL |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
> 
> diff --git a/INSTALL b/INSTALL
> index 6afbd90..11c388b 100644
> --- a/INSTALL
> +++ b/INSTALL
> @@ -19,7 +19,8 @@ insmod btrfs.ko
>  
>  The Btrfs utility programs require libuuid to build.  This can be found
>  in the e2fsprogs sources, and is usually available as libuuid or
> -e2fsprogs-devel from various distros.
> +e2fsprogs-devel from various distros.  The other dependency is libattr
> +(libattr1-dev in Debian-based distros).
>  
>  Building the utilities is just make ; make install.  The programs go
>  into /usr/local/bin.  The mains commands available are:
> -- 
> 1.7.6.3
Scratch that, it seems it''s indeed not needed.  The thing is that every
time a set up a new test machine I have to install libattr1-dev, but I
never looked into it in detail.

So libattr1-dev is needed because mkfs includes attr/xattr.h to be able
to create FS from image with extended attributes.  I think it''s safe to
change attr/xattr.h include to sys/xattr.h and thus get rid of this
build dependency.  (attr/xattr.h seems to be there for compatibility
with SGI IRIX)

I built mkfs with sys/xattr.h and created FS from directory containing
files with xattrs, it worked.

Thanks,

		Ilya
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs"
in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

On Wed, 2 Nov 2011 20:52:28 +0000
Leonidas Spyropoulos <artafinde@gmail.com> wrote:
> This might interest you.
Thanks
> ---------- Forwarded message ----------
> From: Ilya Dryomov <idryomov@gmail.com>
> Date: Wed, Nov 2, 2011 at 8:42 PM
> Subject: Re: [PATCH] Btrfs-progs: mention libattr dependency in INSTALL
file
> To: linux-btrfs@vger.kernel.org
> Cc: Chris Mason <chris.mason@oracle.com>, Hugo Mills
<hugo@carfax.org.uk>
> 
> 
> On Wed, Nov 02, 2011 at 05:14:33PM +0200, Ilya Dryomov wrote:
> > Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
> > ---
> >  INSTALL |    3 ++-
> >  1 files changed, 2 insertions(+), 1 deletions(-)
> >
> > diff --git a/INSTALL b/INSTALL
> > index 6afbd90..11c388b 100644
> > --- a/INSTALL
> > +++ b/INSTALL
> > @@ -19,7 +19,8 @@ insmod btrfs.ko
> >
> >  The Btrfs utility programs require libuuid to build.  This can be
found
> >  in the e2fsprogs sources, and is usually available as libuuid or
> > -e2fsprogs-devel from various distros.
> > +e2fsprogs-devel from various distros.  The other dependency is
libattr
> > +(libattr1-dev in Debian-based distros).
> >
> >  Building the utilities is just make ; make install.  The programs go
> >  into /usr/local/bin.  The mains commands available are:
> > --
> > 1.7.6.3
I see. I did check the INSTALL file when I first had the error but it
didn''t mention any dependencies. Incidentally, I get libuuid from
util-linux.
> 
> Scratch that, it seems it''s indeed not needed.  The thing is that
every
> time a set up a new test machine I have to install libattr1-dev, but I
> never looked into it in detail.
> 
> So libattr1-dev is needed because mkfs includes attr/xattr.h to be able
> to create FS from image with extended attributes.  I think it''s
safe to
> change attr/xattr.h include to sys/xattr.h and thus get rid of this
> build dependency.  (attr/xattr.h seems to be there for compatibility
> with SGI IRIX)
> 
> I built mkfs with sys/xattr.h and created FS from directory containing
> files with xattrs, it worked.
It does work, I used it to format the root partition I''m booted into at
the moment. But as I said before, I don''t use any extended attributes.
It would be nice to have the choice and not be expected to install
libattr.

Thanks,
Andy
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs"
in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html