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

Recognize receive|recv output for -u detection #135

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

saper
Copy link

@saper saper commented Oct 29, 2019

zfs help output on FreeBSD 11.2-STABLE:

receive|recv [-vnsFu] <filesystem|volume|snapshot>
receive|recv [-vnsFu] [-o origin=<snapshot>] [-d | -e] <filesystem>
receive|recv -A <filesystem|volume>

we should detect -u in the same way we try it for -x

zfs help output on FreeBSD 11.2-STABLE:

	receive|recv [-vnsFu] <filesystem|volume|snapshot>
	receive|recv [-vnsFu] [-o origin=<snapshot>] [-d | -e] <filesystem>
	receive|recv -A <filesystem|volume>

we should detect -u in the same way we try it for -x
@ppbrown
Copy link
Member

ppbrown commented Mar 1, 2020

Howdy,

I vaguel recall modifying zrep detection related to this.
Please let me know whether the latest version addresses your issue or not?

@saper
Copy link
Author

saper commented Mar 4, 2020

A brief look at the changes since b95bd99 does not indicate that this got fixed, but I'll check this.

@mahush
Copy link

mahush commented Jun 25, 2021

Howdy,

I vaguel recall modifying zrep detection related to this.
Please let me know whether the latest version addresses your issue or not?

Hi @ppbrown,

i am a freebsd user (version 11.4) and hit this problem in zrep v1.8.0 too.

thanks

@ppbrown
Copy link
Member

ppbrown commented Jun 27, 2021

Hi mahush,
there have been a bunch of commits since the official 1.8.0 release.
Could you please try the latest in git and see if you still have the problem?

( https://github.com/bolthole/zrep/raw/master/zrep )

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.

None yet

3 participants