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.

Sign up for GitHub

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

Jump to bottom

bak_print_cmd fails when cfg['bak_open_exec'] missing #71

Open
ChanceNCounter opened this issue Oct 6, 2021 · 3 comments
Open

bak_print_cmd fails when cfg['bak_open_exec'] missing #71

ChanceNCounter opened this issue Oct 6, 2021 · 3 comments
Labels
bug Something isn't working

Comments

@ChanceNCounter
Copy link
Collaborator

Needs a try/except or a sanity check or something
@ChanceNCounter ChanceNCounter added the bug Something isn't working label Oct 6, 2021
@rexdivakar
Copy link

I can aslo look into this !

@ChanceNCounter
Copy link
Collaborator Author

It occurs to me that bak depends on an XDG-compliant, Unix-like system, which should always have diff. It's probably reasonable to just change the default config value to diff.

@ChanceNCounter
Copy link
Collaborator Author

Clearer: bak doesn't demand XDG compliance, but it falls back on the XDG spec when the XDG environment variables are absent.

Point is, not-Windowsness means diff should "always" exist.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ChanceNCounter @rexdivakar