commit 503f76a6802c7963a0defa7685a1f49e4bb4a787
author Brett Johnson <Brett.Michael.Johnson@gmail.com> Thu Apr 10 16:30:38 2014 -0700
committer Brett Johnson <Brett.Michael.Johnson@gmail.com> Fri Apr 18 16:35:36 2014 -0700
tree ea948f835a126b58cf358a65d4d34b2208a98f87
parent 9677f2de0834184c9761c35e45c7c7d22d7f3e0e

Fix b/13673493: Better warning message if start path is not shared.

Currently, if a folder on a server is not shared, the adaptor generates
the warning:
"The path ... is not a valid path. The path does not exist or it is not
 a file or directory."

If a folder on a server is not shared, the adaptor has no idea if it
exists or not.  This change simply changes the error message slightly:
"The path ... is not a valid path. The path does not exist, or it is
 not shared, or it is not a file or directory."

This phrasing leaves open the possibility of supporting local file
systems.

Code Review: http://codereview.appspot.com/86630044

src/com/google/enterprise/adaptor/fs/FsAdaptor.java

diff --git a/src/com/google/enterprise/adaptor/fs/FsAdaptor.java b/src/com/google/enterprise/adaptor/fs/FsAdaptor.java
index 7d528bf..fc3fd4d 100644
--- a/src/com/google/enterprise/adaptor/fs/FsAdaptor.java
+++ b/src/com/google/enterprise/adaptor/fs/FsAdaptor.java
@@ -230,9 +230,10 @@
             "DFS link path of the form \\\\host\\ns\\link.");
       }
     }
-    if (!isSupportedPath(rootPath)) {
+    if (!delegate.isDirectory(rootPath)) {
       throw new IOException("The path " + rootPath + " is not a valid path. "
-          + "The path does not exist or it is not a file or directory.");
+          + "The path does not exist, or it is not a directory, or it is not "
+          + "shared.");
     }
 
     builtinPrefix = context.getConfig().getValue(CONFIG_BUILTIN_PREFIX);