On Wed, 11 Jul 2007, Paul Cochrane wrote:

> To be able to configure parrot to build with icc (the intel C
> compiler) one currently needs a command line which looks like this:
> 
> perl Configure.pl --cc=icc --link=icc --ld=icc
> --ccflags="<icc-header-path> -no-gcc" --ccwarn="-wd269 -Wall -Wcheck
> -w2"
> 
> However, the only things which need to be specified here are:
> 
> (a) the cc setting
> (b) the icc header path
> 
> the rest is set in the linux hints file.
> 
> So what's the problem you ask?  Well, to get icc to see its own header
> files, one needs to specify the icc header path in the --ccflags
> option.  This overwrites any settings given in the hints file. 

Then I'd say the hints file is broken.  Unless it has good reason, the 
hints file shouldn't normally remove command-line information.  In perl5 
hints, the usual idiom is to add to existing ccflags, not replace them.

> It
> would be great to give a configure command like this instead:
> 
> perl Configure.pl --cc=icc --ccflags-append=<icc-header-path>

For consistency, I would think there should also be an -append option for 
every Configure variable.  (Whether you wish to write it as

        --ccflags-append=icc_header_path

or
        --append ccflags=icc_header_path

or some other syntax is, at some level, irrelevant.  What I'm saying is 
that instead of introducing an append syntax specific to ccflags, you 
should introduce a generic append syntax and then use it (if needed) for 
ccflags.

I'm also advocating revising the hints file so the append syntax isn't 
needed!

-- 
    Andy Dougherty              [EMAIL PROTECTED]

Reply via email to