This is the mail archive of the binutils@sourceware.cygnus.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: binutils support for a.out-arm-netbsd target


Hi Roland,

  Thanks for this patch.  [Sorry about the delay in responding :-(]

  Anyway I have applied the patch, but I would like to make a comment
about the ChangeLog entries that you supplied:

: Index: bfd/ChangeLog
: ===================================================================
: RCS file: /cvs/binutils/binutils/bfd/ChangeLog,v
: retrieving revision 1.193
: diff -u -b -r1.193 ChangeLog
: --- bfd/ChangeLog	1999/08/19 13:04:45	1.193
: +++ bfd/ChangeLog	1999/08/19 21:52:36
: @@ -1,3 +1,13 @@
: +1999-08-19  Roland McGrath  <roland@baalperazim.frob.com>
: +
: +	* config.bfd (arm-*-netbsd*): New target.
: +	* configure.in (armnetbsd_vec): New target vector.
: +	* targets.c (bfd_target_vector): Add &armnetbsd_vec.
: +
: +	* Makefile.am (BFD32_BACKENDS): Add armnetbsd.lo.
: +	(ALL_MACHINES_CFILES): Add armnetbsd.c.
: +	(armnetbsd.lo): New rule with deps.
: +
:  1999-08-19  Nick Clifton  <nickc@cygnus.com>


Firstly, please could you not supply them as context diffs, but just
as straight pieces of ASCII.  The ChangeLogs are changing so rapidly
that we are rarely able to apply a context diff without massaging it
first by hand.

Secondly, please could you include *all* of the files that are
affected by your patch in the appropriate ChangeLog entry.  For
example in the bfd ChangeLog entry above you omitted the fact that
Makefile.in had been regenerated and that a new file, armnetbsd.c, had 
been added to the directory.

Cheers
	Nick

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