-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
[INSTALL] LESS errors on gulp build on fresh install #6470
Comments
This is either causing or related to another problem I'm having: In previous builds, after running Why would |
FWIW, I get the same as your error # 2 using semantic ui 2.3.3 when running the build task. Oddly enough, a fresh install is fine. Though, it means every time I want to change any tiny little thing I have to uninstall and reinstall semantic :-/ UPDATE: for me, this only seems to happen in a folder where I already have a gulpfile. Workaround: just put semantic into a clean folder. It's annoying, but workable. |
For anyone else having this problem, @sciyoshi pointed out the fix. |
This happened to me also after gulp-less got updated to 4.0.0. It seems that escaping things with I tried this on the cases with the errors and succeeded to build again after "unescaping" variables in about 4 or 5 places. Only calc() instances with a negative variable inside (as shown in the error) seem to be affected.. |
Dang, didn't see this PR until now: #6512 That should take care of it :) |
This will be fixed with #6512 Please see my note on that PR |
Steps
% cd semantic
% gulp build
Result
Two errors:
[Error: variable @googleProtocol is undefined in file /Users/n/webroot/semantic/src/definitions/globals/site.less line no. 25]
[Error: Operation on an invalid type in file /Users/n/webroot/semantic/src/themes/default/elements/label.variables line no. 133]
Complete Output
Versions
Local:
Semantic UI 2.3.2
gulp 3.9.1
gulp-less 4.0.0
Global:
Node 10.5.0
NPM 6.1.0
LESS 3.5.3
The text was updated successfully, but these errors were encountered: