<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "http://bugs.gentoo.org/bugzilla.dtd">

<bugzilla version="2.22.7"
          urlbase="http://bugs.gentoo.org/"
          maintainer="bugzilla@gentoo.org"
>

    <bug>
          <bug_id>190534</bug_id>
          
          <creation_ts>2007-08-28 14:28 0000</creation_ts>
          <short_desc>sci-physics/cernlib-2006 completely ignores CFLAGS &amp; FFLAGS on sparc and chooses very bad defaults.</short_desc>
          <delta_ts>2007-09-17 14:15:17 0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>Gentoo Linux</product>
          <component>Applications</component>
          <version>2006.1</version>
          <rep_platform>Sparc</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          
          <priority>P2</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>59506</blocked>
          
          <everconfirmed>1</everconfirmed>
          <reporter>fmccor@gentoo.org</reporter>
          <assigned_to>sci-physics@gentoo.org</assigned_to>
          <cc>sparc@gentoo.org</cc>

      

      
          <long_desc isprivate="0">
            <who>fmccor@gentoo.org</who>
            <bug_when>2007-08-28 14:28:59 0000</bug_when>
            <thetext>sci-physics/cernlib-2006 does build on sparc.  However, it ignores CFLAGS (and FFLAGS) from make.conf, substituting its own defaults.  For CFLAGS, it forces -O3 (in general, not a good idea) ignores -mcpu= from make.conf CFLAGS, and even sometimes FORCES -mcpu=v7.  This results in building the C portion for essentially an SS2 and quite suboptimal performance on any sparc system anyone is likely to have.

Fortran build is similar, although I did not notice any forced -mcpu=v7 flags.

Since it builds, I&apos;m giving it ~sparc for testing, but it really should use my CFLAGS/FFLAGS, not what it thinks I need unless there is some compelling reason to do otherwise.  Please see Bug 59506 for an explanation.</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>fmccor@gentoo.org</who>
            <bug_when>2007-08-28 15:18:55 0000</bug_when>
            <thetext>Add CC sparc. </thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>bicatali@gentoo.org</who>
            <bug_when>2007-08-28 18:52:51 0000</bug_when>
            <thetext>I will take a deeper look how to enforce users CFLAGS/FFLAGS as well as a decent src_test procedure. Useful for other arches anyway.

</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>bicatali@gentoo.org</who>
            <bug_when>2007-09-03 15:58:12 0000</bug_when>
            <thetext>
just commited some cernlib fixes allowing user&apos;s flags and a src_test. Give it a try when you can. I will also propagate these changes in other cernlib packages (paw, geant, cernlib-montecarlo).
</thetext>
          </long_desc>
          <long_desc isprivate="0">
            <who>fmccor@gentoo.org</who>
            <bug_when>2007-09-17 13:57:37 0000</bug_when>
            <thetext>(In reply to comment #3)
&gt; just commited some cernlib fixes allowing user&apos;s flags and a src_test. Give it
&gt; a try when you can. I will also propagate these changes in other cernlib
&gt; packages (paw, geant, cernlib-montecarlo).
&gt; 

That fixes the CFLAGS, thanks.  Also, it does run many tests now.  Apparently, it runs them successfully (at least, src_test does not take the &quot;die&quot; branch, and as far as I could tell from watching it run, everything reported success).

So far as I am concerned, this bug is fixed and you can close it.

Thanks for your help.</thetext>
          </long_desc>
      
    </bug>

</bugzilla>