Remove support for "all" workarounds

This should never be used. We may add new workarounds in the future,
and they should only be enabled explicitly when needed.

2 files changed, 4 insertions, 6 deletions

diff --git a/ChangeLog b/ChangeLog
index 7d7381d..a1c11d6 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -8,7 +8,10 @@ Unreleased Changes
 * Dropped support for the *nodelaysrv* workaround. The same effect
   (enabling NODELAY on the server side *and* enabling X11 forwarding)
   can be achieved by explicitly passing `-o ForwardX11`
-
+* Removed support for `-o workaround=all`. Workarounds should always
+  enabled explicitly and only when needed. There is no point in always
+  enabling a potentially changing set of workarounds.
+  
 Release 2.9 (2017-04-17)
 ------------------------
 
diff --git a/sshfs.c b/sshfs.c
index a2f23b5..b926f6a 100644
--- a/sshfs.c
+++ b/sshfs.c
@@ -422,10 +422,6 @@ static struct fuse_opt workaround_opts[] = {
 	SSHFS_OPT("none",       truncate_workaround, 0),
 	SSHFS_OPT("none",       buflimit_workaround, 0),
 	SSHFS_OPT("none",       fstat_workaround, 0),
-	SSHFS_OPT("all",        rename_workaround, 1),
-	SSHFS_OPT("all",        truncate_workaround, 1),
-	SSHFS_OPT("all",        buflimit_workaround, 1),
-	SSHFS_OPT("all",        fstat_workaround, 1),
 	SSHFS_OPT("rename",     rename_workaround, 1),
 	SSHFS_OPT("norename",   rename_workaround, 0),
 	SSHFS_OPT("truncate",   truncate_workaround, 1),
@@ -3318,7 +3314,6 @@ static void usage(const char *progname)
 "                           cache if full (default: 5)\n"
 "    -o workaround=LIST     colon separated list of workarounds\n"
 "             none             no workarounds enabled\n"
-"             all              all workarounds enabled\n"
 "             [no]rename       fix renaming to existing file (default: off)\n"
 "             [no]truncate     fix truncate for old servers (default: off)\n"
 "             [no]buflimit     fix buffer fillup bug in server (default: on)\n"