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

path is too long #5446

Closed
infofromca opened this issue Jun 28, 2015 · 14 comments
Closed

path is too long #5446

infofromca opened this issue Jun 28, 2015 · 14 comments
Assignees
Labels
Milestone

Comments

@infofromca
Copy link
Contributor

when install it from dev or 1.9.x, 1.9.1 today's version,
got this error. (visual studio 2013)
too-long-gulp
"~/Modules/Orchard.DynamicForms/node_modules/gulp-less/node_modules/gulp-util/node_modules/dateformat/node_modules/meow/node_modules/indent-string/node_modules/repeating/node_modules/is-finite/node_modules/"

@infofromca
Copy link
Contributor Author

@sebastienros
Copy link
Member

I know it has been fixed in the latest npm version.

@bleroy
Copy link
Member

bleroy commented Jul 23, 2015

Link?

@bleroy
Copy link
Member

bleroy commented Jul 23, 2015

Duplicate: #5561

@Skrypt
Copy link
Contributor

Skrypt commented Jul 23, 2015

We should then automate the removal of those node_modules before publishing.
Or we could also excluse those folder from publishing.

http://blogs.msdn.com/b/webdev/archive/2010/04/22/web-deployment-excluding-files-and-folders-via-the-web-application-s-project-file.aspx

@Skrypt
Copy link
Contributor

Skrypt commented Jul 23, 2015

Not working either.

http://stackoverflow.com/questions/25237823/actually-excluding-files-from-publish-without-verifying-it

Though the guy seems to have a workaround by installing some (if not all) of the dependecies globally:

npm install -g package

and then installing into the project using

npm install --link

@Skrypt
Copy link
Contributor

Skrypt commented Jul 23, 2015

Added this in PRE build event command line of the 3 projects that holds node_modules folders.

if exist "$(ProjectDir)node_modules"  ( rimraf $(ProjectDir)node_modules )

It deletes the folders but that's not really an adequate solution. Though it fixes the publish problem automatically. You also need to install rimraf globally before :

npm install rimraf -g

@DaRosenberg
Copy link
Member

This will be fixed as part of #5450.

@dcinzona
Copy link
Contributor

Agree with @Skrypt - Installing globally isn't really a viable solution for everyone using the source. Not to mention that this would force npm to reinstall the node modules every start up and that gulp would fail after deployments.

@DanielStolt I mentioned this in the dupe issue: I had success creating a generic project and adding it to the solution, adding the gulpfile and package.json manifest to it. This allows me to download all of the npm modules into a separate directory in the root and avoid deployment issues. Just need to remember to add it to all solutions (Azure Cloud Service and Orchard.Web).

As you had mentioned before, including a .gitignore and .tfignore config in there would be beneficial as to not commit those node modules to source control.

@DaRosenberg
Copy link
Member

@dcinzona This sounds like what I have in mind? Going to work on it this weekend.

@dcinzona
Copy link
Contributor

Awesome! Looking forward to it
Thanks @DanielStolt

@DaRosenberg DaRosenberg self-assigned this Jul 26, 2015
@DaRosenberg
Copy link
Member

Fixed in 6da6c5c.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

7 participants
@dcinzona @infofromca @bleroy @sebastienros @Skrypt @DaRosenberg and others