Line 6446 of the "configure" script shipped with the beta 29 is broken; the line
reads:

    ax_compiler_version=`cat conftest.out | 's,.*[[:space:]],,'`

which is obviously nonsense, and should probably read instead:

    ax_compiler_version=`cat conftest.out | sed 's,.*[[:space:]],,'`

This doesn't show when using the standard settings, but gets in the way when
trying to configure for the Intel C++ compiler using the following settings:

../configure --prefix=/foo/bar COMPILED_BY="foobar" CXX=icpc

on a Debian "Etch" 4.0r5 AMD64 Linux using the 11.0 Intel(R)64 version of the
intel compiler.

Don't know whether this is a general autoconf bug though, or POV-Ray specific.

Post a reply to this message

clipka <nomail@nomail> wrote:
>     ax_compiler_version=`cat conftest.out | sed 's,.*[[:space:]],,'`

  I know nitpicking about "useless use of cat" is rather old, but would
it really hurt if that was written in the "kosher" way, ie:

    ax_compiler_version=`sed 's,.*[[:space:]],,' conftest.out`

  If nothing else, it's more efficient that way.

-- 
                                                          - Warp

Post a reply to this message

Warp <war### [at] tagpovrayorg> wrote:
> clipka <nomail@nomail> wrote:
> >     ax_compiler_version=`cat conftest.out | sed 's,.*[[:space:]],,'`
>
>   I know nitpicking about "useless use of cat" is rather old, but would
> it really hurt if that was written in the "kosher" way, ie:
>
>     ax_compiler_version=`sed 's,.*[[:space:]],,' conftest.out`
>
>   If nothing else, it's more efficient that way.

Maybe it has something to do with compatibility? Maybe some exotic systems that
run into trouble with this 'kosher' approach?

Then again it just may have to do with Unix people's brain cells being
hard-wired to using pipes whenever they hear "sed", "grep", "awk" or similar.

Anyway, we're not talking about runtime performance of POV here, just some
installation thing that runs fast enough as it is.

Post a reply to this message

clipka <nomail@nomail> wrote:
> Anyway, we're not talking about runtime performance of POV here, just some
> installation thing that runs fast enough as it is.

  There's no reason to not do it correctly.

-- 
                                                          - Warp

Post a reply to this message

Warp <war### [at] tagpovrayorg> wrote:
> clipka <nomail@nomail> wrote:
> > Anyway, we're not talking about runtime performance of POV here, just some
> > installation thing that runs fast enough as it is.
>
>   There's no reason to not do it correctly.

I don't know, maybe this is all generated by Autogen, which (if I understand the
concept correctly) would move this issue out of the scope of the POV team.

Post a reply to this message

clipka wrote:
> Line 6446 of the "configure" script shipped with the beta 29 is broken;
> the line reads:
> 
>     ax_compiler_version=`cat conftest.out | 's,.*[[:space:]],,'`
> 
> which is obviously nonsense, and should probably read instead:
> 
>     ax_compiler_version=`cat conftest.out | sed 's,.*[[:space:]],,'`
> 
> This doesn't show when using the standard settings, but gets in the way
> when trying to configure for the Intel C++ compiler using the following
> settings:
> 
> ../configure --prefix=/foo/bar COMPILED_BY="foobar" CXX=icpc
> 
> on a Debian "Etch" 4.0r5 AMD64 Linux using the 11.0 Intel(R)64 version of
> the intel compiler.
> 
> Don't know whether this is a general autoconf bug though, or POV-Ray
> specific.

If that line isn't in configure.ac (or configure.in, not sure what POV calls
it), it's an autoconf problem.

Post a reply to this message

> Don't know whether this is a general autoconf bug though, or POV-Ray specific.

	That's POV specific -- you know, typically the kind of things that tend to
happen in (though long-standing) beta software.
	Thanks for the report anyway  :)

	- NC

Post a reply to this message