On 10/24/2016 06:16 PM, Moritz Klammler wrote:
Jeff Law <l...@redhat.com> writes:

On 10/24/2016 02:44 AM, Richard Biener wrote:
On Fri, Oct 7, 2016 at 3:10 PM, Moritz Klammler <mor...@klammler.eu> wrote:
I would like to bump my patch that makes the
`contrib/download_prerequisites` script verify the checksums of the
downloaded packages and augments it with a few additional options.
All feedback I have received has been incorporated.  Is it okay like
this?

Below is again the latest iteration of the patch.

Looks generally ok.  Can you remove -eu from the /bin/sh command and
leave the copyright in place?

The patch is ok with that change.  Do you have commit privileges?
I'm pretty sure Moritz doesn't have commit privs.  So I reverted the
gratutious changes to the copyright notice and cobbled together a
changlog and committed the result.

jeff


Thank you very much both of you.

Indeed, I do not have commit privileges.  In fact, this was my very
first patch to be accepted.  I did went through the copyright assignment
process already, though.

Now that the patch is applied, can this bug be closed?

    https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61439
Done.  And ChangeLog updated to include appropriate tags.

jeff

Reply via email to