From 80c4809aee19901b4a63398a3cbe370756dd0e98 Mon Sep 17 00:00:00 2001 From: Andreas Hocevar Date: Fri, 17 Jul 2015 18:07:39 +0200 Subject: [PATCH] Do not provide an AMD environment to ol.ext modules This fixes issues with ol.source.Vector being unable to use ol.ext.rbush, but it also means that potential future external modules without node module loader support will not work. If we ever depend on such a module, we can replace "var define;" with a minimal AMD define shim. --- tasks/build-ext.js | 1 + 1 file changed, 1 insertion(+) diff --git a/tasks/build-ext.js b/tasks/build-ext.js index 120a32ed27..b750882a94 100644 --- a/tasks/build-ext.js +++ b/tasks/build-ext.js @@ -50,6 +50,7 @@ function wrapModule(mod, callback) { '(function() {\n' + 'var exports = {};\n' + 'var module = {exports: exports};\n' + + 'var define;\n' + '/**\n' + ' * @fileoverview\n' + ' * @suppress {accessControls, ambiguousFunctionDecl, ' +