This is the mail archive of the ecos-patches@sources.redhat.com mailing list for the eCos project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Patch: Platform HAL (ecos) port for Samsung's ARM9 processor board, SMDK2410


"Michael Anburaj" <embeddedeng@hotmail.com> writes:

> Hi Nick,
> 
> >I've just tried applying this patch. Unfortunately, your mail reader
> 
> I did the following on RH 9:
> $ gzip smdk2410.patch
> 
> & sending it from Mozilla :)
> 
> Thanks a lot,
> -Mike.
> 

Great, that was much better. I've now checked it all it. You should do
a "cvs update -d" and make sure everything is how you expect.

A couple of points:

1. I suspect you are an emacs user, since you left all the *~ files
   in. this nearly doubled the size of the patch. For future
   reference, adding -x "*~" to the diff line would eliminate these.

2. You didn't include any ChangeLogs in the patch. In this instance
   I've written them for you since they are trivial. However,
   ChangeLogs from the actual author are to be preferred, and we
   maintainers, being essentially lazy, would be far less inclined to
   accept a patch without them :-).

-- 
Nick Garnett                    eCos Kernel Architect
http://www.ecoscentric.com      The eCos and RedBoot experts


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