This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH 2/2] [GOLD] Add plugin API for processing plugin-added input files
- From: "Sriraman Tallam via binutils" <binutils at sourceware dot org>
- To: Stephen Crane <sjc at immunant dot com>
- Cc: Cary Coutant <ccoutant at gmail dot com>, Binutils <binutils at sourceware dot org>
- Date: Fri, 13 Oct 2017 22:47:11 -0700
- Subject: Re: [PATCH 2/2] [GOLD] Add plugin API for processing plugin-added input files
- Authentication-results: sourceware.org; auth=none
- References: <CAB7K5r44EtJiXZOiQzM+qqymKjooOTbyb7gikfOxbk7yoyrywg@mail.gmail.com> <CAAs8HmyFiWr6SKG-p+_zHGVBw7sSAoudO5b0qe_zaVPLK1HGmw@mail.gmail.com> <CAB7K5r4DaPN+bq=evZv5zmiEA3BQ8RCv-8ex6889J-AyNyCxdA@mail.gmail.com>
- Reply-to: Sriraman Tallam <tmsriram at google dot com>
On Fri, Oct 13, 2017 at 6:00 PM, Stephen Crane <sjc@immunant.com> wrote:
> On Wed, Oct 11, 2017 at 12:09 AM, Sriraman Tallam <tmsriram@google.com> wrote:
>> On Mon, Sep 25, 2017 at 6:46 PM, Stephen Crane <sjc@immunant.com> wrote:
>>> +# Copyright (C) 2011-2017 Free Software Foundation, Inc.
>>> +# Written by Sriraman Tallam <tmsriram@google.com>.
>>
>> Copy paste error.
>
> It seemed wrong to put my own name there, since I just slightly
> modified what you wrote. Would the following be appropriate, or should
> I just take your name off entirely?
>
> # Copyright (C) 2017 Free Software Foundation, Inc.
> # Originally written by Sriraman Tallam <tmsriram@google.com>, altered by
> # Stephen Crane <sjc@immunant.com>.
Thanks for being careful with this but I am perfectly ok with taking
my name off entirely.
>
> Honestly, the more I think about it, I think it might just be best to
> use a single shell script for both tests and not duplicate code. The
> tests are distinct and use different plugins, but the validation is
> identical. However, using the same script could make it unclear which
> test failed when there is a failure. I'm not sure which is more
> important here.
I would say the latter is more important. Could we not just `source`
.the other script?
Thanks
Sri
>
>>> +# The goal of this program is to verify if plugin section reordering and
>>> +# assignment works for plugin-created object files. File final_layout.cc is in
>>> +# this test.
>>
>> again copy paste error.
>
> Indeed, should be plugin_final_layout.cc. I'll change that.
>
>
>> It looks fine to me but I cannot approve.
>> Thanks
>> Sri
>
> Thanks for looking through this.
>
> - stephen