nielash
nielash
@ncw I see that `TestFixCase` is now failing in a different way, and I think I see why -- the `dirCache` is not getting properly flushed after the first move....
> I don't understand why that fix changes the directory cache? The `DirMove` method calls `f.dirCache.DirMove` to set up the move: https://github.com/rclone/rclone/blob/2abeda5961615c304348c4886a1d42f4ae30c876/backend/onedrive/onedrive.go#L1717 and then flushes the cache after the move:...
### TestApplyTransforms Yes, I think we should go ahead with https://github.com/nielash/rclone/commit/da487ba695d85af4751f799702f9b8e4fcf50bdc. I tested it with Dropbox just now and confirmed this is the issue: ``` check_test.go:575: Fs is incapable of...
@ncw I tried `operations.DirMove` and it worked like a charm. Seems like a cleaner solution for multiple issues here. Thoughts? https://github.com/nielash/rclone/commit/34c632e51d568d865e515a7efba6a9504466b516
> In general you should always use `cache.Get` instead of calling NewFs directly. That said, we are going to ditch that code path so this is just an FYI! Thanks,...
Update: the latest fixes have helped a lot. We're down to just a few outliers now. - `Linkbox`: see [comment here](https://github.com/rclone/rclone/issues/7591#:~:text=will%20bring%20one.-,Note%20that%20linkbox%20has%20some%20overrides%20in%20the%20integration%20test%20config,was%20deemed%20impossible%2C%20I%20suspect%20we%27ll%20reach%20the%20same%20conclusion%20here.,-That%20fix%20looks) - `mailru` (and `chunker` wrapping `mailru`): The error is:...
> Yes your fix seems like the correct one if you want to have a go :-) Here's my proposed fix: https://github.com/rclone/rclone/pull/7636
That seems to have solved it 🙂
> This is a fun idea :-) > > I pulled it locally and had a play. Thanks! 😃 > * Needs help - I expect `?` or `h` to...
> That shouldn't have imported it at all as no import formats are set... That is a very good point! > I guess you were using the `-M` flag here?...