Merge branch 'refs/heads/t/sideband-cleanup'

started on 2016-07-03, cooking since 2016-07-16

para_server announces sideband as an optional feature, which is
pointless since sideband connections have become mandatory in
paraslash-0.5.x, so they are always used. This patch set removes
the feature negitiation during the initial handshake. The sideband
protocol remains and is not affected.

Overwiew of the handshake in v0.5.x:
	(SA) server announces sideband
	(CC) client fails connection if server did not announce sideband
	(CR) client requests sideband
	(SC) server fails connection if sideband was not requested
	already broken
		- client-0.4/server-0.5 (!CR, SC)
		- client-0.5/server-0.4 (!SA, CC)

Conversion plan:
	rm (CC): breaks nothing
	rm (SC): breaks nothing
	rm (CR): breaks cl-0.6/server-0.5 (!CR, SC)
	rm (SA): breaks cl-0.5/server-0.6 (!SA, CC)
	server fails connection if client requests sideband

In this series only the first two conversion steps are done as they
can be merged early without breaking anything.

* refs/heads/t/sideband-cleanup:
  server: Do not fail if client does not request sideband.
  client: No longer fail connection if sideband was not announced.

diff --combined error.h
index 337160c9a6c3d19c33c9192ae95ba7262eba8db6,3bfd8a96cf306bfa1c86930a7d364753dcc761fb..3fda5787774ce739754a58d74706c3081f1e9da5
--- 1/error.h
--- 2/error.h
+++ b/error.h
@@@ -103,7 -103,7 +103,7 @@@ extern const char **para_errlist[]
  
  
  #define AFH_RECV_ERRORS \
 -      PARA_ERROR(AFH_RECV_BAD_FILENAME, "invalid file name"), \
 +      PARA_ERROR(AFH_RECV_BAD_FILENAME, "no file name given"), \
  
  
  #define OGG_AFH_COMMON_ERRORS \
@@@ -306,7 -306,6 +306,6 @@@
        PARA_ERROR(SERVER_EOF, "connection closed by para_server"), \
        PARA_ERROR(SERVER_CMD_SUCCESS, "command terminated successfully"), \
        PARA_ERROR(SERVER_CMD_FAILURE, "command failed"), \
-       PARA_ERROR(INCOMPAT_FEAT, "client/server incompatibility"), \
  
  
  #define NET_ERRORS \