On Tue, Dec 22, 2009 at 19:03, Marcel Telka <marcel.telka at sun.com> wrote:
> On Tue, Dec 22, 2009 at 06:43:47PM -0500, Stefan Teleman wrote:
>> http://cr.opensolaris.org/~steleman/6912643/
>
> Why you added --localstatedir=/var ?
> I am unable to find any reason for that in the CR 6912643.
The reason you are unable to find any reason for --localstatedir in
6912643 is: 6912643 has nothing to do with ${localstatedir}.
The reason for passing --localstatedir to ./configure has been
discussed here several times, in the past.
Every single autoconf-generated ./configure script places
${localstatedir} in ${prefix}/var. This would mean: /usr/var. This
location is not possible in Solaris.
However, that is not the point of this CR, or of the code review request.
--Stefan
--
Stefan Teleman
KDE e.V.
stefan.teleman at gmail.com
