From c0638ca9d3f8c0c92a8909b3e98dcac351385374 Mon Sep 17 00:00:00 2001
From: Calen Pennington <cale@edx.org>
Date: Tue, 21 Nov 2017 14:57:46 -0500
Subject: [PATCH 1/2] Use file watches rather that polling for asset watching

---
 pavelib/assets.py                  |  7 +++----
 pavelib/paver_tests/test_assets.py | 10 +++++-----
 2 files changed, 8 insertions(+), 9 deletions(-)

diff --git a/pavelib/assets.py b/pavelib/assets.py
index 3009844fbf..7824c880c6 100644
--- a/pavelib/assets.py
+++ b/pavelib/assets.py
@@ -16,7 +16,7 @@ from paver import tasks
 from paver.easy import call_task, cmdopts, consume_args, needs, no_help, path, sh, task
 from watchdog.events import PatternMatchingEventHandler
 from watchdog.observers.api import DEFAULT_OBSERVER_TIMEOUT
-from watchdog.observers.polling import PollingObserver
+from watchdog.observers import Observer
 
 from openedx.core.djangoapps.theming.paver_helpers import get_theme_paths
 
@@ -792,7 +792,7 @@ def execute_webpack_watch(settings=None):
     # from Watchdog like the other watchers do.
     run_background_process(
         'STATIC_ROOT_LMS={static_root_lms} STATIC_ROOT_CMS={static_root_cms} $(npm bin)/webpack {options}'.format(
-            options='--watch --watch-poll=200 --config={config_path}'.format(
+            options='--watch --config={config_path}'.format(
                 config_path=Env.get_django_setting("WEBPACK_CONFIG_PATH", "lms", settings=settings)
             ),
             static_root_lms=Env.get_django_setting("STATIC_ROOT", "lms", settings=settings),
@@ -852,7 +852,6 @@ def watch_assets(options):
     themes = get_parsed_option(options, 'themes')
     theme_dirs = get_parsed_option(options, 'theme_dirs', [])
 
-    # wait comes in as a list of strings, define the default value similarly for convenience.
     default_wait = [unicode(DEFAULT_OBSERVER_TIMEOUT)]
     wait = float(get_parsed_option(options, 'wait', default_wait)[0])
 
@@ -863,7 +862,7 @@ def watch_assets(options):
         theme_dirs = [path(_dir) for _dir in theme_dirs]
 
     sass_directories = get_watcher_dirs(theme_dirs, themes)
-    observer = PollingObserver(timeout=wait)
+    observer = Observer(timeout=wait)
 
     CoffeeScriptWatcher().register(observer)
     SassWatcher().register(observer, sass_directories)
diff --git a/pavelib/paver_tests/test_assets.py b/pavelib/paver_tests/test_assets.py
index ee76f76bce..b14370b44d 100644
--- a/pavelib/paver_tests/test_assets.py
+++ b/pavelib/paver_tests/test_assets.py
@@ -6,7 +6,7 @@ from unittest import TestCase
 import ddt
 from mock import patch
 from paver.easy import call_task, path
-from watchdog.observers.polling import PollingObserver
+from watchdog.observers import Observer
 
 from pavelib.assets import COLLECTSTATIC_LOG_DIR_ARG, collect_assets
 
@@ -221,7 +221,7 @@ class TestPaverWatchAssetTasks(TestCase):
         Test the "compile_sass" task.
         """
         with patch('pavelib.assets.SassWatcher.register') as mock_register:
-            with patch('pavelib.assets.PollingObserver.start'):
+            with patch('pavelib.assets.Observer.start'):
                 with patch('pavelib.assets.execute_webpack_watch') as mock_webpack:
                     call_task(
                         'pavelib.assets.watch_assets',
@@ -232,7 +232,7 @@ class TestPaverWatchAssetTasks(TestCase):
 
                     sass_watcher_args = mock_register.call_args_list[0][0]
 
-                    self.assertIsInstance(sass_watcher_args[0], PollingObserver)
+                    self.assertIsInstance(sass_watcher_args[0], Observer)
                     self.assertIsInstance(sass_watcher_args[1], list)
                     self.assertItemsEqual(sass_watcher_args[1], self.expected_sass_directories)
 
@@ -248,7 +248,7 @@ class TestPaverWatchAssetTasks(TestCase):
         ])
 
         with patch('pavelib.assets.SassWatcher.register') as mock_register:
-            with patch('pavelib.assets.PollingObserver.start'):
+            with patch('pavelib.assets.Observer.start'):
                 with patch('pavelib.assets.execute_webpack_watch') as mock_webpack:
                     call_task(
                         'pavelib.assets.watch_assets',
@@ -262,7 +262,7 @@ class TestPaverWatchAssetTasks(TestCase):
                     self.assertEqual(mock_webpack.call_count, 1)
 
                     sass_watcher_args = mock_register.call_args_list[0][0]
-                    self.assertIsInstance(sass_watcher_args[0], PollingObserver)
+                    self.assertIsInstance(sass_watcher_args[0], Observer)
                     self.assertIsInstance(sass_watcher_args[1], list)
                     self.assertItemsEqual(sass_watcher_args[1], self.expected_sass_directories)
 

From f4b62b066cd908966a093d3361ee00bf1543a2dc Mon Sep 17 00:00:00 2001
From: Calen Pennington <cale@edx.org>
Date: Tue, 21 Nov 2017 16:16:20 -0500
Subject: [PATCH 2/2] Don't have webpack watch node_modules or .git for changes

---
 webpack.dev.config.js | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/webpack.dev.config.js b/webpack.dev.config.js
index 6f52e8309d..193647682c 100644
--- a/webpack.dev.config.js
+++ b/webpack.dev.config.js
@@ -54,5 +54,8 @@ module.exports = Merge.smart(commonConfig, {
                 ]
             }
         ]
+    },
+    watchOptions: {
+        ignored: [/node_modules/, /\.git/]
     }
 });