Re: Autoconf-refactor bugs?

On Fri, Aug 13, 2010 at 11:32 PM, Andrew Beverley <andy_at_andybev.com> wrote:
> Hi,
>
> Firstly, I'm slightly confused as to whether I am seeing the
> autoconf-refactor work in my copy of trunk (I have run bzr update). I
> didn't think I'd seen it go in, but delving into configure.in I think it
> might be in there... what should I look for to check whether it's
> definitely what I'm looking at?

It is, it was merged two days ago. And you're right, it does have a few bugs.
Sorry about those.

> Secondly, assuming it is in, there appear to be 2 small problems:
>
> 1. --disable-inline doesn't work for me. On investigation, this seems to
> be because enable_inline is set to "yes" at line 296, which overrides
> the later AC_ARG_ENABLE(inline), regardless of the setting. Also, I had
> to add in enable_inline=$enableval after the AC_ARG_ENABLE check (once
> I'd removed the enable_inline=yes) to get it to work.

Fixed, in a slightly different way.

> 2. Line 1344 (if "$squid_host_os" = "solaris" ; then) appears to be
> missing a 'test'.

Fixed.

> Sorry if I've jumped the gun and am actually looking at old code.

You have not, and thank you for checking things out.
There is one more bug, related to the default hosts file and default
http and icp ports handling.
I've also fixed those.

-- 
    /kinkie