LocalDirRemoteFs::list_recursive is not portable on Windows

[oldoldman]

Hi guys,
Please have look at this function

cube/rust/cubestore/cubestore/src/remotefs/mod.rs

Line 406 in 91e3af6

pub async fn list_recursive(

On Windows , file returned by dir.next_entry() will have \ as its path seperator. But in the rest of the code, cubstored has unix path seperator / in mind. The consequence is LocalDirRemoteFs::list_recursive will return nothing on Windows.

[github-actions]

If you are interested in working on this issue, please go ahead and provide PR for that.
We'd be happy to review it and merge it.
If this is the first time you are contributing a Pull Request to Cube, please check our contribution guidelines.
You can also post any questions while contributing in the #contributors channel in the Cube Slack.

[igorlukanin]

Hey @oldoldman, thanks for reporting! Would you be open to contribute a fix to this one?

[oldoldman]

Hey @oldoldman, thanks for reporting! Would you be open to contribute a fix to this one?

I've proposed the PR which keeps pending for some days. Not know what the next.

[srh]

@oldoldman Thank you for the bug report and PR.