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

Cannot track error properly. #91

Closed
vladikoff opened this issue Jul 11, 2014 · 6 comments
Closed

Cannot track error properly. #91

vladikoff opened this issue Jul 11, 2014 · 6 comments

Comments

@vladikoff
Copy link

Running

 myth main.css some.css

Outputs:

$ myth main.css some.css

       error · RangeError: Maximum call stack size exceeded


/Users/vfilippov/npmlocal/lib/node_modules/myth/bin/logger.js:44
    err.stack.split('\n').slice(1).forEach(function(line){
              ^
TypeError: Cannot call method 'split' of undefined
    at Object.exports.throw (/Users/vfilippov/npmlocal/lib/node_modules/myth/bin/logger.js:44:15)
    at /Users/vfilippov/npmlocal/lib/node_modules/myth/bin/myth:92:19
    at ReadStream.onEnd (/Users/vfilippov/npmlocal/lib/node_modules/myth/node_modules/read-file-stdin/node_modules/stream-to-array/index.js:38:5)
    at ReadStream.g (events.js:180:16)
    at ReadStream.EventEmitter.emit (events.js:117:20)
    at _stream_readable.js:919:16
    at process._tickCallback (node.js:419:13)
@MoOx
Copy link
Contributor

MoOx commented Jul 12, 2014

Can you share your file(s) somewhere ?
Or maybe try with latest version (current master) ?

@miketaylr
Copy link

Hi! Our grunt file is at https://github.com/webcompat/webcompat.com/, you should be able to just clone and npm install && grunt to reproduce. I'm wondering if this isn't somehow related to having different versions of autoprefixer (as a standalone dep and as a myth dep)? cc @magsout

@miketaylr
Copy link

So this is weird (to me), webcompat/webcompat.com@3fbcaf7 fixed the issue. However, myth seems to be producing invalid CSS which causes other grunt tasks to explode. See webcompat/webcompat.com#168 (comment).

@MoOx
Copy link
Contributor

MoOx commented Jul 14, 2014

I successfully reproduce this error with 1.0.4

  21 passing (2s)
  2 failing

  1) features should add vars support:
     Maximum call stack size exceeded


  2) browser should add vars support:
     Maximum call stack size exceeded




make: *** [test] Error 2

That being said, I cannot reproduce it with current version. So closing since 1.1.0 is out.
@miketaylr for the other issue you got, it has already been fixed too. Please checkout latest version.

@MoOx MoOx closed this as completed Jul 14, 2014
@miketaylr
Copy link

Thanks @MoOx! Confirmed fixed on 1.1.0.

@magsout
Copy link

magsout commented Jul 14, 2014

👍

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

4 participants