From 28155a41475ac69f048d8f497f6dded283c28f99 Mon Sep 17 00:00:00 2001
From: Bruno Binet <bruno.binet@gmail.com>
Date: Thu, 14 Mar 2013 15:14:15 +0100
Subject: [PATCH] Fix check for missing goog.require directives

Take this example:
```
goog.require('ol.projection.addCommonProjections');

ol.projection.addCommonProjections();
```

Without this patch, the build_check_requires_timestamp target would report
that ``ol.projection`` is missing.
This is because ``ol.projection.addCommonProjections();`` would match both
``ol.projection.addCommonProjections and ``ol.projection`` provide
directives.
---
 build.py | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/build.py b/build.py
index c322d3fd44..c773041edb 100755
--- a/build.py
+++ b/build.py
@@ -311,7 +311,7 @@ def build_check_requires_timestamp(t):
     all_provides.discard('ol')
     all_provides.discard('ol.Map')
     all_provides.discard('ol.MapProperty')
-    provide_res = dict((provide, re.compile(r'\b%s\b' % (re.escape(provide)),)) for provide in all_provides)
+    provide_res = [(provide, re.compile(r'\b%s\b' % (re.escape(provide)),)) for provide in sorted(all_provides, reverse=True)]
     missing_count = 0
     for filename in sorted(t.dependencies):
         if filename in INTERNAL_SRC or filename in EXTERNAL_SRC:
@@ -328,8 +328,9 @@ def build_check_requires_timestamp(t):
             if m:
                 requires.add(m.group(1))
                 continue
-            for provide, provide_re in provide_res.iteritems():
+            for provide, provide_re in provide_res:
                 if provide_re.search(line):
+                    line = line.replace(provide, '')
                     uses.add(provide)
         if filename == 'src/ol/renderer/layerrenderer.js':
             uses.discard('ol.renderer.Map')