Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

m css yields: (node:25556) Warning: Accessing non-existent property 'lineno' #1052

Closed
RogerHaase opened this issue Jul 23, 2020 · 0 comments
Closed
Labels

Comments

@RogerHaase
Copy link
Member

Upgrading Node to version 14 results in following harmless warning messages:

    (moin-venv-python) C:\git\moin>m css
    Running Stylus to update Modernized theme CSS files...
    (node:25556) Warning: Accessing non-existent property 'lineno' of module exports inside circular dependency
    (Use `node --trace-warnings ...` to show where the warning was created)
    (node:25556) Warning: Accessing non-existent property 'filename' of module exports inside circular dependency
    (node:25556) Warning: Accessing non-existent property 'lineno' of module exports inside circular dependency
    (node:25556) Warning: Accessing non-existent property 'filename' of module exports inside circular dependency
    Success: Modernized CSS files updated.

    Starting directory is C:\git\moin\src\moin\themes\modernized\static\css
    Running lessc to update Basic theme CSS files...
    Success: Basic theme CSS files updated.

The modernized css file is created correctly.

Node nodejs/node#32987 blames it on Stylus stylus/stylus#2534.

Stylus says The issue was fixed by #2538. Current stylus 0.54.8 does not display these warnings anymore.

But we use Stylus 42 per docs:

sudo npm install stylus@0.42.2 -g  # Windows: npm install stylus@0.42.2 -g
# we need 0.42.2 because with more recent versions, --compress compresses
# complete output to 1 line (0.42.2 compresses to 1 line per rule)
stylus -V  # show version number to prove it works

Stylus has been PITA, best alternative is to change Modernized theme to use CSS Variables.

Alternatives:

  • use Node version 13.
  • ignore the harmless messages
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant