Fix issue with npm package.js (issue 7849048)

1 view
Skip to first unread message

a...@chromium.org

unread,
Mar 22, 2013, 12:55:43 PM3/22/13
to pete...@google.com, usrb...@yahoo.com, traceur-comp...@googlegroups.com, re...@codereview-hr.appspotmail.com
Reviewers: peterhal, usrbincc,

Message:
I'm not really sure how to test this without pushing to npmjs.org

Description:
Fix issue with npm package.js

The main field should point to a file that can be directly required.
With this
change the following should work

npm install traceur -g

and in your node file

var traceur = require('traceur');

BUG=None


Please review this at https://codereview.appspot.com/7849048/

Affected files:
M package.json


Index: package.json
diff --git a/package.json b/package.json
index
5a30e06248df5d675863ce1cf330c55d67daeaf1..2ecd854d75b0248bb8d15ef0c600fb99fc2ecaa9
100644
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
{
"name": "traceur",
- "version": "0.0.2",
+ "version": "0.0.3",
"description": "Experimental ES6 to ES5 compiler",
"keywords": [
"javascript",
@@ -18,7 +18,7 @@
"engines": {
"node": ">=0.8"
},
- "main": "./bin/traceur.js",
+ "main": "./src/node/traceur.js",
"bin": {
"traceur": "./traceur"
},


a...@chromium.org

unread,
Mar 22, 2013, 2:55:44 PM3/22/13
to pete...@google.com, usrb...@yahoo.com, traceur-comp...@googlegroups.com, re...@codereview-hr.appspotmail.com
Committed patchset #1 manually as r74e5bd0 (presubmit successful).

https://codereview.appspot.com/7849048/
Reply all
Reply to author
Forward
0 new messages