<div dir="ltr">Created a ticket <a href="https://trac.mpich.org/projects/mpich/ticket/2216">https://trac.mpich.org/projects/mpich/ticket/2216</a> <br></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature"><div dir="ltr">--Junchao Zhang</div></div></div>
<br><div class="gmail_quote">On Tue, Jan 13, 2015 at 11:47 AM, Balaji, Pavan <span dir="ltr"><<a href="mailto:balaji@anl.gov" target="_blank">balaji@anl.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
I agree with Dave.  The F90 check was added because of idiot users who don't care about warnings and tell us that things don't work as expected.  His suggestion to check for F90 != FC is a better option too.  There'd be very few cases where F90 would need to be different from FC.  In such cases, Mike's suggestion for a configure option might do the trick.<br>
<br>
Can someone create a ticket?  I'm on vacation, I shouldn't even be responding to emails.<br>
<br>
  -- Pavan<br>
<div><div class="h5"><br>
> On Jan 13, 2015, at 11:00 PM, Blocksome, Michael <<a href="mailto:michael.blocksome@intel.com">michael.blocksome@intel.com</a>> wrote:<br>
><br>
> Could an option be added to configure similar to "-Wall -Werr" ? For example,<br>
><br>
>  ../mpich.git/configure --with-configure-warnings=abort<br>
><br>
> Then these over-zealous configure errors could be turned into easily ignored configure warnings in the general case?<br>
><br>
> -----Original Message-----<br>
> From: Jeff Hammond [mailto:<a href="mailto:jeff.science@gmail.com">jeff.science@gmail.com</a>]<br>
> Sent: Tuesday, January 13, 2015 11:12 AM<br>
> To: <a href="mailto:devel@mpich.org">devel@mpich.org</a><br>
> Subject: Re: [mpich-devel] overzealous F90/F90FLAGS error<br>
><br>
> On Tue, Jan 13, 2015 at 9:03 AM, Dave Goodell (dgoodell)<br>
> <<a href="mailto:dgoodell@cisco.com">dgoodell@cisco.com</a>> wrote:<br>
>> On Jan 13, 2015, at 10:53 AM, Jeff Hammond <<a href="mailto:jeff.science@gmail.com">jeff.science@gmail.com</a>> wrote:<br>
>><br>
>>> On Tue, Jan 13, 2015 at 8:50 AM, Dave Goodell (dgoodell)<br>
>>> <<a href="mailto:dgoodell@cisco.com">dgoodell@cisco.com</a>> wrote:<br>
>>>><br>
>>>> Rather than remove this code altogether, it might be better to change the logic to complain more like this (untested):<br>
>>>><br>
>>>> if test -n "$F90" && test "x$F90" != "x$FC" ; then<br>
>>>>   AC_MSG_ERROR([F90 set but not equal to FC.  This is almost certainly a mistake.  Either unset F90 or set F90="$FC"])<br>
>>>> elif test -n "$F90FLAGS" && test "x$F90FLAGS" != "x$FCFLAGS" ; then<br>
>>>>   AC_MSG_ERROR([F90FLAGS set but not equal to FCFLAGS.  This is almost certainly a mistake.  Either unset F90FLAGS or set F90FLAGS="$FCFLAGS"])<br>
>>><br>
>>> Except there are systems where FC=xlf2008_r F90=xlf90_r F77=xlf77_r is<br>
>>> the right thing to do.  Thankfully, I no longer have to deal with such<br>
>>> system, so I bring this up as an academic point.<br>
>><br>
>> I don't feel strongly about this, since I don't have to do much support of situations where users would make an F90/FC mistake.  But lots of past users of MPICH (from before the switch to using FC instead of F90) are likely to have scripts/documentation that uses F90 and is oblivious to the fact that it will be ignored.  So those users will have a pretty bad experience when their MPI installation uses a different Fortran compiler than they were expecting.  A plain warning from configure probably isn't enough for them to realize they made a mistake.<br>
><br>
> Yeah, this is real problem.  My solution would be to use F90 as FC if<br>
> and only if F90 is defined to be a non-MPI compiler while FC is not<br>
> defined at all.<br>
><br>
> I agree that the warning will quickly disappear in the stream of<br>
> output and will not be effective.<br>
><br>
>> Also, as far as I recall, FC!=F90 is completely at odds with the current MPICH (and OMPI) build system(s).  I don't know whether those systems are correct or not, but it's how they are deliberately designed and implemented.<br>
><br>
> True, but since MPICH doesn't reference F90 at any point, it should be<br>
> perfectly valid for a user to set FC=ifort and F90="mpifort".<br>
><br>
>> I've now used up all of my "care about what happens to Fortran" points for the month, so it's back to lurking for me...<br>
><br>
> But there are 17 days left in January!!! :-)<br>
><br>
> Jeff<br>
><br>
> --<br>
> Jeff Hammond<br>
> <a href="mailto:jeff.science@gmail.com">jeff.science@gmail.com</a><br>
> <a href="http://jeffhammond.github.io/" target="_blank">http://jeffhammond.github.io/</a><br>
> _______________________________________________<br>
> To manage subscription options or unsubscribe:<br>
> <a href="https://lists.mpich.org/mailman/listinfo/devel" target="_blank">https://lists.mpich.org/mailman/listinfo/devel</a><br>
> _______________________________________________<br>
> To manage subscription options or unsubscribe:<br>
> <a href="https://lists.mpich.org/mailman/listinfo/devel" target="_blank">https://lists.mpich.org/mailman/listinfo/devel</a><br>
<br>
</div></div>--<br>
Pavan Balaji  ✉️<br>
<a href="http://www.mcs.anl.gov/~balaji" target="_blank">http://www.mcs.anl.gov/~balaji</a><br>
<div class="HOEnZb"><div class="h5"><br>
_______________________________________________<br>
To manage subscription options or unsubscribe:<br>
<a href="https://lists.mpich.org/mailman/listinfo/devel" target="_blank">https://lists.mpich.org/mailman/listinfo/devel</a></div></div></blockquote></div><br></div>