serverless-bundle icon indicating copy to clipboard operation
serverless-bundle copied to clipboard

Node-sass new dependancy requires python

Open WarlaxZ opened this issue 4 years ago • 5 comments

I get why you are trying to add convenience by adding support for css and sass files - however this now means the build machine needs python on, which was never a requirement before. Is there a way to disable this requirement?

WarlaxZ avatar Mar 29 '21 09:03 WarlaxZ

and it is failing in many cicd pipelines because of this

hansheng654 avatar Apr 06 '21 02:04 hansheng654

I recall we had removed this in a previous release, does it not work?

https://github.com/AnomalyInnovations/serverless-bundle/pull/188

jayair avatar Apr 09 '21 01:04 jayair

image This fails at npm 7 because it installs all peer dependencies automatically. when doing so, It is using node-gyp which is a requirement from node-sass which is a requirement from serverless-bundle, which runs "sh -c node-gyp rebuild" in cicd, then it fails at not finding a package called pangocairo which is a dependency for canvas. How on earth does building serverless code ultimately ended up requiring some UI Rendering library for canvas???

hansheng654 avatar Apr 09 '21 06:04 hansheng654

How on earth does building serverless code ultimately ended up requiring some UI Rendering library for canvas???

This doesn't help me fix your issue.

The community added this to support including sass files. People use serverless-bundle to bundle quite a few different types of files.

Can you create a repo and steps to reproduce?

jayair avatar Apr 09 '21 17:04 jayair

This is causing an error on Node 16+ envs. Just try to build the package on it

fabiosenracorrea avatar Sep 16 '22 16:09 fabiosenracorrea