This is the mail archive of the binutils@sources.redhat.com mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: luciano gemme


Hi Gemme,

: I submit the change for objcopy utility as previously declared in a
: previous mail.

: I've already sent the copyright form properly signed

I am currently waiting to here from the FSF lawyers that the copyright
assignment has gone through.  Once it has I will consider applying
your patch.

Please note however, that there are a couple of problems with your
patch:

  * You did not include any ChangeLog entries for the files that you
    have modified.  These really should be included.

  * You have added documentation for the new command line switches to
    objcopy.1 (good) but you forgot to also add the documentation to
    binutils.texi (bad).

  * You have added a new command line switch --srec-forceS3 to only
    allow the generation of S3 srecords.  Really this switch should
    take an argument of the type of srecord that should be forced, ie
    --srec-force={1|2|3}.

Cheers
	Nick

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]