This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [Patch] Edit contrib/ files to download gfortran prerequisites
- From: Richard Biener <richard dot guenther at gmail dot com>
- To: Damian Rouson <damian at sourceryinstitute dot org>
- Cc: gcc patches <gcc-patches at gcc dot gnu dot org>, gfortran <fortran at gcc dot gnu dot org>, Jerry DeLisle <jvdelisle at charter dot net>
- Date: Mon, 23 Oct 2017 12:54:22 +0200
- Subject: Re: [Patch] Edit contrib/ files to download gfortran prerequisites
- Authentication-results: sourceware.org; auth=none
- References: <etPan.59c2d116.44baeccc.18c@sourceryinstitute.org> <CAFiYyc2YtEtjP_QC_ux_XU6kVJ69C8QHRt0Q2fDm8U-vuiKBtg@mail.gmail.com> <etPan.59ea941f.46020ae.1c2b@sourceryinstitute.org>
On Sat, Oct 21, 2017 at 2:26 AM, Damian Rouson
<damian@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.
But the change doesn't make sense without the build actually picking up things.
> Ok for trunk?
No.
Thanks,
Richard.
> Damian
>
>
>
>
> On September 21, 2017 at 12:40:49 AM, Richard Biener (richard.guenther@gmail.com(mailto:richard.guenther@gmail.com)) wrote:
>
>> On Wed, Sep 20, 2017 at 10:35 PM, Damian Rouson
>> wrote:
>> > Attached is a patch that adds the downloading of gfortran prerequisites OpenCoarrays and MPICH in the contrib/download_prerequisites script. The patch also provides a useful error message when neither wget or curl are available on the target platform. I tested this patch with several choices for the command-line options on macOS (including --md5 and --sha512) and Ubuntu Linux (including --sha512). A suggested ChangeLog entry is
>> >
>> > * contrib/download_prerequisites: Download OpenCoarrays and MPICH.
>> > * contrib/prerequisites.sha5: Add sha512 message digests for OpenCoarrays and MPICH.
>> > * contrib/prerequisites.md5: Add md5 message digests for OpenCoarrays and MPICH.
>> >
>> >
>> > OK for trunk? If so, I’ll ask Jerry to commit this. I don’t have commit rights.
>>
>> Can you make this optional similar to graphite/isl? Also I see no support in
>> the toplevel build machinery to build/install the libs as part of GCC
>> so how does
>> that work in the end?
>>
>> Thanks,
>> Richard.
>>
>> > Damian