On Fri, Jan 10, 2003 at 01:23:45PM -0500, Green, Paul
wrote:> The following patch still applies cleanly to the current cvs copy of rsync.
Or did before the most recent Makefile.in changes. It's easy to merge
this one problem, though.
> Does anyone object to having these changes applied now, during the
> pre-release phase?
Here are my comments on the changes:
+ The Makefile.in changes look very safe and needed.
+ The install-sh change to the dsttmp value looks good.
? I have a question about the portability of the u_FOO -> uFOO_t
changes. The former is the BSD syntax for the unsigned FOO typedefs,
and the latter is what, POSIX? The changes work on Linux, at least.
Perhaps we should just make these changes and try it out on the
compile farm.
+ The inet_pton changes look right to make the code consistent. The
only possible glitch might be a system that has a prototype for
inet_pton() but not the library code -- if the prototype conflicts,
the compile would fail (there are ways to work around this, but let's
worry about that if we actually find some weird system with this
problem).
I'd be glad to check in the "+" changes now if Dave thinks now is
a good
time.
..wayne..