https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6698

--- Comment #9 from Vernon Schryver <[email protected]> 2011-11-13 15:46:14 UTC 
---
There is a minor issue with the committed change.  Line 523 is now
  $pgmpath = '/var/dcc/' . $pgm;
I think it should be the value that can see in 
http://www.dcc-servers.net/dcc/dcc-tree/misc/DCC.pm 
  $pgmpath = '/usr/local/dcc/' . $pgm;
Or the /var/dcc/ line could be kept and these 2 lines added 
  $pgmpath = '/usr/local/dcc/' . $pgm;
  return $pgmpath if -x $pgmpath;

The intent in using /usr/local/dcc was to look in a common 
choice for the DCC `./configure --bindir=DIR` value.
The default value is /usr/local/bin

I apologize for not noticing this issue before the change 
was committed.


A separate issue is what should be done with the copy of DCC.pm
that has been in versions of DCC for the last 18 months.
It's always asking for trouble to have 2 copies of a value,
not to mention code.  I could remove the copy in the DCC source
after the SpamAssassin version is released.  When might that be?
I expect to make a DCC release in the next few months.
Deleting the copy of DCC.pm in the DCC source would inconvenience
installations that are looking in the DCC tarball for DCC.pm.  
Some of them are using the fact that the DCC version of DCC.pm
is actually generated by the DCC ./configure script.  See 
http://www.dcc-servers.net/dcc/dcc-tree/misc/DCC.pm.in
Do you have any suggestions?

-- 
Configure bugmail: 
https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

Reply via email to