swift, s3, googlecloudstorage: Don't delete the container/bucket if fs wasn't at root - fixes #172

googlecloudstorage/googlecloudstorage.go

@@ -407,11 +407,14 @@ func (f *Fs) Mkdir() error {
 	return err
 }
 
-// Rmdir deletes the bucket
+// Rmdir deletes the bucket if the fs is at the root
 //
 // Returns an error if it isn't empty: Error 409: The bucket you tried
 // to delete was not empty.
 func (f *Fs) Rmdir() error {
+	if f.root != "" {
+		return nil
+	}
 	return f.svc.Buckets.Delete(f.bucket).Do()
 }
 

s3/s3.go

@@ -487,10 +487,13 @@ func (f *Fs) Mkdir() error {
 	return err
 }
 
-// Rmdir deletes the bucket
+// Rmdir deletes the bucket if the fs is at the root
 //
 // Returns an error if it isn't empty
 func (f *Fs) Rmdir() error {
+	if f.root != "" {
+		return nil
+	}
 	req := s3.DeleteBucketInput{
 		Bucket: &f.bucket,
 	}

swift/swift.go

@@ -371,10 +371,13 @@ func (f *Fs) Mkdir() error {
 	return f.c.ContainerCreate(f.container, nil)
 }
 
-// Rmdir deletes the container
+// Rmdir deletes the container if the fs is at the root
 //
 // Returns an error if it isn't empty
 func (f *Fs) Rmdir() error {
+	if f.root != "" {
+		return nil
+	}
 	return f.c.ContainerDelete(f.container)
 }