Bug 11252

Summary: rebuild_zebra.pl should not have --munge-config
Product: Koha Reporter: Galen Charlton <gmcharlt>
Component: SearchingAssignee: Galen Charlton <gmcharlt>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: fridolin.somers
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Medium patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 11252: remove deprecated -munge-config switch from rebuild_zebra.pl
Bug 11252: remove deprecated -munge-config switch from rebuild_zebra.pl
[PASSED QA] Bug 11252: remove deprecated -munge-config switch from rebuild_zebra.pl

Description Galen Charlton 2013-11-14 18:46:13 UTC
The --munge-config option of rebuild_zebra.pl attempts to create a valid Zebra configuration.  While there was a period in the 3.0 days where this might have been useful, nowadays this option would either not work, or if it does work, be actively dangerous.
Comment 1 Galen Charlton 2013-11-14 18:50:57 UTC Comment hidden (obsolete)
Comment 2 Chris Cormack 2013-12-22 22:11:53 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2013-12-26 13:03:33 UTC
Created attachment 23820 [details] [review]
[PASSED QA] Bug 11252: remove deprecated -munge-config switch from rebuild_zebra.pl

The -munge-config switch has been deprecated for years, and
trying to use it would either not work at all or, if it did "work",
almost certainly damage one's Zebra configuration for Koha.

This patch removes this switch.

To test:

[1] Run rebuild_zebra.pl and verify that no mention is made
    of -munge-config.
[2] Run rebuild_zebra.pl to index records in one's test database
    and verify that there are no regressions.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Removing a really dangerous option

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Passes all tests and QA script.
Ran rebuild_zebra.pl with various options and confirmed
that data was reindexed successfully.
No regressions found.
Comment 4 Galen Charlton 2013-12-26 15:25:17 UTC
Pushed to master.
Comment 5 Fridolin Somers 2014-01-14 08:33:31 UTC
Patch pushed to 3.14.x, will be in 3.14.2.