Variables in LESS imports.

[berarma]

I've just noticed mini-asset won't find dependencies declared with an import that uses a variable in LESS.

I must say I don't really need this to work since the files I'm importing are vendor files that I won't change. Touching my files is enough to rebuild it. Just noticed it and thought I would report about it.

This will be hard to fully resolve. A possible workaround would be to have an option to disable incremental building or a configurable path to check for changes on any file inside it.

[markstory]

Could you give an example/sample of problematic files? Are you using node LESS, or the PHP implementations?

[berarma]

This is a simple example:

@root-dir: vendor/twbs/bootstrap/less;
@import "@{root-dir}/bootstrap.less";

I'm using node LESS. Compilation works well, it's only that the incremental builds fail to see changes on files imported in this way. Since this imports have to be resolved at compile time I guess it would be hard to know the exact dependencies before compiling.

[markstory]

Ah ok. Resolving those variable imports will be nigh on impossible without a sophisticated CSS parser. While the code you have could be solved, other cases like

// variables.less
@rootdir: some/path

// app.less
@import 'variables.less';
@import "@{rootdir}/other.less";

Will not be handled as easily.

[berarma]

I came across this issue while trying to work around issue #21. I don't really need it. Given the complexity of a proper fix, and because it's only affecting the incremental builds, I think this could wait. One easy way to deal with filter dependencies that are hard to solve like this one would be using the workarounds I proposed in my first message or something alike.