shallow-backup icon indicating copy to clipboard operation
shallow-backup copied to clipboard

Windows Compatibility

Open saadmanrafat opened this issue 7 years ago • 6 comments

I was wondering if this could be extended to work on Windows.

saadmanrafat avatar Oct 15 '18 05:10 saadmanrafat

Yeah, this isn't limited to just macOS, but it would require a bunch of refactoring and testing that just isn't a priority of mine since I don't use Windows. I'm definitely open to a PR, though.

alichtman avatar Oct 15 '18 05:10 alichtman

Implementation Plan

  • [ ] Use os.path.join obsessively, for all paths

Is this it?

alichtman avatar Oct 18 '18 22:10 alichtman

Yes but another issue is inquirer, which doesn't work on Windows. Other viable choices are prompt_toolkit and Pyinquirer. Each has it's pros and cons. But using any one of them could lead to losing some features the repo already supports like the List prompt.

saadmanrafat avatar Oct 18 '18 22:10 saadmanrafat

One possibility is forking this project and maintaining a Windows version separately. I don't have the bandwidth to manage that, but I'd be open to someone else maintaining it.

alichtman avatar Oct 18 '18 22:10 alichtman

Another possibility is try / except importing inquirer and if that fails, import some other library that's Windows compatible. This is probably the best solution, as it would avoid forking the project. I still don't use Windows, though, so I won't be fixing this.

alichtman avatar Apr 25 '19 10:04 alichtman

try except might work. I'll look into it.

saadmanrafat avatar Apr 26 '19 13:04 saadmanrafat