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

Create fsck.zfs, splat, and zhack man pages. #1359

Closed
wants to merge 1 commit into from

Conversation

dajhorn
Copy link
Contributor

@dajhorn dajhorn commented Mar 17, 2013

And update the automake templates to install them.

The splat documentation is included here in anticipation of moving
the splat utility from zfsonlinux/spl to zfsonlinux/zfs.

Supplements #518

And update the automake templates to install them.

The splat documentation is included here in anticipation of moving
the splat utility from zfsonlinux/spl to zfsonlinux/zfs.

Supplements openzfs#518
@ryao
Copy link
Contributor

ryao commented Mar 17, 2013

Why would SPLAT move from the SPL to ZFS? It seems to belong in the SPL.

@behlendorf
Copy link
Contributor

@dajhorn Thank you, I'll read them over and get them merged.

@ryao As mentioned in previous issue, I think it now makes sense to start whittling away at the SPL over the next year and merge what's left in to the ZFS tree under the CDDL. That will vastly simplify the packaging and build process which is good for everyone. So eventually all that code will be in one package.

But since we're not quite there yet, for now the splat man page probably should live with the kernel module. @dajhorn do you have any objection to me moving splat.1 over to the SPL, we should be able to leave the CDDL header since we're just talking about a man page.

@dajhorn
Copy link
Contributor Author

dajhorn commented Mar 19, 2013

@behlendorf, nope, anywhere that fits.

@behlendorf
Copy link
Contributor

Merged as ac826e1

@behlendorf behlendorf closed this Mar 21, 2013
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