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

Default install of big throws on missing modules #11

Open
luk- opened this issue Apr 25, 2013 · 4 comments
Open

Default install of big throws on missing modules #11

luk- opened this issue Apr 25, 2013 · 4 comments

Comments

@luk-
Copy link

luk- commented Apr 25, 2013

I installed big in a local node_modules folder and started a webserver just using the admin resource. It threw errors for missing both resource & resources modules.

@Marak
Copy link
Member

Marak commented Apr 25, 2013

Well that's not good. They use to be listed as regular dependencies but now are peerDependencies as per 9769f8c

I don't think this is the correct usage of peerDependencies

@luk-
Copy link
Author

luk- commented Apr 25, 2013

I don't think this is the correct usage of peerDependencies

I agree with this.

@Marak Marak closed this as completed in 9d23963 Apr 25, 2013
@Marak Marak reopened this Apr 25, 2013
@erobit
Copy link

erobit commented Jun 6, 2013

The error is being thrown for me when executing my /app/index.js file. At this point, any /app/node_modules/big/node_modules/resources that are used in the file are copied to the /app/resources/ folder. The copied resources, cannot resolve the require('resource') package (installed by default as big dependencies) if the resource/resources packages haven't been manually installed through npm install resource / npm install resources.

I'm not sure if this is by design, with the assumption that big is to be installed as a global module. I wasn't able to get big to work as a global module, but it could have been due to NODE_PATH issues or other.

I guess for the time being, the documentation could be updated (How to create a big app?) to indicate that along with big installed globally, resource and resources need to be installed when creating a new app.

@Marak
Copy link
Member

Marak commented Jun 20, 2013

Big no now zero-lines long. require('resource') is now used instead

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

3 participants