Browse Source

Bug 25524: Remove --line-regexp option in log4perl_component()

The current grep expression uses -x|--line-regexp, but that causes
the regexp to fail to match since we're not specifying a regexp
that can match a whole line. We can either provide a better regexp,
or remove that --line-regexp option.

This patch removes the --line-regexp (ie -x) option from the grep
in log4perl_component, so that it detects logger components correctly.

To Test:
0) Apply patch
1) Build Koha package
2) Look at /etc/koha/sites/kohadev/log4perl.conf in an existing
Koha instance (e.g. on koha-testing-docker), and note it has
components for z3950, api, and sip
3) Install package
4) Note that /etc/koha/sites/kohadev/log4perl.conf is the same as before
5) Remove the "log4perl.logger.sip and log4perl.appender.SIP lines from
/etc/koha/sites/kohadev/log4perl.conf
6) Reinstall package
7) Note that the missing lines have been appended to the file at
/etc/koha/sites/kohadev/log4perl.conf

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
20.05.x
David Cook 4 years ago
committed by Martin Renvoize
parent
commit
c6ed5b5da1
Signed by: martin.renvoize GPG Key ID: 422B469130441A0F
  1. 2
      debian/koha-common.postinst

2
debian/koha-common.postinst

@ -136,7 +136,7 @@ log4perl_component()
local config=$1
local component=$2
if grep -q -x "log4perl.logger.$component" $config; then
if grep -q "log4perl.logger.$component" $config; then
return 0
else
return 1

Loading…
Cancel
Save