Skip to content
This repository has been archived by the owner on Dec 4, 2018. It is now read-only.

Remove trailing comma, to fix JSON parsing. #110

Closed
wants to merge 1 commit into from

Conversation

adamgaynor
Copy link

NPM fails to parse this JSON file when running npm install.

@dominictarr
Copy link
Owner

what npm version is this? that is actuall couchdb output that has been in this repo forever this is a bug in npm

@dominictarr
Copy link
Owner

what npm version? I am unable to reproduce this with either npm 3.8 or 4.0.2

@adamgaynor
Copy link
Author

I had an issue running NPM 1.3.10, with node 0.10.25.

Everything worked fine until a few days ago, when NPM started throwing npm ERR! registry error parsing json, so this may not be the actual cause of the error.

I realize this is an older version of NPM, but figure that fixing a syntax error is never a bad thing.

@dominictarr
Copy link
Owner

I'm doubtful that this file is the cause of your error. Npm shouldn't even parse that file while installing.
using nvm, I tried node 0.10.25, with npm 1.3.24 and installed this fine, same with node 0.10.48 and npm 2.15.1

My bet is that what you are getting is either the registry giving an invalid response (or something breaking it) or your cache.

@adamgaynor
Copy link
Author

It does look like that was the case, with the registry giving an invalid response.
Updating node and npm also fixed the issue, thank you for the suggestion!

@adamgaynor adamgaynor closed this Nov 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants