configure.ac          |   12 ++++++------
 static/README.wasm.md |    2 +-
 2 files changed, 7 insertions(+), 7 deletions(-)

New commits:
commit 3918e3e22ebf76a9f9ee2ddce41083c3db2aa6de
Author:     Moritz Duge <moritz.d...@allotropia.de>
AuthorDate: Tue Mar 11 22:47:14 2025 +0100
Commit:     Moritz Duge <moritz.d...@allotropia.de>
CommitDate: Wed Mar 12 18:37:20 2025 +0100

    WASM clarified --with-wasm-module
    
    Option is for WASM only, so shouldn't be named --with-main-module.
    
    Also the option's default value changed since it was introduced, so the
    documentation became outdated. See commits:
    
    fe678ffc6f6f0e851c3b7b7b0c8a1b8219b819a2 Emscripten: No need to pass 
--with-main-module into build configure
    437b4d802ccec2eb1395a4e4137a4a0af039de5a Fix default value of Emscripten 
--with-main-module
    8aef8c3a1451b2fce37a21870a004855822ee335 Make it possible to run either 
Writer or Calc in the same Emscripten thing
    97ee126d69859d01abfbdb55450d27afb71bf9b1 WASM add Calc as optional build 
result, make it build & run
    
    Change-Id: I29aeeb9a840ec07e46cbf153c535b3fca9d88cd1
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/182798
    Tested-by: Jenkins
    Reviewed-by: Moritz Duge <moritz.d...@allotropia.de>

diff --git a/configure.ac b/configure.ac
index 62280b34fc63..37d1a5064370 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2170,11 +2170,11 @@ AC_ARG_ENABLE(wasm-strip,
         [Strip the static build like for WASM/emscripten platform.]),
 ,)
 
-AC_ARG_WITH(main-module,
-    AS_HELP_STRING([--with-main-module=<writer/calc/impress>],
+AC_ARG_WITH(wasm-module,
+    AS_HELP_STRING([--with-wasm-module=<writer/calc/impress>],
         [Specify which main module to build for wasm.
-        Default value is 'writer'.]),
-, [with_main_module='calc writer'])
+        Default value is 'calc writer'.]),
+, [with_wasm_module='calc writer'])
 # Don't include impress by default. Debug builds become too large for Chromium
 # with 'calc impress writer' enabled. (buffer source exceeds maximum size)
 
@@ -4178,7 +4178,7 @@ if test "$cross_compiling" = "yes"; then
         ENABLE_WASM_STRIP_BASIC_DRAW_MATH_IMPRESS=TRUE
         ENABLE_WASM_STRIP_CALC=TRUE
         ENABLE_WASM_STRIP_WRITER=TRUE
-        for i in $with_main_module; do
+        for i in $with_wasm_module; do
             case "$i" in
             calc)
                 ENABLE_WASM_STRIP_ACCESSIBILITY=
@@ -4192,7 +4192,7 @@ if test "$cross_compiling" = "yes"; then
                 ENABLE_WASM_STRIP_BASIC_DRAW_MATH_IMPRESS=
                 ;;
             *)
-                AC_MSG_ERROR([Unknown --with-main-module "$i"])
+                AC_MSG_ERROR([Unknown --with-wasm-module "$i"])
             esac
         done
         if test "$ENABLE_WASM_STRIP_BASIC_DRAW_MATH_IMPRESS" = TRUE; then
diff --git a/static/README.wasm.md b/static/README.wasm.md
index 76a956219c71..4cab873c7a0c 100644
--- a/static/README.wasm.md
+++ b/static/README.wasm.md
@@ -433,7 +433,7 @@ For instance, this autogen.input works for me:
 `--disable-crashdump`
 `--host=wasm32-local-emscripten`
 `--disable-gui`
-`--with-main-module=writer`
+`--with-wasm-module=writer`
 `--with-package-format=emscripten`
 
 For building LO core for use in COWASM, it is known to work to use

Reply via email to