Well, I looked through the old code (manually, like an archeologist),
and it goes way, way, way back to version 1.2 (6 years, 4 months
ago):

http://cvs.apache.org/viewcvs.cgi/modperl/Makefile.PL?r1=1.2&r2=1.3&diff_format=h

First seen here on line 294 as:
 $EXTRA_CFLAGS .= join(" ", split(",",  " $PERL_EXTRA_CFLAGS"))
                     if $PERL_EXTRA_CFLAGS;

A subsequent checkin, 1.4, changes it to the form we see today. It
was the first revision to actually set $PERL_EXTRA_CFLAGS to
something, apparently relating to expiremental features. Revision
1.144 just moves that block around.

Unfortunately I couldn't find any mailing list traffic for January,
1998, and the log messages aren't terribly helpful either.

The problem is that as of 1.160, $PERL_EXTRA_CFLAGS gets set to
include $Config::Config{ccflags}, which may contain all sorts of
crazy strings that wasn't taken into account at 1.2.

As to the question about the flag causing this mess, "-Wa,xarch=v9",
I don't think it's an absolute requirement for 64-bit, but I believe
it's added for you if you build perl on solaris sparcv9 64-bit.

--- Joe Schaefer <[EMAIL PROTECTED]> wrote:
> Stas Bekman <[EMAIL PROTECTED]> writes:
> 
> [...]
> 
> > It should be possible to automate this search by getting cvs
> version bumped up
> > incrementally and grepping for that split line, untill the change
> is
> > found. It'd be a nice addition to our toolset. 
> 
> What's wrong with cvs annotate here?
> 
> gemini:~/src/apache/modperl$ cvs annotate Makefile.PL | grep
> PERL_EXTRA_CFLAGS
> 
> Annotations for Makefile.PL
> ***************
> 1.174        (dougm    23-Dec-00): $PERL_EXTRA_CFLAGS = "";
> 1.4          (hartill  10-Feb-98):      $PERL_EXTRA_CFLAGS .= "
> -D${k}=1";
> 1.86         (dougm    28-Aug-98):      $PERL_EXTRA_CFLAGS .= "
> -D$_=1";
> 1.81         (dougm    30-Jul-98):     $PERL_EXTRA_CFLAGS .= "
> -DPERL_SAFE_STARTUP=1";
> 1.201        (dougm    23-May-02):     $PERL_EXTRA_CFLAGS .= "
> $Config{ccflags}";
> 1.201        (dougm    23-May-02): $PERL_EXTRA_CFLAGS =~
> s/-D_GNU_SOURCE//;
> 1.144        (dougm    07-Mar-00):      if($PERL_EXTRA_CFLAGS) {
> 1.144        (dougm    07-Mar-00):          $PERL_EXTRA_CFLAGS =
> join(" ", split(",",  $PERL_EXTRA_CFLAGS));
> 1.144        (dougm    07-Mar-00):          $PERL_EXTRA_CFLAGS =~
> s/\s+/ /g;
> 1.144        (dougm    07-Mar-00):          $PERL_EXTRA_CFLAGS .= "
> -g";
> 1.144        (dougm    07-Mar-00):      $PERL_EXTRA_CFLAGS .= "
> -DPERL_DESTRUCT_LEVEL=$PERL_DESTRUCT_LEVEL"
> 1.55         (dougm    25-Jun-98):      if($PERL_EXTRA_CFLAGS) {
> 1.55         (dougm    25-Jun-98):          $cmd .=
> qq(CFLAGS="$PERL_EXTRA_CFLAGS" );
> 1.4          (hartill  10-Feb-98):     $inc .= "
> $PERL_EXTRA_CFLAGS" if $PERL_EXTRA_CFLAGS;
> gemini:~/src/apache/modperl$ cvs log -r1.144  Makefile.PL
> 
> RCS file: /home/cvspublic/modperl/Makefile.PL,v
> Working file: Makefile.PL
> head: 1.219
> branch:
> locks: strict
> access list:
> keyword substitution: kv
> total revisions: 219;   selected revisions: 1
> description:
> ----------------------------
> revision 1.144
> date: 2000/03/07 03:12:36;  author: dougm;  state: Exp;  lines: +26
> -25
> fix Makefile.PL if $USE_APXS && $PERL_DEBUG
>
=============================================================================
> 
> Doesn't look like the cvs log is going to be of much help;
> perhaps checking the list archives in Feb/March 2000 will
> turn up something useful.
> 
> -- 
> Joe Schaefer
> 
> 
> -- 
> Report problems: http://perl.apache.org/bugs/
> Mail list info: http://perl.apache.org/maillist/modperl.html
> List etiquette:
> http://perl.apache.org/maillist/email-etiquette.html
> 




        
                
__________________________________
Do you Yahoo!?
SBC Yahoo! - Internet access at a great low price.
http://promo.yahoo.com/sbc/

-- 
Report problems: http://perl.apache.org/bugs/
Mail list info: http://perl.apache.org/maillist/modperl.html
List etiquette: http://perl.apache.org/maillist/email-etiquette.html

Reply via email to