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

improve installation from source steps #630

Closed
wants to merge 2 commits into from

Conversation

Projects
None yet
3 participants
@danielb2
Copy link

commented Mar 16, 2018

some steps were missing

danielb2 added some commits Mar 16, 2018

@danielb2

This comment has been minimized.

Copy link
Author

commented Mar 17, 2018

Seems it should actually be using this documentation as a reference: https://openzfsonosx.org/wiki/Install

but I haven't tried this yet. If someone can confirm I should link to this, I'll update the PR

@lundman

This comment has been minimized.

Copy link
Contributor

commented Mar 18, 2018

Hmm yes, I threw in some helpful hints into the readme when we first started, but since then the wiki has much more details, perhaps even too much. The compiling instructions in readme should probably be retired, unless others have opinions...

@lundman lundman force-pushed the openzfsonosx:master branch from 69e481c to 0de42f0 Apr 5, 2018

@lundman lundman force-pushed the openzfsonosx:master branch from 6917ec8 to be5efa7 Jan 29, 2019

JMoVS added a commit that referenced this pull request Feb 22, 2019

Remove install directions as commented in PR 630
As suggested by lundman in #630 (comment) - installation instructions should probably be removed

JMoVS added a commit to openzfsonosx/spl that referenced this pull request Feb 22, 2019

Remove install instructions from Readme
As outlined here openzfsonosx/zfs#630 (comment) and proposed in openzfsonosx/zfs#688 - removes the installation instructions from the Readme

JMoVS added a commit that referenced this pull request Feb 25, 2019

Remove install directions as commented in PR 630
As suggested by lundman in #630 (comment) - installation instructions should probably be removed
@JMoVS

This comment has been minimized.

Copy link
Contributor

commented Feb 25, 2019

@danielb2 Thank you for your pull request. Are you still interested/willing to update your PR?

@JMoVS JMoVS closed this Mar 4, 2019

JMoVS added a commit to openzfsonosx/spl that referenced this pull request Mar 4, 2019

Remove install instructions from Readme
As outlined here openzfsonosx/zfs#630 (comment) and proposed in openzfsonosx/zfs#688 - removes the installation instructions from the Readme
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.