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

[DOC] mkdir /blazingsql #93

Closed
gumdropsteve opened this issue Oct 18, 2019 · 1 comment
Closed

[DOC] mkdir /blazingsql #93

gumdropsteve opened this issue Oct 18, 2019 · 1 comment

Comments

@gumdropsteve
Copy link
Contributor

Report incorrect documentation

Location of incorrect documentation
https://github.com/BlazingDB/pyBlazing#install-using-conda

Describe the problems or issues found in the documentation
When installing with Conda, changing the first two commands from this:

mkdir /blazingsql # Make a blazingsql directory in root folder for Apache Calcite schema management. This will requirement will be removed soon.

chown <user_name> /blazingsql

to this:

sudo mkdir /blazingsql
sudo chmod 777 /blazingsql

resolved my comment in issue #72

Steps taken to verify documentation is incorrect
Installed in multiple Conda environments, with both blazingsql and blazingsql-nightly

Suggested fix for documentation
Add sudo before mkdir /blazingsql in the instructions to make the first line read:

sudo mkdir /blazingsql

Not sure changing the 2nd line is needed, or that 777 instead of <user_name> is a preferable option

@felipeblazing
Copy link
Contributor

We have made that directory no longer necessary. This has been fixed with PR PR-94. A new build should go out by the end of the day that will solve this issue for conda installs

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

No branches or pull requests

2 participants