On 10/21/2017 11:17 AM, Bernhard Reutner-Fischer wrote: > On 21 October 2017 at 02:26, Damian Rouson <dam...@sourceryinstitute.org> > wrote: >> >> Hi Richard, >> >> Attached is a revised patch that makes the downloading of Fortran >> prerequisites optional via a new --no-fortran flag that can be passed to >> contrib/download_prerequisites as requested in your reply below. >> >> As Jerry mentioned in his response, he has been working on edits to the >> top-level build machinery, but we need additional guidance to complete his >> work. Given that there were no responses to his request for guidance and >> it’s not clear when that work will complete, I’m hoping this minor change >> can be approved independently so that this patch doesn’t suffer bit rot in >> the interim. >> >> Ok for trunk? > > + die "Invoking wget and curl in the 'download_prerequisites' script failed." > > I suggest "Neither wget nor curl found, cannot download tarballs." > > As an open-mpi user i question why this hardcodes MPICH, fwiw. > > thanks, >
Hi Bernhard, Well, open-mpi does not yet support failed images so we are focusing on mpich. Maybe what we should do is default to mpich if not otherwise specified and provide a -mpi=mpich or -mpi=open-mpi, or similar. Then later we can add any other implementations that show up. Cheers, Jerry
