acl: refactor error messages in copy-acl.c

Hi Jim, After adding support for all platforms, copy-acl.c contains 7 times the error message regarding the source file, and 11 times the error message regarding the destination file. This begs for refactoring. Doing this refactoring ahead of the other changes will make these changes simpler. OK?

Re: [PATCH] DragonFly BSD support - reworked

On Sun, Jun 01, 2008 at 11:22:28PM +0200, Bruno Haible wrote: > > File offset is wrong after fseek: 17. > > FAIL: test-fflush > > That may be better in the new tarball. Yepp, works fine now. > > test-lseek.c:49: assertion failed > > Abort trap (core dumped) > > FAIL: test-lseek.sh > > This is c

acl: rework platform support macro

Hi Jim, Here's the first proposed part: rework acl.m4 - so that it detects all known ACL flavours (except native Woe32), - for maintainability: put the tests one after an other. Currently the test for POSIX-draft-like platform is buried inside an AC_SEARCH_LIBS that tests for Solaris s

Re: [PATCH] DragonFly BSD support - reworked

VOROSKOI Andras wrote: > diff -aur testdir-stdioext/gllib/stdio-impl.h > testdir-stdioext-dfly/gllib/stdio-impl.h > --- testdir-stdioext/gllib/stdio-impl.h 2008-05-01 16:38:47 +0200 > +++ testdir-stdioext-dfly/gllib/stdio-impl.h2008-06-01 22:52:47 +0200 > @@ -35,7 +35,7 @@ >

acl: new tests

Hi Jim, Now I've got support for ACL for these platforms: Linux, FreeBSD, MacOS X, Tru64, IRIX (POSIX like API), Solaris, probably also Cygwin (Solaris API), HP-UX, AIX. For the start, I added the unit tests. It will certainly break on some platforms temporarily, but this is harmless sinc

Re: [PATCH] DragonFly BSD support - reworked

On Sun, Jun 01, 2008 at 06:43:19PM +0200, Bruno Haible wrote: > The first hunk (file freadahead.c) is correct when looking at the DragonFlyBSD > I applied that. Thanks. > The second hunk (file freading.c) appears to be a nop to me; why should I > apply > this? Well, skip that. testdir-stdioext

Re: [PATCH] DragonFly BSD support - reworked

VOROSKOI Andras wrote: > I've updated the original patch a bit. I've reworked the #ifdef > __DragonFly__ parts to avoid duplicates. > > About the third hunk: this new patch does not touch that part. The first hunk (file freadahead.c) is correct when looking at the DragonFlyBSD CVS http://www.dr

Re: [PATCH] DragonFly BSD support - reworked

Hi, I've updated the original patch a bit. I've reworked the #ifdef __DragonFly__ parts to avoid duplicates. About the third hunk: this new patch does not touch that part. Let me know if you still got problems with it. PS: please cc me, i'm off-list -- voroskoi diff --git a/lib/freadahead.c b

Re: system(), Solaris, and the 'execute' module

Hi Eric, > | It seems that your problem is that you want execute a script via system() > | and at the same time force a particular shell, namely the $CONFIG_SHELL > found > | at configure time? > > Something along those lines - POSIX doesn't require m4 to use system(2), > merely that it treat th