Re: [rt.cpan.org #48918] Feature request: Fail if ExtUtils::CBuilder is build_requires and have_compiler is false

Eric Wilhelm <[email protected]>
Newsgroups gmane.comp.lang.perl.modules.module-build
Message-ID <[email protected]>
# from David Golden via RT
# on Sunday 23 August 2009 20:41:

>* In have_c_compiler, cache the results in the stash, not the
>properties, because it isn't actually a property, but we do want it to
>persist per session

I'm not sure what you mean by "per session", but $self->{stash} is not 
persisted in _build/.  So, it might not be a 'property', but 
have_c_compiler() should somehow be cached by write_config() because 
EU::CB's have_compiler() is an expensive operation of attempting to 
compile a small c program.  Thus, if it was true at configure time, we 
don't want to repeat that effort at build time.

I'm guessing that "just stick it in properties" was the easiest way to 
make this work when it was written.  If there is some sort of "internal 
use only" equivalent of properties, I'm not seeing it.  Do we need to 
add one?

I'm inclined to just call it a {_have_c_compiler} or even leave it as a 
{have_c_compiler} property.

--Eric
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.