Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

Fails to compile arithmetic on result of map-get #1966

Closed
akiellor opened this issue May 13, 2017 · 6 comments
Closed

Fails to compile arithmetic on result of map-get #1966

akiellor opened this issue May 13, 2017 · 6 comments

Comments

@akiellor
Copy link

  • NPM version: 3.10.10
  • Node version: v6.10.0
  • Node Process:
{ http_parser: '2.7.0',
  node: '6.10.0',
  v8: '5.1.281.93',
  uv: '1.9.1',
  zlib: '1.2.8',
  ares: '1.10.1-DEV',
  icu: '58.2',
  modules: '48',
  openssl: '1.0.2k' }
  • Node Platform: linux
  • Node architecture: x64
  • node-sass version:
node-sass	4.5.2	(Wrapper)	[JavaScript]
libsass  	3.5.0.beta.2	(Sass Compiler)	[C/C++]
  • npm node-sass versions (npm ls node-sass): 4.5.2

When encountering a syntax, or compilation issue:

If you delete this text without following it, your issue will be closed.

Minimal reproducible scenario:

root@f55809ebd5c9:/usr/src/app# node-sass <<EOS
\$map-spaces: (
  xs: 3rem / 16
);
\$var: map-get(\$map-spaces, 'xs') * 5;

.size {
  height: \$var;
}
EOS
{
  "status": 1,
  "file": "stdin",
  "line": 4,
  "column": 7,
  "message": "Undefined operation: \"3rem/16 times 5\".",
  "formatted": "Error: Undefined operation: \"3rem/16 times 5\".\n        on line 4 of stdin\n>> $var: map-get($map-spaces, 'xs') * 5;\n   ------^\n"
}

This example works fine in https://www.sassmeister.com/ and outputs the following:

.size {
  height: 0.9375rem;
}
@nschonni
Copy link
Contributor

Can you file an issue with https://github.com/sass/libsass/issues and link it back here, thanks!

@xzyfer
Copy link
Contributor

xzyfer commented May 16, 2017

This is already fixed on LibSass master and will be in node-sass on the next bump.

@nschonni nschonni added this to the 4.6 milestone May 16, 2017
@saper saper modified the milestones: 4.6, libsass-3.4.6 Nov 12, 2017
@gauthierm
Copy link

What's the ETA for this fix? libsass 3.5.0.beta.3 was released in February and node-sass is still using 3.5.0.beta.2 containing the bug.

@gauthierm
Copy link

sass/libsass#2309 is the fixed issue in libsass.

@gauthierm
Copy link

#1935 is a duplicate of this issue.

@nschonni
Copy link
Contributor

Duplicate of #1935

@nschonni nschonni marked this as a duplicate of #1935 Dec 20, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants