TypeError: Arguments to path.resolve must be strings

[mishanga]

CLI:

/* test.roo */
div { background: url(../../../logo.png) }

./node_modules/.bin/roole test.roo

Result — OK:

/* test.css */
div {
    background: url(../../../logo.png);
}

---

Require:

require('roole').compile(
    'div { background: url(../../../logo.png) }',
    {},
    function(err, res){
        console.log(err || res);
    }
);

Result — Error:

./node_modules/.bin/roole test.js
[TypeError: Arguments to path.resolve must be strings]

Hack: compile('...', { out: './' }, function(...){...}

Debug info:

node -v
v0.10.9
 ./node_modules/.bin/roole -v
0.5.1

[curvedmark]

You didn't provide the filename option, so roole tried to resolve the relative url to undefined, and the path module complained.

Looks like a more reasonable error message should be generated when required options are missing.

[mishanga]

Maybe better to use './' by default when no path provided?

[curvedmark]

Good idea. But filename has to be absolute. Maybe use cwd when no value is provided.

[mishanga]

CWD will cause few additional ../ in path. Where can I read about filename option?

[curvedmark]

What do you mean by "few additional ../"? I'm talking about process.cwd() here.

the readme file of roole-parser contains the description of filename option.

[curvedmark]

the out option should also be provided, it should get a default value too.

[curvedmark]

Introduced a base option, which will be used if filename or out is not provided. see roole-evaluator