Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cli: Add --pretend option #63

Closed
wants to merge 1 commit into from

Conversation

greezybacon
Copy link
Collaborator

Along with an option parser. Requires Bash 4.0 or greater for associative arrays. My guess is that by now everybody is using it?

This really helped my diagnose bitpocket syncing issues where files were not synced properly. It displays everything on the output and creates the diff files in the .bitpocket/tmp folder, but it does not actually sync anything and it does not update the state files. This is better that the list command because it shows what would be synchronized by bitpocket in both directions without actually doing it.

(Actually, macOS hasn't made it to bash 4.0 yet)

@aggsol
Copy link

aggsol commented Aug 24, 2017

Bash 4.0 was released in 2009(!). I vote against any legacy support of older Bash versions.

Along with an option parser. Requires Bash 4.0 or greater for associative
arrays. My guess is that by now everybody is using it?
@greezybacon
Copy link
Collaborator Author

Ok, I rebased it so it works with bash v3. I'm trying to get bitpocket to work on macOS, and that what it uses ...

@greezybacon
Copy link
Collaborator Author

This has been merged into the master branch and released in https://github.com/sickill/bitpocket/releases/tag/v0.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants