cvs commit: src/usr.bin/tar fts.c fts.h

Peter Jeremy peter.jeremy at alcatel.com.au
Tue May 4 16:26:43 PDT 2004


On 2004-May-04 10:21:02 -0700, Tim Kientzle <kientzle at freebsd.org> wrote:
>kientzle    2004/05/04 10:21:02 PDT
>
>  FreeBSD src repository
>
>  Added files:
>    usr.bin/tar          fts.c fts.h 
>  Log:
>  Include local copies of fts.c and fts.h to be used ONLY on non-FreeBSD
>  systems.  The fts.h here is an exact copy of include/fts.h (except for
>  an initial explanatory comment and the revision tags, of course).  The
>  fts.c here is slightly modified from lib/libc/gen/fts.c so it can
>  compile correctly on non-FreeBSD systems.

Since a colleague of mine has just finished porting fts(3) to Tru64, I
was interested in the changes you needed to make.  Having looked, it
seems that your changes are primarily aimed at Linux compatibility
rather than general portability - it's unlikely that your fts.c will
be usable on anything other than Linux systems due to the '#ifdef LINUX'
near the top - if this was '#ifndef __FreeBSD__', the portability would
be enhanced.

Also, why wasn't this done as a repocopy?  This would have made it far
easier to see the changes.

-- 
Peter Jeremy


More information about the cvs-src mailing list