jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andrew Gaul <notificati...@github.com>
Subject Re: [jclouds/jclouds] Ensure that jclouds filesystem provider consistently uses / (#1024)
Date Wed, 26 Oct 2016 00:12:13 GMT
andrewgaul commented on this pull request.



>      * @return
     */
-   private static String normalize(String pathToBeNormalized) {
-      if (null != pathToBeNormalized && pathToBeNormalized.contains(BACK_SLASH))
{
-         if (!BACK_SLASH.equals(File.separator)) {
-            return pathToBeNormalized.replace(BACK_SLASH, File.separator);
-         }
+   private static String normalize(String path) {
+      if (null != path) {

Windows has other restricted characters:

http://stackoverflow.com/a/31976060/2800111

We should not strive to make the filesystem blobstore work as similar on Linux as Windows
by crippling the former.  Rather we should make the filesystem blobstore as closely match
real blobstores as the environment allows.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1024
Mime
View raw message