Holoviews: Update installation docs for windows users

Created on 19 Apr 2018  路  6Comments  路  Source: holoviz/holoviews

In the installation guide we have:

image

One user on windows has reported that pip install 'holoviews[all]' does not work while pip install holoviews[all] does. I don't really want to think about shell quoting across operating systems but won't the latter work for all systems? In what situation are the quoted versions necessary?

@jbednar @ceball Any thoughts on this?

docs

Most helpful comment

Sounds like double quotes will consistently work across all platforms I know about (bash, tcsh, windows), so I guess the simplest thing is for us to use those and not mention anything about quoting.

All 6 comments

Brackets are used by some UNIX shells (e.g. tcsh) for their own purposes, leading to confusing and obscure errors:

tcsh> pip install holoviews[all]
pip: No match.

I was actually surprised to see that they are not used by bash, which is by far more popular than tcsh, so if the quotes cause problems on Windows, I guess the right thing to do is to take them out of the suggested commands, and then mention that they may need quoting on some systems.

Quoting should be fine on windows but it'd be " not ' (in the regular cmd.exe; in powershell it's probably more like what you're used to).

So is the recommendation to update the docs to use double quotes? @jbednar Is that also ok for tsch?

Edit: @jakirkham I may have accidentally pinged you when I meant to ping @jbednar . Oops!

No worries. :)

Sounds like double quotes will consistently work across all platforms I know about (bash, tcsh, windows), so I guess the simplest thing is for us to use those and not mention anything about quoting.

This will be addressed once the PR referenced above is merged.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

Forander picture Forander  路  6Comments

ahuang11 picture ahuang11  路  3Comments

obust picture obust  路  5Comments

CurtLH picture CurtLH  路  3Comments

asmith26 picture asmith26  路  4Comments