Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 24 Oct 2005 22:15:55 -0700
From:      "David O'Brien" <obrien@FreeBSD.org>
To:        Alexander Leidinger <Alexander@Leidinger.net>
Cc:        cvs-ports@FreeBSD.org, cvs-all@FreeBSD.org, ports-committers@FreeBSD.org
Subject:   Re: cvs commit: ports/editors/vim Makefile
Message-ID:  <20051025051555.GD55418@dragon.NUXI.org>
In-Reply-To: <20051024143613.hu726dyokc4wco88@netchild.homeip.net>
References:  <200510240034.j9O0YDgG047096@repoman.freebsd.org> <20051024143613.hu726dyokc4wco88@netchild.homeip.net>

next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, Oct 24, 2005 at 02:36:13PM +0200, Alexander Leidinger wrote:
> "David E. O'Brien" <obrien@FreeBSD.org> wrote:
> 
> >obrien      2005-10-24 00:34:13 UTC
> >
> > FreeBSD ports repository
> >
> > Modified files:
> >   editors/vim          Makefile
> > Log:
> > Attempt to fix to respect non-standard LOCALBASE/X11BASE settings.
> > Let me just take this opportunity to express how much I hate autoconf's
> > attempt at containing AI, and note the size of hammer one has to use to
> > knock some real knowledge into its head.
> 
> In your commit you use PREFIX instead of LOCALBASE/X11BASE, so this commit
> message confuses me.
> 
> I've compiled vim 6.[34] on Solaris 10 recently and I had no problem to use
> "--prefix=",

Correct.  And on FreeBSD too.

> so I doubt that replacing /usr/local with PREFIX as a reinplace
> command is supposed to fix the destination.

The issue is the autoconfig script has /usr/local/include and
/usr/local hard coded in it (ac_default_prefix=/usr/local).

> But PREFIX is not right for a non-standard LOCALBASE/X11BASE. What is
> this commit supposed to do?

To tell the truth I'm not sure the exact case Kris was trying to allow.
You are right that ac_default_prefix=${LOCALBASE} may be a better
substitution.

-- 
-- David  (obrien@FreeBSD.org)



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20051025051555.GD55418>