gnucash maint: Multiple changes pushed
John Ralls
jralls at code.gnucash.org
Sun Feb 7 17:19:37 EST 2021
Updated via https://github.com/Gnucash/gnucash/commit/ba0e4128 (commit)
via https://github.com/Gnucash/gnucash/commit/fe4f9ed6 (commit)
via https://github.com/Gnucash/gnucash/commit/9386b276 (commit)
from https://github.com/Gnucash/gnucash/commit/fa1be8f3 (commit)
commit ba0e412815e712d4daf8100d0bc157f81f361701
Author: John Ralls <jralls at ceridwen.us>
Date: Sun Feb 7 13:45:59 2021 -0800
Bug 798112 - An error occurred while processing mysql ...
A "Feature" of MYSQL is that it allows C-style backslash escapes
in string constants and replaces them with the actual character
(e.g. \n is converted to 0x0a). This causes round trip problems
if the escape is one of the allowed ones and a MYSQL error if it
isn't.
Disable the feature so that MYSQL follows the SQL standard.
commit fe4f9ed64ab84ed19db2c2bf520c29b399ad70e4
Author: John Ralls <jralls at ceridwen.us>
Date: Fri Feb 5 15:05:29 2021 -0800
[test-backend-dbi] Fix copying hidden property.
So that it actually copies it to the *other* root account.
commit 9386b276e5faaf692b33e757f86acbdbf8eb2f5c
Author: Micha Lenk <micha at lenk.info>
Date: Sat Jan 16 15:30:07 2021 +0100
Make prototypes static for autoclear unit test
With the explicit prototypes not marked as static unit test builds on Debian
Buster (using buster-backports) fail with
error: testing::internal::ParamGenerator<TestCase*> gtest_InstantiationAutoClearTestAutoClearTest_EvalGenerator_() was declared extern and later static [-fpermissive]
According to the comment preceding the declaration the only intent of the
explicit prototype (in newer versions of googletest implicitly defined by the
GTEST macros) seems to be to silence a warning which would cause a build
failure on Ubuntu 18.04 when using -Werror.
Builds on Debian unstable seem to build just fine without this explicit
declaration, hence I consider it safe to just drop it. However, builds on
Ubuntu 18.04 then fail (see above).
Making the prototypes as static should make both build environments happy.
Summary of changes:
libgnucash/app-utils/test/test-autoclear.cpp | 6 +++---
libgnucash/backend/dbi/gnc-backend-dbi.cpp | 19 ++++++++++++++-----
libgnucash/backend/dbi/test/test-dbi-stuff.cpp | 2 +-
3 files changed, 18 insertions(+), 9 deletions(-)
More information about the gnucash-patches
mailing list