--import-root is being ignored about preprocessor HOT 12 CLOSED

Do you get the same problem -i?

from preprocessor.

yup, same thing

fyi I'm on mac os

node v6.9.1
npm 3.10.8
suitcss 3.0.0

from preprocessor.

@giuseppeg I think it might be since we added this merging of options - https://github.com/suitcss/preprocessor/blob/master/bin/suitcss#L113-L123

It comes in from the bin as importRoot but the node API is expecting root. We might need to translate that option manually.

@casio You can work round it with a config file until we get a fix out :)

from preprocessor.

It comes in from the bin as importRoot but the node API is expecting root. We might need to translate that option manually.

Right.

FWIW the snippet you linked has not been released yet.

from preprocessor.

FWIW the snippet you linked has not been released yet.

Oh yeah! Needs more than my cursory glance then

from preprocessor.

I'll investigate this later

from preprocessor.

@simonsmith thx for the hint (and the prompt reaction)!
however, no luck even with a config file - "root": "./src" seems to be ignored, as well.

also, I found that using json as the config fmt, raises: SyntaxError: Unexpected token : .
using a js file does not raise, though.

want me to create a new issue for this one?

from preprocessor.

@casio Yes please. I'll investigate that too

from preprocessor.

done. see #65

from preprocessor.

so I started working on a fix for importRoot a little: https://github.com/casio/preprocessor/tree/fix/import-root

the root vs importRoot ambiguity should be handled as well, I guess? which one do you prefer?
also related: should output go there as well?

from preprocessor.

Sorry @casio I totally missed that you had already started working on a fix there! Do you fancy rebasing that branch and opening a PR? We can move discussion there :)

from preprocessor.

see #69

from preprocessor.