loleaflet/Makefile.am |   10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

New commits:
commit 684a0d2ef43d17a2e939ec021caf6519a40c0cce
Author:     Henry Castro <[email protected]>
AuthorDate: Wed May 27 11:31:03 2020 -0400
Commit:     Henry Castro <[email protected]>
CommitDate: Fri May 29 04:22:50 2020 +0200

    loleaflet: makefile: replace ENABLE_DEBUG var
    
    Now, it does not depend on ./configure script
    it is configurable to set BUNDLE=DEBUG or RELEASE (i.e. Android platform)
    
    Change-Id: Id43f156a05ea22f74334e7fdbe60d53b8b5a2891
    Reviewed-on: https://gerrit.libreoffice.org/c/online/+/94982
    Tested-by: Jenkins
    Tested-by: Jenkins CollaboraOffice <[email protected]>
    Reviewed-by: Henry Castro <[email protected]>

diff --git a/loleaflet/Makefile.am b/loleaflet/Makefile.am
index 4f6c29a19..47fd0c4c5 100644
--- a/loleaflet/Makefile.am
+++ b/loleaflet/Makefile.am
@@ -2,6 +2,8 @@ MAKEFLAGS = --no-builtin-rules
 CTAGS = ctags
 
 L10N_PO = $(wildcard $(srcdir)/po/*.po)
+BUNDLE ?= $(if $(filter true,$(ENABLE_DEBUG)),DEBUG,RELEASE)
+IS_DEBUG = $(if $(filter DEBUG,$(BUNDLE)),true,)
 
 if ENABLE_ANDROIDAPP
 DIST_FOLDER = $(abs_top_srcdir)/android/lib/src/main/assets/dist
@@ -141,7 +143,7 @@ LOLEAFLET_CSS_M4 = $(strip $(foreach 
file,$(LOLEAFLET_CSS),$(notdir $(file))))
 $(eval $(call file_targets,$(LOLEAFLET_CSS)))
 
 NODE_MODULES_JS =\
-       $(if $(filter 
true,$(ENABLE_DEBUG)),node_modules/hammerjs/hammer.js,node_modules/hammerjs/hammer.min.js)
 \
+       $(if 
$(IS_DEBUG),node_modules/hammerjs/hammer.js,node_modules/hammerjs/hammer.min.js)
 \
        node_modules/jquery/dist/jquery.js \
        node_modules/jquery-mousewheel/jquery.mousewheel.js \
        node_modules/jquery-contextmenu/dist/jquery.contextMenu.js \
@@ -326,8 +328,6 @@ INTERMEDIATE_DIR ?= $(abs_builddir)/build
 
 EXTRA_DIST = $(shell find . -type f -not -path './.git/*' | sed 's/.\///')
 
-BUNDLE ?= $(if $(filter true,$(ENABLE_DEBUG)),DEBUG,RELEASE)
-
 define bundle_loleaflet
        $(if $(filter DEBUG,$(1)),\
                @touch $@,
@@ -414,7 +414,7 @@ endif
 
 $(DIST_FOLDER)/admin-bundle.js: $(LOLEAFLET_ADMIN_DST) \
        $(INTERMEDIATE_DIR)/admin-src.js
-       @NODE_PATH=$(abs_builddir)/node_modules:$(INTERMEDIATE_DIR) $(NODE) 
node_modules/browserify/bin/cmd.js -g browserify-css $(if $(filter 
true,$(ENABLE_DEBUG)),--debug,-g uglifyify) -o $@ $(srcdir)/admin/main-admin.js
+       @NODE_PATH=$(abs_builddir)/node_modules:$(INTERMEDIATE_DIR) $(NODE) 
node_modules/browserify/bin/cmd.js -g browserify-css $(if 
$(IS_DEBUG),--debug,-g uglifyify) -o $@ $(srcdir)/admin/main-admin.js
 
 $(INTERMEDIATE_DIR)/admin-src.js: $(LOLEAFLET_ADMIN_ALL)
        @mkdir -p $(dir $@)
@@ -448,7 +448,7 @@ $(DIST_FOLDER)/loleaflet.html: 
$(srcdir)/html/loleaflet.html.m4 \
        $(DIST_FOLDER)/global.js \
        $(DIST_FOLDER)/bundle.js
        @echo "Generating loleaflet.html..."
-       @m4 -PE -DDEBUG=$(ENABLE_DEBUG) \
+       @m4 -PE -DDEBUG=$(IS_DEBUG) \
                -DIOSAPP=$(ENABLE_IOSAPP) \
                -DGTKAPP=$(ENABLE_GTKAPP) \
                -DANDROIDAPP=$(ENABLE_ANDROIDAPP) \
_______________________________________________
Libreoffice-commits mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to