Less.js: Source map bug introduced in 2.7.0, but passes tests

Created on 8 May 2016  ·  19Comments  ·  Source: less/less.js

Will put out a hotfix soon if the fix can be confirmed. It looks like PR #2834 introduced a bug that produces a blank source map, and throws an error like:

Invalid mapping: {"generated":{"line":1103,"column":2},"source":"file.less","original":{"line":1,"column":null},"name":null}

(Also discussed at #2881.)

Reverting #2834 seems to fix it, but looking for confirmation.

bug high priority

Most helpful comment

This just broke my builds, need a patch asap...

All 19 comments

This issue should stay open until a Less.js test is added which catches this issue, since the source map tests didn't detect the bug.

I'm experiencing the same issue.
I executed a git bisect and the first commit that broke the build was 470af20f641c9b5389475733dfe341aa4c180fba which is in the PR you noted in your issue

better unpublish this version

better unpublish this version

It depends. For some the sourcemaps are totally insignificant if compared to for example https://github.com/less/less.js/issues/2841 of the previous release.

In case it can help, I could reproduce the issue with something like:

Less:

.dl-horizontal {
    @media (min-width: 768px) {
        dt {
            float: left;
        }
    }
}

Gulp task:

return gulp.src('./src/lib.less')
    .pipe(sourcemaps.init())
    .pipe(less())
    .pipe(sourcemaps.write('.'))
    .pipe(gulp.dest(bundleFolder));

This just broke my builds, need a patch asap...

Please fix this bug asap, or I can only froze the version to 2.6.X.
And I wish this bug could be fixed before tomorrow morning.
Appreciate it very much.

Production chaos on this side as well...

Crashes my gulp build as well

If you're having trouble with a dependency using less as a dependency and simply can't downgrade in package.json.
Do a npm shrinkwrap --dev and then modify the version of less to 2.6.1 then full npm reinstall will fix it until less.js is fixed

Clearly the impact is widespread. It blocked my production release. Yes, please implement a hotfix, and provide workarounds, but in the meantime, for the love of all that is good and holy:

$ npm unpublish [email protected]

Unless you will have 2.7.1 published within, say, the next hour, nip this in the bud and unpublish.

Any line of sight for a fix on this yet?

I've published 2.7.1 as a hotfix for 2.7.0, but it's unclear from #2881 if there isn't still an outstanding bug affecting source maps (multiple intersecting bugs from same commit?), or if that was a pre-existing bug which was introduced separately.

Please report back if 2.7.1 is working for you.

works for me, thanks @matthew-dean

the bootprint-integration-test had failed with [email protected] and it succeeds with 2.7.1. Works for me. Thanks @matthew-dean

Awesome. Sorry for the broken build everyone, but glad it's working now. Will close this.

@matthew-dean good job

@olivierlesnicki 👍

Hi, If my project have been compile all less file to css file already with chrun2 program.
I'm not use less.js.
Could you tell me how to do about debug css show less in get element inspector develope.
How to config sauce map?

Was this page helpful?
0 / 5 - 0 ratings

Related issues

chricken picture chricken  ·  6Comments

rejas picture rejas  ·  6Comments

BrianMulhall picture BrianMulhall  ·  4Comments

Oskariok picture Oskariok  ·  6Comments

joe223 picture joe223  ·  4Comments