<div dir="ltr">Sir,<div><br></div><div>I will edit the current patch accordingly and will take care of this in the future.</div><div><br></div><div>Thanks and RegardsĀ </div><div>Abhinav Jain</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Feb 20, 2018 at 7:31 PM, Gedare Bloom <span dir="ltr"><<a href="mailto:gedare@rtems.org" target="_blank">gedare@rtems.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Oops, I should read my mail better. Thanks for breaking this out Chris. Abhinav,<br>
<span class=""><br>
On Tue, Feb 20, 2018 at 1:12 AM, Abhinav Jain <<a href="mailto:jainab.2009@gmail.com">jainab.2009@gmail.com</a>> wrote:<br>
> Sir,<br>
><br>
> I have attached the patch file with this mail. I have tried to follow all<br>
> the conventions that were listed in the User Git page. I request you to<br>
> please check and guide me whether I have done it correctly or not or whether<br>
> something more is to be done.<br>
><br>
</span>I won't comment on the code, I'll let Chris do that. However, this<br>
patch has a few issues that should be addressed.<br>
1. Use line breaks in your commit message. About 70-80 characters per<br>
line max, please.<br>
2. The "short message" can omit "function added to", basically all<br>
patches add some code, so this is a bit redundant. You can just say<br>
"Check the validity ..."<br>
3. The commit message should use "Closes #nnnn." somewhere, usually in<br>
the end of the commit, if it is fixing/closing a ticket.<br>
4. The commit message may be less verbose, if the extra details about<br>
the change are already in the ticket.<br>
5. Avoid adding extra white spaces randomly, e.g. hunk #2 of the<br>
patch, and avoid introducing white space new lines where one exists,<br>
creating 2 blank lines in a row.<br>
6. You may like to try to get git-send-email to work for you. It is a<br>
little nicer for submitting patches to mailing list.<br>
<a href="https://devel.rtems.org/wiki/Developer/Git/Users#Configuringgit-send-emailtouseGMail" rel="noreferrer" target="_blank">https://devel.rtems.org/wiki/<wbr>Developer/Git/Users#<wbr>Configuringgit-send-<wbr>emailtouseGMail</a><br>
<span class="HOEnZb"><font color="#888888"><br>
-Gedare<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
> Thanks and Regards<br>
> Abhinav Jain<br>
><br>
> On Tue, Feb 20, 2018 at 4:13 AM, Chris Johns <<a href="mailto:chrisj@rtems.org">chrisj@rtems.org</a>> wrote:<br>
>><br>
>> On 19/02/2018 21:19, Abhinav Jain wrote:<br>
>> > I have made the changes suggested by you in the code and hopefully, the<br>
>> > issue<br>
>> > will be resolved as if the .git file is not found in the directory, the<br>
>> > process<br>
>> > will not go ahead and hence the wrong git repository will not be<br>
>> > changed.<br>
>><br>
>> Excellent. Please post for review.<br>
>><br>
>> > I request to please guide me whether anything more is to be done in the<br>
>> > code or<br>
>> > should I proceed with a pull request.<br>
>><br>
>> I assume you mean a github pull request. RTEMS uses patches sent to this<br>
>> list<br>
>> for review. The top page of the Wiki has a section called RTEMS Developer<br>
>> Information and in that section are links to User Git access and<br>
>> submitting<br>
>> patches. You can also attach the patch to the ticket.<br>
>><br>
>> Chris<br>
><br>
><br>
</div></div></blockquote></div><br></div>