build: switch from libsass-python to node-sass

package.json

@@ -43,6 +43,7 @@
     "jwt-decode": "^2.2.0",
     "moment": "2.20.1",
     "moment-timezone": "0.5.14",
+    "node-sass": "4.14.1",
     "picturefill": "3.0.2",
     "popper.js": "1.12.9",
     "prop-types": "15.6.0",

pavelib/assets.py

@@ -509,11 +509,6 @@ def _compile_sass(system, theme, debug, force, timing_info):
     :param timing_info: list variable to keep track of timing for sass compilation
     """
 
-    # Note: import sass only when it is needed and not at the top of the file.
-    # This allows other paver commands to operate even without libsass being
-    # installed. In particular, this allows the install_prereqs command to be
-    # used to install the dependency.
-    import sass
     if system == "common":
         sass_dirs = get_common_sass_directories()
     else:
@@ -555,11 +550,14 @@ def _compile_sass(system, theme, debug, force, timing_info):
                 sass_dir=sass_source_dir,
             ))
         else:
-            sass.compile(
-                dirname=(sass_source_dir, css_dir),
-                include_paths=COMMON_LOOKUP_PATHS + lookup_paths,
-                source_comments=source_comments,
-                output_style=output_style,
+            include_paths = COMMON_LOOKUP_PATHS + lookup_paths
+            sh(
+                "node-sass"
+                + f" --output-style {output_style}"
+                + (" --source-comments"  if source_comments else "")
+                + " ".join(f"--include-path {path}" for path in include_paths)
+                + sass_source_dir  # input dir
+                + css_dir  # output dir
             )
 
         # For Sass files without explicit RTL versions, generate

requirements/edx/base.txt

@@ -658,9 +658,7 @@ learner-pathway-progress==1.3.3
 levenshtein==0.20.9
     # via python-levenshtein
 libsass==0.10.0
-    # via
-    #   -r requirements/edx/paver.txt
-    #   ora2
+    # via ora2
 loremipsum==1.0.5
     # via ora2
 lti-consumer-xblock==7.2.1

requirements/edx/paver.in

@@ -12,7 +12,6 @@
 
 edx-opaque-keys                     # Create and introspect course and xblock identities
 lazy                                # Lazily-evaluated attributes for Python objects
-libsass==0.10.0                     # Python bindings for the LibSass CSS compiler
 markupsafe                          # XML/HTML/XHTML Markup safe strings
 mock                                # Stub out code with mock objects and make assertions about how they have been used
 path                                # Easier manipulation of filesystem paths

requirements/edx/paver.txt

@@ -16,8 +16,6 @@ idna==3.4
     # via requests
 lazy==1.5
     # via -r requirements/edx/paver.in
-libsass==0.10.0
-    # via -r requirements/edx/paver.in
 markupsafe==2.1.2
     # via -r requirements/edx/paver.in
 mock==5.0.1
@@ -41,7 +39,6 @@ requests==2.28.2
     # via -r requirements/edx/paver.in
 six==1.16.0
     # via
-    #   libsass
     #   paver
     #   python-memcached
 stevedore==4.1.1

requirements/edx/testing.txt

@@ -63,7 +63,6 @@ attrs==22.2.0
     #   jsonschema
     #   lti-consumer-xblock
     #   openedx-events
-    #   outcome
     #   pytest
 babel==2.11.0
     # via