[GitHub] incubator-zeppelin pull request: Install guide modifications based...

2016-03-04 Thread jeffsteinmetz
Github user jeffsteinmetz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/543#issuecomment-192379971 I too confirmed, the build runs in the VM without NPM / Node installed. The VM installs node and ruby (for docs only) so that a developer can bu

[GitHub] incubator-zeppelin pull request: Install guide modifications based...

2016-03-04 Thread dcardon
Github user dcardon commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/543#issuecomment-192377106 Thanks for following up, @felixcheung! I've modified the changes to only call out the possible need to run `git` through `https://` instead of `git:/

[GitHub] incubator-zeppelin pull request: Install guide modifications based...

2016-03-03 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/543#issuecomment-192082492 @dcardon @jeffsteinmetz where are we on this? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] incubator-zeppelin pull request: Install guide modifications based...

2015-12-21 Thread jeffsteinmetz
Github user jeffsteinmetz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/543#issuecomment-166465873 We could reference this VM guide earlier in the Zeppelin documentation. Let me know if you think there is a documentation location at the top of the p

[GitHub] incubator-zeppelin pull request: Install guide modifications based...

2015-12-21 Thread dcardon
Github user dcardon commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/543#issuecomment-166446141 Thanks, @jeffsteinmetz! I didn't know that guide existed. I did run into issues running this on Windows, but eventually this does work (after manual Ansibl

[GitHub] incubator-zeppelin pull request: Install guide modifications based...

2015-12-20 Thread jeffsteinmetz
Github user jeffsteinmetz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/543#issuecomment-166158698 If you follow the virtual machine instructions here, it installs with all required dependencies. You will see that grunt and bower are not required

Re: [GitHub] incubator-zeppelin pull request: Install guide modifications based...

2015-12-19 Thread Jeff Steinmetz
What commits added the dependency on grunt and bower? I will need to see if this effects the scripts/vagrant VM script as well. On Wed, Dec 16, 2015 at 2:08 PM, dcardon wrote: > Github user dcardon commented on the pull request: > > > https://github.com/apache/incubator-zeppelin/pull/543#issue

[GitHub] incubator-zeppelin pull request: Install guide modifications based...

2015-12-16 Thread dcardon
Github user dcardon commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/543#issuecomment-165268485 ### What is this PR for? A minor (but hopefully useful to others) documentation change. ### What type of PR is it? Documentation ###

[GitHub] incubator-zeppelin pull request: Install guide modifications based...

2015-12-16 Thread dcardon
GitHub user dcardon opened a pull request: https://github.com/apache/incubator-zeppelin/pull/543 Install guide modifications based on recent build attempts I found that an install on a fresh VM didn't succeed without a few global npm installations and without telling git to used htt