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

mean package newPkg fails #3

Closed
theneilmeister opened this issue Sep 12, 2014 · 2 comments · May be fixed by pesaply/mean#30, pesaply/mean#31, enterstudio/payroll-mean#28, pesaply/mean#33 or enterstudio/payroll-mean#30

Comments

@theneilmeister
Copy link

I'm having an issue with creating a new mean package "testPackage".
Running OSX 10.9, npm 1.4.23 rocking mean v0.5.26.

Steps:
npm install -g meanio
npm install -g mean-cli

mean init testApp
cd testApp
npm install
grunt // Works great :)

// then
mean package testPackage
grunt // Fails :(

Terminal output:
Running "clean:0" (clean) task
Done, without errors.
Execution Time (2014-09-12 16:48:47 UTC)
loading tasks 880ms ▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇ 99%
Total 887ms

Running "clean:0" (clean) task
Running "jshint:all" (jshint) task

packages/custom/testpackage/app.js
27 | Testpackage.aggregateAsset('css', testpackage + '.css');
^ 'testpackage' is not defined.

1 error in 47 files
Warning: Task "jshint:all" failed. Use --force to continue.

Aborted due to warnings.

Many thanks guys

@fyockm
Copy link
Contributor

fyockm commented Sep 12, 2014

@theneilmeister I see the problem. Fix forthcoming.

@fyockm fyockm closed this as completed in 0ebb6ac Sep 12, 2014
@fyockm
Copy link
Contributor

fyockm commented Sep 12, 2014

@theneilmeister Thanks for pointing out the issue! published as mean-cli@0.5.27, available now via npm. Update with $ npm update -g mean-cli@latest

Note: only mean-cli needs to be installed globally. We are planning to split meanio and mean-cli into separate modules and are currently in a transitionary period. Right now, both repos are exactly the same. But in the near future, mean-cli should be installed global and meanio will be a mean app dependency.

Jeff-Lewis pushed a commit to Jeff-Lewis/meanio that referenced this issue May 15, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment