Skip to content
This repository has been archived by the owner on Aug 22, 2023. It is now read-only.

Cannot build standalone browser version anymore #76

Closed
MoOx opened this issue Jun 24, 2014 · 4 comments
Closed

Cannot build standalone browser version anymore #76

MoOx opened this issue Jun 24, 2014 · 4 comments

Comments

@MoOx
Copy link
Contributor

MoOx commented Jun 24, 2014

Hey @ianstormtaylor it seems with some node or browserify update we cannot make a working standalone myth.js anymore.
I just run make myth.js & try to run test (I've resolved #69) but got an error that seems related to node stuff (process.version is unknown).
Btw, I saw a lot of changes in that file. Not sure why.
Any clues about that ?

~/Sync/Development/myth/myth master  
❯ npm i              
is-browser@2.0.0 node_modules/is-browser

node-watch@0.3.4 node_modules/node-watch

pad-component@0.0.1 node_modules/pad-component

write-file-stdout@0.0.2 node_modules/write-file-stdout

rework-rebeccapurple@1.0.1 node_modules/rework-rebeccapurple

commander@2.1.0 node_modules/commander

rework-font-variant@0.0.3 node_modules/rework-font-variant

colors@0.6.2 node_modules/colors

rework-calc@0.2.2 node_modules/rework-calc

read-file-stdin@0.0.4 node_modules/read-file-stdin
└── stream-to-array@1.0.0

rework-hex-alpha@1.0.0 node_modules/rework-hex-alpha
└── rgb@0.0.1

rework-vars@3.1.1 node_modules/rework-vars
├── rework-visit@1.0.0
└── balanced-match@0.1.0

mocha@1.15.1 node_modules/mocha
├── diff@1.0.7
├── growl@1.7.0
├── commander@2.0.0
├── mkdirp@0.3.5
├── jade@0.26.3 (commander@0.6.1, mkdirp@0.3.0)
├── debug@1.0.2 (ms@0.6.2)
└── glob@3.2.3 (inherits@2.0.1, graceful-fs@2.0.3, minimatch@0.2.14)

rework-color-function@1.0.0 node_modules/rework-color-function
├── balanced-match@0.0.0
└── css-color-function@0.1.0 (rgb@0.0.1, debug@0.7.4, color@0.5.0)

rework@1.0.0 node_modules/rework
├── convert-source-map@0.3.4
├── mime@1.2.11
├── debug@1.0.2 (ms@0.6.2)
└── css@2.0.0 (urix@0.1.0, source-map-resolve@0.1.4, source-map@0.1.34)

autoprefixer@0.8.20131213 node_modules/autoprefixer
├── css-parse@1.6.0
└── css-stringify@1.4.1 (source-map@0.1.34)

rework-import@1.0.0 node_modules/rework-import
├── parse-import@0.1.3
├── find-file@0.1.4
├── graceful-fs@2.0.3
└── css@1.6.0 (css-parse@1.7.0, css-stringify@1.4.1)

browserify@3.46.1 node_modules/browserify
├── builtins@0.0.4
├── https-browserify@0.0.0
├── tty-browserify@0.0.0
├── constants-browserify@0.0.1
├── process@0.7.0
├── path-browserify@0.0.0
├── os-browserify@0.1.2
├── inherits@2.0.1
├── commondir@0.0.1
├── domain-browser@1.1.2
├── defined@0.0.0
├── stream-combiner@0.0.4
├── duplexer@0.1.1
├── shell-quote@0.0.1
├── deep-equal@0.1.2
├── shallow-copy@0.0.1
├── xtend@3.0.0
├── parents@0.0.2
├── console-browserify@1.0.3
├── querystring-es3@0.2.0
├── punycode@1.2.4
├── string_decoder@0.0.1
├── url@0.10.1
├── util@0.10.3
├── events@1.0.1
├── timers-browserify@1.0.1 (process@0.5.2)
├── stream-browserify@0.1.3 (process@0.5.2)
├── vm-browserify@0.0.4 (indexof@0.0.1)
├── assert@1.1.1 (util@0.10.2)
├── resolve@0.6.3
├── browser-resolve@1.2.4
├── subarg@0.0.1 (minimist@0.0.10)
├── glob@3.2.11 (minimatch@0.3.0)
├── through2@0.4.2 (xtend@2.1.2, readable-stream@1.0.27-1)
├── concat-stream@1.4.6 (typedarray@0.0.6, readable-stream@1.1.13-1)
├── JSONStream@0.7.4 (through@2.3.4, jsonparse@0.0.5)
├── deps-sort@0.1.2 (through@2.3.4, minimist@0.0.10, JSONStream@0.6.4)
├── crypto-browserify@1.0.9
├── http-browserify@1.3.2 (Base64@0.2.1)
├── syntax-error@1.1.0 (esprima-fb@3001.1.0-dev-harmony-fb)
├── browserify-zlib@0.1.4 (pako@0.2.3)
├── umd@2.0.0 (through@2.3.4, rfile@1.0.0, ruglify@1.0.0, uglify-js@2.4.14)
├── browser-pack@2.0.1 (through@2.3.4, JSONStream@0.6.4, combine-source-map@0.3.0)
├── derequire@0.8.0 (estraverse@1.5.0, esrefactor@0.1.0, esprima-fb@3001.1.0-dev-harmony-fb)
├── buffer@2.1.13 (ieee754@1.1.3, base64-js@0.0.7)
├── module-deps@2.0.6 (minimist@0.0.10, stream-combiner@0.1.0, readable-stream@1.0.27-1, duplexer2@0.0.2, detective@3.1.0)
└── insert-module-globals@6.0.0 (process@0.6.0, through@2.3.4, lexical-scope@1.1.0)

~/Sync/Development/myth/myth master  14s
❯ make myth.js       

~/Sync/Development/myth/myth master*  11s
❯ npm test           

> myth@1.0.4 test /Users/MoOx/Sync/Development/myth/myth
> make test


/Users/MoOx/Sync/Development/myth/myth/myth.js:12550
    process.version.match(/^v0\.6\.[0-2]|^v0\.5\./)) {
                    ^
TypeError: Cannot call method 'match' of undefined
    at Object.<anonymous> (/Users/MoOx/Sync/Development/myth/myth/myth.js:12550:21)
    at Object.FWaASH (/Users/MoOx/Sync/Development/myth/myth/myth.js:12758:4)
    at s (/Users/MoOx/Sync/Development/myth/myth/myth.js:1:573)
    at /Users/MoOx/Sync/Development/myth/myth/myth.js:1:624
    at Object.<anonymous> (/Users/MoOx/Sync/Development/myth/myth/myth.js:12372:1)
    at Object../polyfills.js (/Users/MoOx/Sync/Development/myth/myth/myth.js:12526:4)
    at s (/Users/MoOx/Sync/Development/myth/myth/myth.js:1:573)
    at /Users/MoOx/Sync/Development/myth/myth/myth.js:1:624
    at Object.<anonymous> (/Users/MoOx/Sync/Development/myth/myth/myth.js:11334:10)
    at Object.FWaASH (/Users/MoOx/Sync/Development/myth/myth/myth.js:11476:4)
    at s (/Users/MoOx/Sync/Development/myth/myth/myth.js:1:573)
    at /Users/MoOx/Sync/Development/myth/myth/myth.js:1:624
    at Object.autoprefixer (/Users/MoOx/Sync/Development/myth/myth/myth.js:8:16)
    at s (/Users/MoOx/Sync/Development/myth/myth/myth.js:1:573)
    at e (/Users/MoOx/Sync/Development/myth/myth/myth.js:1:744)
    at /Users/MoOx/Sync/Development/myth/myth/myth.js:1:762
    at a (/Users/MoOx/Sync/Development/myth/myth/myth.js:1:119)
    at Object.<anonymous> (/Users/MoOx/Sync/Development/myth/myth/myth.js:1:309)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at Object.<anonymous> (/Users/MoOx/Sync/Development/myth/myth/test/index.js:8:15)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at /Users/MoOx/Sync/Development/myth/myth/node_modules/mocha/lib/mocha.js:157:27
    at Array.forEach (native)
    at Mocha.loadFiles (/Users/MoOx/Sync/Development/myth/myth/node_modules/mocha/lib/mocha.js:154:14)
    at Mocha.run (/Users/MoOx/Sync/Development/myth/myth/node_modules/mocha/lib/mocha.js:341:31)
    at Object.<anonymous> (/Users/MoOx/Sync/Development/myth/myth/node_modules/mocha/bin/_mocha:351:7)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Function.Module.runMain (module.js:497:10)
    at startup (node.js:119:16)
    at node.js:906:3
make: *** [test] Error 8
npm ERR! Test failed.  See above for more details.
npm ERR! not ok code 0
``
@ianstormtaylor
Copy link
Contributor

Hmm not sure :/ I remember having some weird browserify issues at one point, but not what they were about. Seemed finicky, but if we can get it working with the latest version that would be ideal

@MoOx
Copy link
Contributor Author

MoOx commented Jun 26, 2014

It seems to me broken since the switch from rework-inline to rework-import (which is a bit weird since this change is related to a merge of rework-inline to rework-import).
I hope I'll found what is the f***.

MoOx added a commit to reworkcss/rework-import that referenced this issue Jun 26, 2014
@MoOx MoOx closed this as completed in 4f71e08 Jun 26, 2014
@MoOx
Copy link
Contributor Author

MoOx commented Jun 26, 2014

It was related to a use of graceful-fs module in rework-import. Sorry for that, fixed.

@ianstormtaylor
Copy link
Contributor

boom nice find!

On Wed, Jun 25, 2014 at 11:54 PM, Maxime Thirouin notifications@github.com
wrote:

It was related to a use of graceful-fs module in rework-import. Sorry for
that, fixed.


Reply to this email directly or view it on GitHub
#76 (comment).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants