New Hacker Slides Graphics for Sandstorm.io Package about hacker-slides

[neynah]

I attempted to create a pull request with these graphics but I'm on a Windows machine (which has a character limit of 260) so I was not able to clone to windows. Could you repackage under vagrant-spk?

[ocdtrekkie]

@neynah Does vagrant fix that? I'm assuming there's a filename somewhere in the package that is too big for Windows?

[neynah]

I believe @zarvox mentioned that it might resolve the issue but I'll double check.

[zarvox]

The issue appears to be that there's a bunch of npm dependencies, and the npm folks have decided to include every package's dependencies in a subfolder of that package called node_modules, recursively. The longest path is static/node_modules/grunt-contrib-qunit/node_modules/grunt-lib-phantomjs/node_modules/phantomjs/node_modules/request/node_modules/form-data/node_modules/combined-stream/node_modules/delayed-stream/test/integration/test-delayed-stream-auto-pause.js which clocks in at a massive 247 characters.

(Actually @neynah if it's 247 characters, then it might succeed (barely) if you try to clone to a very short path like: cd C:\src && git clone https://github.com/jacksingleton/hacker-slides hs )

Git, like most software that runs on Windows, does not appear to use the extended-length path option. If it did, that would be one way to fix the issue.

If, rather than including the static dependencies outright, the package pulls them in via npm or bower in the guest, it's possible that the Virtualbox shared folder driver will use the "long paths" Windows API, whereas git on the host does not. It's also possible that Virtualbox will also be constrained by the path limit, which means that contributors would need to attempt to clone the repo from the VM and do everything from within.

[neynah]

@zarvox Just tried it again in C: and still got:

error: unable to create file static/node_modules/grunt-contrib-qunit/node_modules/grunt-lib-phantomjs/node_modules/phant
omjs/node_modules/request/node_modules/form-data/node_modules/combined-stream/node_modules/delayed-stream/test/integrati
on/test-delayed-stream-auto-pause.js (Filename too long)

[neynah]

I got it working. Thanks @zarvox. Will pull request now.

[jacksingleton]

Hey Nena! Sorry for the late response, I'm on vacation right now and get back the week if the 24th.

Thanks for taking the time to put the pull request together! (and for creating the graphics for that matter, looks awesome - clean and simple)

I may get to the pr this week, but if not count on the week of the 24th.

Jack

[ocdtrekkie]

Can be closed.