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

Use short options for zfs send command #185

Merged
merged 3 commits into from
Feb 13, 2023
Merged

Use short options for zfs send command #185

merged 3 commits into from
Feb 13, 2023

Conversation

bk
Copy link
Contributor

@bk bk commented Feb 5, 2023

This PR replaces long zfs send options (such as --large-block) with short options (such as -L). The aim is to support pulling backup data from older zfs versions such as OpenZFS 0.6.x.

I also fixed a few typos in user-facing messages.

@psy0rz
Copy link
Owner

psy0rz commented Feb 5, 2023

regression tests failed, can you check out whats going on? thanks :)

@bk
Copy link
Contributor Author

bk commented Feb 10, 2023

Sorry about that. The destroymissing test referenced one of the strings I had changed in #627fa87. Should be fixed now. All tests pass on my own system, at least.

@psy0rz
Copy link
Owner

psy0rz commented Feb 13, 2023

no need to be sorry, thanks :)

@coveralls
Copy link

coveralls commented Feb 13, 2023

Coverage Status

Coverage: 84.898%. Remained the same when pulling 3886301 on bk:master into 60840a4 on psy0rz:master.

@psy0rz psy0rz merged commit a078be3 into psy0rz:master Feb 13, 2023
This pull request was closed.
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.

3 participants