<div dir="ltr"><div><div>Sir,</div><div><br></div><div>I have tried to incorporate the changes suggested by you. </div><div>I request you to please check the new patch file and guide me if some more improvement is needed to be done.</div><div><br></div><div>Thanks and Regards </div><div>Abhinav Jain</div></div><div><br></div><div><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Feb 20, 2018 at 8:27 PM, Abhinav Jain <span dir="ltr"><<a href="mailto:jainab.2009@gmail.com" target="_blank">jainab.2009@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><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><span class="gmail-"><div><br></div><div>Thanks and Regards </div><div>Abhinav Jain</div></span></div><div class="gmail-HOEnZb"><div class="gmail-h5"><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:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Oops, I should read my mail better. Thanks for breaking this out Chris. Abhinav,<br>
<span><br>
On Tue, Feb 20, 2018 at 1:12 AM, Abhinav Jain <<a href="mailto:jainab.2009@gmail.com" target="_blank">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/D<wbr>eveloper/Git/Users#Configuring<wbr>git-send-emailtouseGMail</a><br>
<span class="gmail-m_-4420512945621906618HOEnZb"><font color="#888888"><br>
-Gedare<br>
</font></span><div class="gmail-m_-4420512945621906618HOEnZb"><div class="gmail-m_-4420512945621906618h5"><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" target="_blank">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>
</div></div></blockquote></div><br></div></div>