Excellent point about not needing to use the "FC" name with its special
On 2/21/2012 1:52 AM, Ralph Castain wrote:
> I went ahead and applied this, with a tweak. There is no reason to
> call our flag "FC" as all we use it for is to call the write wrapper.
> So I renamed it to something less problematic.
> On Feb 21, 2012, at 1:20 AM, Paul H. Hargrove wrote:
>> And while we are looking at examples/Makefile on Solaris-10, why are
>> the F77 examples getting built w/ mpif90?
>> Because w/ the Solaris make setting FC also silently sets F77 (yes, I
>> am NOT kidding)!
>> So, reordering the F77= and FC= lines in Makefile resolves that
>> Attached is my patch to fix both F77/FC and the "better" ompi_info
>> queries mentioned in my previous post.
>> This REPLACES the patch in the previous post.
>> On 2/20/2012 11:36 PM, Paul H. Hargrove wrote:
>>> The addition on Monday of the Java cases to examples/Makefile has
>>> shown that the default "make" in Solaris-10 will stop on the first
>>> failed grep command in the "all" rule:
>>>> $ make
>>>> mpicc -g -o hello_c hello_c.c
>>>> mpicc -g -o ring_c ring_c.c
>>>> mpicc -g -o connectivity_c connectivity_c.c
>>>> mpic++ -g -o hello_cxx hello_cxx.cc
>>>> mpic++ -g -o ring_cxx ring_cxx.cc
>>>> mpif90 -g hello_f77.f -o hello_f77
>>>> mpif90 -g ring_f77.f -o ring_f77
>>>> mpif90 -g hello_f90.f90 -o hello_f90
>>>> mpif90 -g ring_f90.f90 -o ring_f90
>>>> *** Error code 1
>>>> The following command caused the error:
>>>> if test "`ompi_info --parsable | grep bindings:java:yes`" != "";
>>>> then \
>>>> make Hello.class; \
>>>> make: Fatal error: Command failed for target `all'
>>> The addition of java did NOT break anything, but exposed a
>>> pre-existing problem which was not evident in my prior testing
>>> because all language bindings were being build prior to adding java.
>>> The attached patch resolves the problem in my (admittedly minimal)
>>> testing with the smallest possible change.
>>> However an entirely different avoids both "test" and "true" and
>>> simply looks like:
>>> @ if ompi_info --parsable | grep bindings:cxx:yes >/dev/null; then
>>> I have *also* tested that approach, and it works fine too.
>>> I *did* warn that the introduction of the java bindings would bring
>>> collateral damage.
>>> I just didn't anticipate encountering it personally.
>>> devel mailing list
>> Paul H. HargrovePHHargrove_at_[hidden]
>> Future Technologies Group
>> HPC Research Department Tel: +1-510-495-2352
>> Lawrence Berkeley National Laboratory Fax: +1-510-486-6900
>> devel mailing list
>> devel_at_[hidden] <mailto:devel_at_[hidden]>
> devel mailing list
Paul H. Hargrove PHHargrove_at_[hidden]
Future Technologies Group
HPC Research Department Tel: +1-510-495-2352
Lawrence Berkeley National Laboratory Fax: +1-510-486-6900