Skip to content

Commit

Permalink
Merge pull request #320 from martinoconnor/master
Browse files Browse the repository at this point in the history
Modify sourcemap paths of imported files
  • Loading branch information
Snugug committed Sep 19, 2015
2 parents 8bfcb19 + 77e66a7 commit 64d93e6
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 3 deletions.
15 changes: 14 additions & 1 deletion index.js
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,7 @@ var gulpSass = function gulpSass(options, sync) {
if (file.sourceMap) {
opts.sourceMap = file.path;
opts.omitSourceMapUrl = true;
opts.sourceMapContents = true;
}

//////////////////////////////
Expand All @@ -65,7 +66,9 @@ var gulpSass = function gulpSass(options, sync) {
filePush = function filePush(sassObj) {
var sassMap,
sassMapFile,
sassFileSrc;
sassFileSrc,
sassFileSrcPath,
sourceFileIndex;

// Build Source Maps!
if (sassObj.map) {
Expand All @@ -75,6 +78,16 @@ var gulpSass = function gulpSass(options, sync) {
sassMapFile = sassMap.file.replace('stdout', 'stdin');
// Grab the base file name that's being worked on
sassFileSrc = file.relative;
// Grab the path portion of the file that's being worked on
sassFileSrcPath = path.dirname(sassFileSrc);
if (sassFileSrcPath) {
//Prepend the path to all files in the sources array except the file that's being worked on
for (sourceFileIndex = 0; sourceFileIndex < sassMap.sources.length; sourceFileIndex++) {
if (sourceFileIndex !== sassMap.sources.indexOf(sassMapFile)) {
sassMap.sources[sourceFileIndex] = path.join(sassFileSrcPath, sassMap.sources[sourceFileIndex]);
}
}
}
// Replace the stdin with the original file name
sassMap.sources[sassMap.sources.indexOf(sassMapFile)] = sassFileSrc;
// Replace the map file with the original file name (but new extension)
Expand Down
4 changes: 2 additions & 2 deletions test/main.js
Original file line number Diff line number Diff line change
Expand Up @@ -438,8 +438,8 @@ describe('gulp-sass -- sync compile', function() {

it('should work with gulp-sourcemaps and autoprefixer with different file.base', function(done) {
var expectedSources = [
'includes/_cats.scss',
'includes/_dogs.sass',
'scss/includes/_cats.scss',
'scss/includes/_dogs.sass',
'scss/inheritance.scss'
];

Expand Down

0 comments on commit 64d93e6

Please sign in to comment.