- Use popt to better effect for the two options that can be both
  set and unset by the user.
- Even more whitespace cleanup.

- Simplified the whole-file option handling.
- Fixed some whitespace issues.

- Simplified the whole-file option handling.
- Cleaned up some whitespace issues.

- Fixed lots of line-indentation problems, including a really huge section
  of non-batch-mode code that was indented at the same level as its
  surrounding "if" statement.
- Moved the heavily-used am_* flags into global extern vars instead of
  externing them again and again in individual functions.
- Got rid of line-ending whitespace.

Cleanup: Added header files to dependancies.

Typo corrections.
Thanks to Paul Slootman <paul@debian.org>

Removing --csum-length option documentation since there is
no such option.

Apply patch from Steve Bonds to repair this test.

Patch from Roderick Schertler (roderick at argon.org) to ignore
non-POSIX directory mode bits that are used by AIX.

Remove RedHat-specific .spec files in favor of the .spec files
maintained under packaging/lsb.

The sender no longer removes the duplicate names from the file list -- we
let the receiver do that, since they are the one that requests names and
we want to be sure the sender has any name the receiver may ask for.

Back out one of the changes in log.c revision 1.62, and always send errors
to the client after multiplexing is enabled.

Patch from jw schultz to reword "link" to "connection" in a couple of
spots.

Update packaging spec files per patch submitted by Horst von Brand.
In the future, we need to be sure to keep *.spec and *.spec.tmpl
identical...the release macro produces *.spec from *.spec.tmpl.

Restored changes accidentally backed out by Dave Dykstra in previous revision.

Apply fix from Horst von Brand. See comments in rsync.spec.

preparing for release of 2.5.6

Add header for 2.5.6 release.

Remove the Cygwin msleep(100) before the generator kills the receiver,
because it caused the testsuite/unsafe-links test to hang.

Change the default of --modify-window back to 0 on Cygwin.

Ack!  I had accidentally ifdefed out the kill from the generator to the
receiver process for every platform except Cygwin.

Change so the delay before generator signals receiver is only done on Cygwin.

Added a TODO item about temporary file names bumping up against the
maximum name length.  (I have an unfinished patch that will address
this).

Move the sleep to workaround the default modify-window of 1 on Cygwin to
the beginning of "checkit" rather than the end of "hands_setup" because
sometimes files are modified just before checkit is called and the copy
finishes within one second so they're considered to be the same time.
I don't think this would be a problem in real life, so just change the
test.

Insert a 100ms sleep just before sending the USR2 signal to the
child receiver process to prevent some hangs on Cygwin.  Anthony
Heading discovered the workaround first and suggested 30ms, and
Greger Cronquist had better luck with 100ms.

Remove the "Connection reset by peer" from TODO

Prevent the "Connection reset by peer" messages often seen from Cygwin.
Result of a lot of discussion over the last year and a half.  Based on
a patch from Randy O'Meara, cleaned up a bit by Max Bowsher.

Update date on man page.

Update rsyncd.conf documentation to be right for rsync server mode over a
remote shell.

Change erroneous references to a --config-file option to the correct --config
option.

Change news item about handling of text mode in files to just permitting
any of the standard line termination styles.

Got rid of recent O_TEXT* and O_BINARY* changes.

Got rid of O_TEXT_STR and added code to strip '\r' from the end of the
lines we read.

Added back the O_BINARY #ifdef.

Got rid of O_TEXT_STR change.

File I/O already handles '\r', so we can remove the O_TEXT flags.

Remove a couple items I know are no longer needed.

Better fix for infinite recursion; don't return from exit_cleanup
unless the nesting is already pretty deep, because there are normal
cases where exit_cleanup is nested shallowly.  Patch from Marc Espie,
posted by Brian Poole.

Open config files in text mode when O_TEXT is defined.  This helps on
Cygwin when the config files are on a filesystem that is mounted in
binary mode.  Patch from Ville Herva.

Tests that use hands_setup to make a test file directory and right
afterward make a copy were failing on cygwin because the default
--modify-windows now 1 on Cygwin.  Adding a 2 second sleep at the end of
hands_setup so that hopefully the tests will succeed because the copy
will be made more than 1 second away from the original.

Don't try to run daemon tests as "nobody".

Getting rid of recent "chown" since we decided to forego trying to run
the daemon tests as "nobody".

Change default of --modify-window on Cygwin from 2 to 1 because that's all
that's needed on FAT filesystems.  NTFS filesystems can do with a window of
0, but it shouldn't hurt because it's highly unlikely that any given file
will be modified within one second of the time that rsync last copied it.

Restore trailing newline character at the end of the file.

Mentioned the fact that config.sub and config.guess got updated.

Updated config.sub and config.guess to latest revision.

Mention test-case fixes.

Moved the chown from the download test to the upload test.

Chown the $TO and $FROM directories to $RSYNCD_UID:$RSYNCD_GID (if we
can, and don't fail if we can't).

Chown the $TO and $FROM directories to $RSYNCD_UID:$RSYNCD_GID (if we
can, and don't fail if we can't).

Put the UID and GID that rsyncd will run as into variables.

Separated a `cat "..."` command out from inside a double-quoted string
so that we don't run afoul of some shells quoting quirks.  (As Brian
Poole suggested.)

Change the name on the --modify-window default for Cygwin to Lapo Luchini.

Change version to working version 2.5.6pre3cvs

Change version to 2.5.6pre2

Declare preserve_perms for latest syscall.o.

Ignore errors from chmod when --preserve-perms/-p/-a are not set.
Gnu cp behaves the same way.

Added a couple missing NEWS items.

Re-activate the piece of code that creates intervening directories
when --relative-paths is used.  The code was accidentally skipped starting
in CVS version 1.32 of receiver.c.  Noticed by Craig Barratt.

Update description of hosts allow for IPv6.  From Bert Vermeulen.

Make the default for --modify-window be 2 on cygwin.

Backed out Paul Green's IRIX patch since it didn't seem to be a part of
the Makefile fix and I like the old EXE syntax better.

Save the value of the test for getaddrinfo defines in the config cache.

Also need to include lib/getnameinfo when the getaddrinfo defines don't exist.

Oops, lib/getaddrinfo wasn't pulled in when the system doesn't define
AI_PASSIVE.

Only look for a system getaddrinfo/getnameinfo if AI_PASSIVE is defined
by system headers.  This fixes compile errors on Irix 6.5.

Fix bug that causes messages like
    rsync: stack overflow in function match_address
on openbsd.  Patch from Brian Poole <raj@cerias.purdue.edu>.

Add unsafe-byname tests with symlink destinations ending in '..'.

Optimized unsafe_symlink() to avoid malloc/free calls.

Added "extern" to io_{read,write}_phase line (they were being
multiply defined).

Got rid of trailing whitespace and tweaked a few things that might
possibly be affecting the IRIX build (but probably not).

Define DBL_EPSILON if it doesn't exist.

Trying out Paul Green's IRIX patch to see if it fixes the syntax
error in the Makefile.

Fixed the bug in clean_flist() where it did not get rid of all duplicate
names if there were more than 2 identical names in a row.

Change rsync help for -a to show that it is equivalent to -rlptgoD.
I've had to go to the man page many times for that information and I
finally got sick of it.

Prevent infinite recursion in exit_cleanup().  Patch from Sviatoslav Sviridov.

Make unsafe_symlink() take const args so that we don't get any
compiler warnings when calling it with a const char *.

Add news item about fix of --copy-unsafe-links

Fix bug in --copy-unsafe that made it totally broken, and re-enable
the tests that tested it.  As far as I can tell, it was always broken
since the day I put it in years ago.  In my investigation into this I
was unable to figure out what in the world I was thinking back then,
to introduce a global variable with the wrong information in it rather
than using a parameter that was already available in readlink_stat
function.  That still bothers me a bit but I decided to stop worrying
about it.

Don't use the return value from sprintf because it doesn't work on Sunos4.

Now that the 2.5.6pre1 snapshot has been made, change the version to
2.5.6pre2cvs.

Change version to 2.5.6pre1

Changed the alloca warning message.

Use the old kludge of using malloc() if alloca() is missing.

Reduced the severity of the warning about missing alloca to a warning, and
changed the test to the common form of prefixing an x to both sides of
an equivalence because it didn't even notice a problem on the cray when
it was instead using test -n.
Also noticed an error in the test for ANSI c, doing a "$xac..." = xno"
when it should have been "x$ac..." = xno, so I fixed that too.

Oops, had the sense of the test for the existence of alloca() backwards.

Make configure bomb if the included popt is needed but alloca is not
available, as apparently is the case on Cray UNICOS.  The AC_FUNC_ALLOCA
autoconf documentation talks about having an included alloca.c and
periodically calling alloca(0) to garbage collect when C_ALLOCA is defined,
but I don't know where to get the code or if there's anybody that cares
enough about the UNICOS port for it to be worth the trouble.

Cast the return from alloca to work better on UNICOS.

Cast the return of alloca to remove a fatal error on Cray UNICOS.

Yet another try at getting this to skip on cygwin.

Hopefully this version will skip correctly on cygwin.

Try a different tact to get cygwin to skip this test.

The inet_pton() man page says we need to pass in a pointer to a
struct in_addr, which means passing &sin.sin_addr instead of
&sin.sin_addr.s_addr.  Also changed the AF_INET6 version to pass
&sin6.sin6_addr.  Hopefully this will fix UNICOS and not break
anyone else.

Cast poptGetOptArg() to remove a compiler warning.

+ The prototype for isc_net_pton() should have been inet_pton().
+ Define IN_LOOPBACKNET if it is not already defined.

Only refer to S_ISVTX if S_ISVTX is defined.

+ Fixed a comment that referred to isc_net_pton() instead of inet_pton().
+ Only prototype inet_pton6() if INET6 is defined.

Don't use '#' in the dsttmp filename.

Paul Green's changes to add $(EXEEXT) and $(CPPFLAGS) as appropriate.

The call to test_skipped if makepath failed didn't work, presumably because
makepath is builtin and there's some problem with it causing /bin/sh to
exit on cygwin.  Parens around the call to makepath should help.