PostCSS could generate wrong source map error #13763

mburger81 · 2018-01-04T15:59:56Z

Ionic version: (check one with "x")
(For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[x] 3.x
[ ] 4.x

I'm submitting a ... (check one with "x")
[x] bug report
[ ] feature request

Please do not submit support requests or "How to" questions here. Instead, please use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/

Current behavior:
If we build a new app using your blank starter or any other template building the android app we are running into this error

[16:52:18]  sass started ... 
Witout `from` option PostCSS could generate wrong source map or do not find Browserslist config. Set it to CSS file path or to `undefined` to prevent this warning
[16:52:19]  sass finished in 1.13 s

Expected behavior:
The bug message should not be

Steps to reproduce:
Create a new blank app using
ionic start postcss
build the android app doing
ionic cordova build android

and you can see log with the warn or error message

[16:57:54]  sass started ... 
Witout `from` option PostCSS could generate wrong source map or do not find Browserslist config. Set it to CSS file path or to `undefined` to prevent this warning
[16:57:55]  sass finished in 793 ms

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):

cli packages: (/usr/lib/node_modules)

    @ionic/cli-utils  : 1.19.0
    ionic (Ionic CLI) : 3.19.0

global packages:

    cordova (Cordova CLI) : 8.0.0 

local packages:

    @ionic/app-scripts : 3.1.6
    Cordova Platforms  : android 7.0.0
    Ionic Framework    : ionic-angular 3.9.2

System:

    Android SDK Tools : 26.1.1
    Node              : v8.9.4
    npm               : 5.6.0 
    OS                : Linux 4.13

Environment Variables:
    ANDROID_HOME : /abcdef/android/sdk

Misc:
    backend : legacy

The text was updated successfully, but these errors were encountered:

This removes following warning: Remove warning: Without `from` option PostCSS could generate wrong source map or do not find Browserslist config. Set it to CSS file path or to `undefined` to prevent this warning `from: the input file name (most runners set it automatically).` Source: https://github.com/postcss/postcss Fixes ionic-team#1359 #13763 ionic-team#1359 ionic-team/ionic-framework#13763

StefanRein · 2018-01-09T11:52:02Z

As the warning says:

Without from option PostCSS could generate wrong source map or do not find Browserslist config. Set it to CSS file path or to undefined to prevent this warning

I added the field from and did set it to undefined and created a pull request.

https://github.com/StefanRein/ionic-app-scripts/blob/29873ae96611ae209ca4b065fd9940d6b7b96bda/src/sass.ts#L285

This resolves this warning. But I don't know if this has further consequences and the Ionic Team needs to fill in there a path to the generated css file or something like that.

@danbucholtz Could you review this please? Our automated builds are failing because of this.
(The "warning" is interpreted as an error..)

Edit: PostCSS uses ultimately console.warn, but this goes on node to the stderr pipe and our TeamFS fails.

stderr is used for warning or error output. Source

StefanRein · 2018-01-09T13:20:13Z

Until this is fixed, one possibility is to revert back some patches before the error was introduced:

npm i -D -E postcss@6.0.14

CHANGELOG.md of PostCSS

6.0.16
Fix warning text (by Michael Keller).
6.0.15
Add warning about missed from option on process().then() call.
Add IE 10 support.

https://github.com/postcss/postcss/blame/master/lib/lazy-result.es6#L192

In the: package.json of ionic-app-scripts
this "postcss": "^6.0.13" installs also the new patch 6.0.15, which has the new warning.

thezakman · 2018-01-18T01:23:26Z

was this fixed?

StefanRein · 2018-01-18T07:07:56Z

@thezakman Not yet. The issue is open. The warning still exists.

AlbenJohn · 2018-01-20T10:53:40Z

find sass.js file. under /node_modules/@ionic/app-scripts/dist/sass.js.
find var postcssOptions and add from: undefined,

Finally should be like this:
var postcssOptions = {
from: undefined,
to: path_1.basename(sassConfig.outFile),
map: autoPrefixerMapOptions
};

StefanRein · 2018-01-20T10:57:10Z

ionic-team/ionic-app-scripts#1364 (comment)

You should not modify any files in the node_modules directory. If you update packages, reinstall them, installing them in a new project, any modifications are gone.

Please use the suggestion above for now.

hanifmazlan · 2018-02-01T03:01:21Z

i solve problem using this ( thanx also to @AlbenJohn)
https://stackoverflow.com/a/48315414

polfernandez · 2018-02-15T11:17:11Z

Any fix on this?

johnroper100 · 2018-02-24T19:18:41Z

While it is not breaking my builds, this warning appears for me as well. This is a one line quick fix, is there already a pull request for this?

StefanRein · 2018-02-25T08:42:21Z

@johnroper100 Yes and it was approved (it's linked above).

Edit: The map files are working correctly.

JosepAlacid · 2018-03-19T10:23:10Z

This warning still apears.

Veyhunk · 2018-04-06T17:56:36Z

This warning still apears.

ajmalhassan · 2018-04-14T04:26:34Z

This warning still appears.

nickthakkar · 2018-04-16T15:23:20Z

@JosepAlacid @Veyhunk @ajmalhassan try using the fix that @hanifmazlan suggested. It actually works for me. https://stackoverflow.com/a/48315414/3288671

bobbino · 2018-04-18T11:37:54Z

True, it does, but as @StefanRein suggests, it's still miles away from any production-like environment. better to avoid node_modules. Still waiting for a proper fix. Any updates from the Ionic Team?

kirillgroshkov · 2018-05-11T13:57:23Z

This warning still apears.

ElissandroMendes · 2018-05-14T13:56:57Z

This warning still apears.

thorizer · 2018-05-20T01:48:52Z

This warning still apears.

th3erik · 2018-05-20T11:01:00Z

This warning still apears.

Ruslancic · 2018-05-21T01:21:37Z

This warning still apears.

StefanRein · 2018-05-31T13:14:30Z

Please stop. This is not helping in any way.
This is an open source project. If the people have time, they will fix it.
There are solutions mentioned above to fix the warning.

Though I don't want to speak on behalf of the IONIC team, I feel like something needs to be said here:

This issue doesn't get closer to resolution in proportion to the number of comments that say "This warning still appears." / "Any update on this" / "Any fix for this" / etc.

If you're stuck, read the thread – there's a workaround, which as the name suggests, will get you working. If you'd like to get updates on this, save some keystrokes by clicking the 'Subscribe' button – that allows you to receive updates in your e-mail. You'll also receive updates every time someone comments "Any update on this?", so please, don't be that person.

jhonnhenry · 2018-06-08T17:48:25Z

This warning still apears.

emcniece · 2018-06-08T17:56:44Z

@jhonnhenry dude, seriously? We know - the issue is still open. Just subscribe and be on your way...

If you insist on leaving a comment, help us be more productive by providing some insight like what versions you're running.

islameg26 · 2018-08-05T22:22:11Z

I found this Workaround

First find sass.js file under /node_modules/@ionic/app-scripts/dist/sass.js. inside search var postcssOptions and add from: undefined,

Finally should be like this:

var postcssOptions = {
from: undefined,
to: path_1.basename(sassConfig.outFile),
map: autoPrefixerMapOptions
};

https://stackoverflow.com/questions/48162738/without-from-option-postcss-could-generate-wrong-source-map-or-do-not-find-bro

This removes following warning: Remove warning: Without `from` option PostCSS could generate wrong source map or do not find Browserslist config. Set it to CSS file path or to `undefined` to prevent this warning `from: the input file name (most runners set it automatically).` Source: https://github.com/postcss/postcss Fixes #1359 #13763 #1359 ionic-team/ionic-framework#13763

trsh · 2018-11-30T10:21:00Z

Still got his warning. Jumping into node_modules is no serious ::/

ionitron-bot · 2018-12-08T09:24:45Z

Thanks for the issue! We have moved the source code and issues for Ionic 3 into a separate repository. I am moving this issue to the repository for Ionic 3. Please track this issue over there.

Thank you for using Ionic!

ionitron-bot · 2018-12-08T09:24:47Z

Issue moved to: ionic-team/ionic-v3#883

StefanRein mentioned this issue Jan 9, 2018

Remove PostCSS warning ionic-team/ionic-app-scripts#1364

Merged

jgerstle mentioned this issue Jan 10, 2018

[1.3.12] Wrong lint warning when import is only use as declaration ionic-team/ionic-app-scripts#1074

Open

ghost mentioned this issue Feb 6, 2018

scrollbars not need but displayed, e.g. on ActionSheet, on Windows Chrome #13966

Closed

paulstelzer added the ionitron: v3 moves the issue to the ionic-v3 repository label Dec 8, 2018

ionitron-bot bot mentioned this issue Dec 8, 2018

PostCSS could generate wrong source map error ionic-team/ionic-v3#883

Open

ionitron-bot bot closed this as completed Dec 8, 2018

ionitron-bot bot locked and limited conversation to collaborators Dec 8, 2018

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

PostCSS could generate wrong source map error #13763

PostCSS could generate wrong source map error #13763

mburger81 commented Jan 4, 2018

StefanRein commented Jan 9, 2018 •

edited

StefanRein commented Jan 9, 2018 •

edited

thezakman commented Jan 18, 2018

StefanRein commented Jan 18, 2018

AlbenJohn commented Jan 20, 2018

StefanRein commented Jan 20, 2018 •

edited

hanifmazlan commented Feb 1, 2018

polfernandez commented Feb 15, 2018

johnroper100 commented Feb 24, 2018

StefanRein commented Feb 25, 2018 •

edited

JosepAlacid commented Mar 19, 2018

Veyhunk commented Apr 6, 2018

ajmalhassan commented Apr 14, 2018

nickthakkar commented Apr 16, 2018

bobbino commented Apr 18, 2018

kirillgroshkov commented May 11, 2018

ElissandroMendes commented May 14, 2018

thorizer commented May 20, 2018

th3erik commented May 20, 2018

Ruslancic commented May 21, 2018

StefanRein commented May 31, 2018 •

edited

jhonnhenry commented Jun 8, 2018

emcniece commented Jun 8, 2018 •

edited

islameg26 commented Aug 5, 2018

trsh commented Nov 30, 2018

ionitron-bot bot commented Dec 8, 2018

ionitron-bot bot commented Dec 8, 2018

PostCSS could generate wrong source map error #13763

PostCSS could generate wrong source map error #13763

Comments

mburger81 commented Jan 4, 2018

StefanRein commented Jan 9, 2018 • edited

StefanRein commented Jan 9, 2018 • edited

thezakman commented Jan 18, 2018

StefanRein commented Jan 18, 2018

AlbenJohn commented Jan 20, 2018

StefanRein commented Jan 20, 2018 • edited

hanifmazlan commented Feb 1, 2018

polfernandez commented Feb 15, 2018

johnroper100 commented Feb 24, 2018

StefanRein commented Feb 25, 2018 • edited

JosepAlacid commented Mar 19, 2018

Veyhunk commented Apr 6, 2018

ajmalhassan commented Apr 14, 2018

nickthakkar commented Apr 16, 2018

bobbino commented Apr 18, 2018

kirillgroshkov commented May 11, 2018

ElissandroMendes commented May 14, 2018

thorizer commented May 20, 2018

th3erik commented May 20, 2018

Ruslancic commented May 21, 2018

StefanRein commented May 31, 2018 • edited

jhonnhenry commented Jun 8, 2018

emcniece commented Jun 8, 2018 • edited

islameg26 commented Aug 5, 2018

trsh commented Nov 30, 2018

ionitron-bot bot commented Dec 8, 2018

ionitron-bot bot commented Dec 8, 2018

StefanRein commented Jan 9, 2018 •

edited

StefanRein commented Jan 9, 2018 •

edited

StefanRein commented Jan 20, 2018 •

edited

StefanRein commented Feb 25, 2018 •

edited

StefanRein commented May 31, 2018 •

edited

emcniece commented Jun 8, 2018 •

edited