includePaths option prepends sass folder path to file locations

[asolopovas]

Hi,

Recently I started to get strange effect, the sourceMap refferal to files from included paths prepends path of sass directory. So in my source-maps refers to files like C:/vhosts/project/resources/assets/sass/C:\vhosts\project/vendor\zurb\foundation\scss\grid\_gutter.scss
instead of 'C:\vhosts\3oakdb/vendor\zurb\foundation\scss\grid_gutter.scss',

here is the link to my webpack.config

[asolopovas]

I have updated my sass-loader-test repo https://github.com/asolopovas/sass-loader-test, to reflect the problem if you start index.html in public folder and inspect p.animate tag in inspect section the path will be shown as D:/path/sass-loader-test/resources/assets/sass/D:\path\sass-loader-test/resources\assets\sass\app.scss

[asolopovas]

Tested it on windows ubuntu bash and sourcemaps path was fine.

[asolopovas]

Same happens if you do run npm run test-source-map on the package itself so clearly is something wrong about it.

[asolopovas]

Please respond someone its been a month since I created an issue. I have spent some time playing around and found that if I changeline 74 of loader.js, to result.map.sourceRoot = ''; problem goes away

[asolopovas]

@jhnns it may be related to #366

[alexander-akait]

@asolopovas wip

[alexander-akait]

@asolopovas hm, works fine for me, can your explain more information? Whatbrowser your are use?If your can create screenshots this bug?

[asolopovas]

@evilebottnawi Hi thanks for responding, here is the screenshots, I have tested it on different machines and few various projects everywhere I get the same problem, I eventually gave up and used gulp sass to generate my stylesheets along with browser-sync to inject them on fly it actually does it a bit faster. If you look at the repository provided above and generate it by simply running npm install + composer install + webpack, it will generate js script with source code in it and then if you open index.html you will see the error that I provided screenshots for.

[alexander-akait]

@asolopovas oh really, tomorrow try to found solution 👍

[alexander-akait]

i hate windows, i spend about 4 hours just to build start to work 😭

[asolopovas]

@evilebottnawi I love windows, I hate their filesystem, they should have adopted it long time ago and nobody would be using mac right now. But if you fiddle a bit you can figure thing oit

[alexander-akait]

@asolopovas i don't use mac 😄

[alexander-akait]

Confirmed, start to searching solution

[alexander-akait]

Seems related to mozilla/source-map#91

[michael-ciniawsky]

webpack-contrib/css-loader#532

[alexander-akait]

Not related to style-loader, feel free to reopen if after merge webpack-contrib/css-loader#532 error still exists