We were leaving doc/config.log and src/or/micro-revision.i in
place during a make clean. Fix that.

Also update the changes file to contain a note on which bug was
fixed by this.

Chris Ball authored 14 years ago and Nick Mathewson committed 14 years ago

0 or 1 should only be passed as the sole argument. Warn for now, reject
in 0.2.3.x.

Chris Ball authored 14 years ago and Nick Mathewson committed 14 years ago

("Application request when we're believed to be offline." ->
 "Application request when we haven't used client functionality lately.")

We already had the country code ?? indicating an unknown country, so all we
needed to do to make unknown countries excludable was to make the ?? code
discoverable.

It's okay to get (say) a SocksPort line in the torrc, and then a
SocksPort on the command line to override it, and then a SocksPort via
a controller to override *that*.  But if there are two occurrences of
SocksPort in the torrc, or on the command line, or in a single SETCONF
command, then the user is likely confused.  Our old code would not
help unconfuse the user, but would instead silently ignore all but
the last occurrence.

This patch changes the behavior so that if the some option is passed
more than once to any torrc, command line, or SETCONF (each of which
coincidentally corresponds to a call to config_assign()), and the
option is not a type that allows multiple occurrences (LINELIST or
LINELIST_X), then we can warn the user.

This closes trac entry 1384.

This fixes the last suggestion of bug #1195.

Linus Nordberg authored 14 years ago and Nick Mathewson committed 14 years ago

* doc/Makefile.am: Change $(VAR:MOD) to ${VAR:MOD} -- make(1) on
NetBSD substitutes '$(:x)' to 'x' rather than the empty string.  This
bites us in doc/ when configured with `--disable-asciidoc'.  Curly
braces should work in all implementations of make(1) but this patch
changes only the places where we use the VAR:MOD expansion.

The tinytest* files went into src/test/.

Also add a changes/ file.