deprecate legacy hold_targeter.pl in favor of new one

[Chris Sharp]

Bug 1596595 introduced a hold targeting process that is faster, more efficient, and integrates better into OpenSRF than the "traditional" hold_targeter.pl script. Since PINES and others have been successfully running the new targeter in production environments for over a year, I propose that we deprecate the old version in favor of the "v2" version.

Branch on the way.

[Chris Sharp (chrissharp123)]

Branch here: http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/csharp/lp1745610_promote_new_hold_targeter_to_default

[Dan Wells (dbw2)]

Bumped this up to "Low"; IMO, it can go in any time before release.

[Bill Erickson (berick)]

Chris, mind applying the file name changes to Open-ILS/src/Makefile.am (core_scripts = ...) and adding a release notes entry explaining that users may need to change their local crontab's to ensure it's running the script they think it's running w/ the correct params?

[Jeff Davis (jdavis-sitka)]

I've retargeted this bug to 3.2-beta. Deprecating functionality doesn't really seem like a point-release kind of change, and the bug is more likely to get attention this way anyway.

[Chris Sharp (chrissharp123)]

Okay, finally saw Bill's request and I've made the requested changes, rebased, and force-pushed to the same location: http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/csharp/lp1745610_promote_new_hold_targeter_to_default

[Bill Erickson (berick)]

Thanks, Chris. Confirmed installer works as expected. Merged to master.