Re: Adding a configuration option.

Duane Wessels wrote:

>
> On Wed, 6 Dec 2000, Joe Cooper wrote:
>
>
>> Hey folks,
>>
>> I've been trying to turn a patch to 'fix' the IE refresh issue into a
>> runtime option, but I'm having a bit of difficulty convincing Squid to
>> parse the new option.
>>
>> I've added the ie_refresh option the cf.data.pre, and I've added the
>> option to the onoff struct in structs.h, and I've added an
>> if(Config.onoff.ie_refresh) check before the relevant code. A recompile
>> later, and restarting Squid with the new option enabled in squid.conf
>> leads to a "2000/12/06 15:59:47| parseConfigFile: line 2006
>> unrecognized: 'ie_refresh on'" error.
>
>
> maybe you forgot 'make install' or you're running an old executable?
>
> try: strings squid | grep ie_refresh

Ok, I've checked the executable, and it is the one I just compiled. I'm
using RPM to build my Squid so every rebuild starts from scratch (so no
chance of old files laying around). I'll do a compile manually to see
if something isn't working as expected. The strings/grep reveals that
there is no such string in the executable. So for whatever reason the
patch isn't making it into the executable.

Thanks for the advice.
                                   --
                      Joe Cooper <joe@swelltech.com>
                  Affordable Web Caching Proxy Appliances
                         http://www.swelltech.com
Received on Wed Dec 06 2000 - 18:59:47 MST