Re-introduce build config to version info reported by LOK

... as an opt-in --with-buildconfig-recorded configure option.
This allows to have the data in the admin console, as implemented in
commit cbfac11330 (Send build config
(configure options) in LOKit version info JSON, 2022-11-07), when
reprobuilds are not required. The default is no build config, which
is compatible with reprobuilds.

This reverts commit 389def8718.

Change-Id: I7f0be489a1c82268d0ca38cb761843c9d432a14b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/159344
Tested-by: Jenkins
Reviewed-by: Thorsten Behrens <thorsten.behrens@allotropia.de>
This commit is contained in:
Mike Kaganski
2023-11-13 10:57:02 +01:00
parent 285a3e8d8b
commit 93b4928ddd
3 changed files with 35 additions and 2 deletions
+9
View File
@@ -0,0 +1,9 @@
/* Options passed to configure script */
#ifndef CONFIG_BUILDCONFIG_H
#define CONFIG_BUILDCONFIG_H
#define BUILDCONFIG ""
#define BUILDCONFIG_RECORDED 0
#endif
+20
View File
@@ -2862,6 +2862,25 @@ AC_ARG_WITH(coredumpctl,
[Use coredumpctl (together with systemd-run) to retrieve core dumps of crashing test
processes.]))
AC_ARG_WITH(buildconfig-recorded,
AS_HELP_STRING([--with-buildconfig-recorded],
[Put build config into version info reported by LOK. Incompatible with reproducible builds.]),
)
AC_MSG_CHECKING([whether to record build config])
if test -z "$with_buildconfig_recorded"; then
with_buildconfig_recorded=no
fi
if test "$with_buildconfig_recorded" = no; then
AC_MSG_RESULT([no])
else
AC_MSG_RESULT([yes])
# replace backslashes, to get a valid c++ string
config_args=$(echo $ac_configure_args | tr '\\' '/')
AC_DEFINE_UNQUOTED([BUILDCONFIG],[["$config_args"]],[Options passed to configure script])
AC_DEFINE([BUILDCONFIG_RECORDED],[1],[Options passed to configure script])
fi
dnl ===================================================================
dnl Do we want to use pre-build binary tarball for recompile
dnl ===================================================================
@@ -15070,6 +15089,7 @@ AC_CONFIG_FILES([config_host.mk
vs-code.code-workspace.template:.vscode/vs-code-template.code-workspace.in])
AC_CONFIG_HEADERS([config_host/config_atspi.h])
AC_CONFIG_HEADERS([config_host/config_buildconfig.h])
AC_CONFIG_HEADERS([config_host/config_buildid.h])
AC_CONFIG_HEADERS([config_host/config_box2d.h])
AC_CONFIG_HEADERS([config_host/config_clang.h])
+6 -2
View File
@@ -12,6 +12,7 @@
#include <svx/sdr/contact/viewcontact.hxx>
#include <svx/svdpage.hxx>
#include <svx/svdpagv.hxx>
#include <config_buildconfig.h>
#include <config_cairo_rgba.h>
#include <config_features.h>
@@ -7242,8 +7243,11 @@ static char* lo_getVersionInfo(SAL_UNUSED_PARAMETER LibreOfficeKit* /*pThis*/)
"\"ProductName\": \"%PRODUCTNAME\", "
"\"ProductVersion\": \"%PRODUCTVERSION\", "
"\"ProductExtension\": \"%PRODUCTEXTENSION\", "
"\"BuildId\": \"%BUILDID\" "
"}"_ustr));
"\"BuildId\": \"%BUILDID\""
#if BUILDCONFIG_RECORDED
", \"BuildConfig\": \"" BUILDCONFIG "\""
#endif
" }"_ustr));
}
static void aBasicErrorFunc(const OUString& rError, const OUString& rAction)