Bug #34394 for File-ExtAttr: Test suite should skip on filesystems with no xattr support when run non-interactively

Mon Mar 24 09:26:17 2008 i386x [...] gmx.de - Ticket created

Subject:

cannot build on ubuntu

trying install with Show quoted text

cpan> install File::ExtAttr

I got errors failed: Operation not supported - perhaps the extended attribute's name needs a "user." prefix? at /home/user/.cpan/build/File-ExtAttr-1.07/blib/lib/File/ExtAttr.pm line 266. Perl v5.8.8 built for i486-linux-gnu-thread-multi Linux ubuntu 2.6.22-14-generic

Sun Mar 30 12:21:26 2008 richdawe [...] cpan.org - Correspondence added

From:

richdawe [...] cpan.org

On Mon Mar 24 09:26:17 2008, eszett wrote: Show quoted text

> trying install with

> cpan> install File::ExtAttr

> I got errors > failed: Operation not supported - perhaps the extended attribute's name > needs a "user." prefix? at > /home/user/.cpan/build/File-ExtAttr-1.07/blib/lib/File/ExtAttr.pm line

266. Show quoted text

> > Perl v5.8.8 built for i486-linux-gnu-thread-multi > Linux ubuntu 2.6.22-14-generic

This isn't the build failing. This is the test suite failing. The test suite currently assumes that it can set extended attributes on the filesystem. This is a bug in the test suite -- it should check whether it can create extended attributes and SKIP the tests if it can't. Perhaps there is a way of getting "cpan install" to skip the test suite? Another option is to mount with the user_xattr option. You would need to mount the filesystem containing /home/user with this option. You can either edit /etc/fstab to make that change permanently, or make it temporarily by remounting with -o user_xattr (e.g.: mount -o remount,user_xattr /home/user).

Sun Mar 30 12:21:28 2008 The RT System itself - Status changed from 'new' to 'open'

Sun Mar 30 14:17:49 2008 i386x [...] gmx.de - Correspondence added

From:

i386x [...] gmx.de

On Sun Mar 30 12:21:26 2008, RICHDAWE wrote: Show quoted text

> Another option is to mount with the user_xattr option.

Thanks, it works! P.S. where can I ask a question about File::ExtAttr? Forum?

Sun Mar 30 14:40:00 2008 richdawe [...] cpan.org - Correspondence added

From:

richdawe [...] cpan.org

On Sun Mar 30 14:17:49 2008, eszett wrote: Show quoted text

> On Sun Mar 30 12:21:26 2008, RICHDAWE wrote:

> > Another option is to mount with the user_xattr option.

> > Thanks, it works! > > > P.S. where can I ask a question about File::ExtAttr? Forum?

There's an file-extattr-users mailing list hosted at Sourceforge: http://sourceforge.net/mail/?group_id=153116

Sat Mar 07 04:35:21 2009 richdawe [...] cpan.org - Subject changed from 'cannot build on ubuntu' to 'Test suite should skip on filesystems with no xattr support'

Sat Mar 07 04:35:44 2009 richdawe [...] cpan.org - Subject changed from 'Test suite should skip on filesystems with no xattr support' to 'Test suite should skip on filesystems with no xattr support when run non-interactively'

Sat Mar 07 04:39:10 2009 richdawe [...] cpan.org - Correspondence added

The test suite relies on the filesystem being mounted with user_xattr (on Linux) or some other option to enable xattr support. If it's not enabled, then all the tests will fail. I get a lot of failure messages from the CPAN Testers, who are running the test suite non-interactively. Most of their systems are not set up correctly for the test suite. There is no point all the tests failing in that case. Rather than failing all the tests, skip them in non-interactive mode when the filesystem does not support xattrs. By "filesystem" I mean the filesystem on which the tests are running.

Sat Mar 07 05:25:02 2009 richdawe [...] cpan.org - Taken

Sat Mar 07 05:25:19 2009 richdawe [...] cpan.org - Correspondence added

This is fixed for Linux in CVS.

Sun Mar 08 12:10:17 2009 richdawe [...] cpan.org - Correspondence added

On Sat Mar 07 05:25:19 2009, RICHDAWE wrote: Show quoted text

> This is fixed for Linux in CVS.

Makefile.PL needs to run setfattr with LANG=C, so that the error messages are in English. Otherwise it won't detect that the filesystem is not mounted with user_xattr.

Fri Jan 28 18:30:16 2011 SILASMONK [...] cpan.org - Correspondence added

On Sun Mar 08 12:10:17 2009, RICHDAWE wrote: Show quoted text

> On Sat Mar 07 05:25:19 2009, RICHDAWE wrote:

> > This is fixed for Linux in CVS.

> > Makefile.PL needs to run setfattr with LANG=C, so that the error > messages are in English. Otherwise it won't detect that the filesystem > is not mounted with user_xattr.

There has been a long standing bug in Debian to get this module packaged. http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=453323 I would be interested in packaging this if he test error was fixed. I suspect that you could use Sys::FileSystem to work out if the file system has the required properties. Another possibility might be to extend the loop back file system to support extended attributes.

Sun Aug 30 02:13:34 2015 SREZIC [...] cpan.org - Correspondence added

On 2009-03-08 12:10:17, RICHDAWE wrote: Show quoted text

> On Sat Mar 07 05:25:19 2009, RICHDAWE wrote:

> > This is fixed for Linux in CVS.

> > Makefile.PL needs to run setfattr with LANG=C, so that the error > messages are in English. Otherwise it won't detect that the filesystem > is not mounted with user_xattr.

It's probably better to set LANG=C (or even better LC_ALL=C) in the setfattr call in the Makefile.PL. Also I don't think that an existing setfattr is necessary to have the test suite passed.