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

Syntax error in demeteorizer.js:251:32 after finding dependencies #69

Closed
DiegoOlivierFernandezPons opened this issue Sep 2, 2014 · 6 comments · Fixed by #87
Closed

Comments

@DiegoOlivierFernandezPons

The issue appears on Windows 8. Seems to work fine on MacOS (both on 0.8.3)

D:\tmp> meteor create --example leaderboard
leaderboard: created.

To run your new app:
cd leaderboard
meteor
D:\tmp> cd .\leaderboard
D:\tmp\leaderboard> demeteorizer
Input: D:\tmp\leaderboard
Output: D:\tmp\leaderboard.demeteorized
Determining Meteor version...
Meteor version: 0.8.3

Bundling Meteor App...
meteor bundle generation complete.
Finding dependencies...

undefined:0

^
SyntaxError: Unexpected end of input
at Object.parse (native)
at C:\Users\user\AppData\Roaming\npm\node_modules\demeteorizer\lib\demeteorizer.js:251:32
at Array.forEach (native)
at Demeteorizer.findDependenciesInFolder (C:\Users\user\AppData\Roaming\npm\node_modules\demeteorizer\lib\demeteorizer.js:228:9)
at C:\Users\user\AppData\Roaming\npm\node_modules\demeteorizer\lib\demeteorizer.js:246:14
at Array.forEach (native)
at Demeteorizer.findDependenciesInFolder (C:\Users\user\AppData\Roaming\npm\node_modules\demeteorizer\lib\demeteorizer.js:228:9)
at C:\Users\user\AppData\Roaming\npm\node_modules\demeteorizer\lib\demeteorizer.js:246:14
at Array.forEach (native)
at Demeteorizer.findDependenciesInFolder (C:\Users\user\AppData\Roaming\npm\node_modules\demeteorizer\lib\d
emeteorizer.js:228:9)

@InconceivableDuck
Copy link
Contributor

The problem is here: Output: D:\tmp\leaderboard.demeteorized

It's not properly putting the .demeteorized folder inside the leaderboard folder.

@seanli
Copy link

seanli commented Sep 23, 2014

@InconceivableDuck Getting the same issue on Mac OS

@fiveisprime
Copy link
Collaborator

@Elesant What version of everything (node, demeteorizer, and meteor) and what is the exact command that you are running?

Also, can you paste the full error text? I'd like to see what it looks like on OS X.

@fiveisprime
Copy link
Collaborator

This appears to be an issue with this module.

See this comment for more details.

@DiegoOlivierFernandezPons
Copy link
Author

My original bug report doesn't use meteorhacks:npm anywhere.

@fiveisprime
Copy link
Collaborator

Daaang, okay. I guess this is just something with 0.8.x. Sorry about that!

fiveisprime pushed a commit that referenced this issue Oct 9, 2014
There is a module that includes a test fixture that is an empty
package.json file which demeteorizer attempts to parse which fails.

Each package.json file is now checked that it contains data and is
skipped if it does not.

Closes #69
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

Successfully merging a pull request may close this issue.

4 participants