eio: use futimesat() if futimes() is not available
Marc Lehmann
schmorp at schmorp.de
Sun Oct 16 02:53:52 CEST 2011
On Sat, Oct 15, 2011 at 05:36:18PM +0200, Ben Noordhuis <info at bnoordhuis.nl> wrote:
> On Sat, Oct 15, 2011 at 10:56, Marc Lehmann <schmorp at schmorp.de> wrote:
> > On Wed, Oct 12, 2011 at 11:06:12PM +0200, Ben Noordhuis <info at bnoordhuis.nl> wrote:
> >> Attached is a patchlet that makes libeio fall back to futimesat() if
> >> the platform doesn't support futimes(), like e.g. SunOS.
> >
> > Hi, thats sane, but libeio cvs already uses futimensat when available
> > (futimesat is obsolete) - can you check whether this works for you?
>
> Sorry, I should've mentioned that in my original email.
>
> - The patch is based against the current CVS HEAD.
>
> - SunOS doesn't have futimensat(). Newer flavours have futimens() but
> the systems I work with don't.
hmm, after researching this a bit, futimensat is at least in sunos 5.9,
5.10 and 5.11, which means the problem should be a non-issue for quite a
while now (5.9 is from 2002). I am not sure the added complexity is worth
it, given that sunos (or actually solaris, sunos is obsolete since the
early 90ies) has had this syscall for so many releases now.
maybe it's not being picked up, because the tets for it checks for posix
2008 availability. relaxing this check to enable more platforms would
potentially be more useful.
--
The choice of a Deliantra, the free code+content MORPG
-----==- _GNU_ http://www.deliantra.net
----==-- _ generation
---==---(_)__ __ ____ __ Marc Lehmann
--==---/ / _ \/ // /\ \/ / schmorp at schmorp.de
-=====/_/_//_/\_,_/ /_/\_\
More information about the libev
mailing list