RSB valid status check (was: Contribute to project)

Abhinav Jain jainab.2009 at gmail.com
Tue Feb 20 16:47:09 UTC 2018


Sir,

I have tried to incorporate the changes suggested by you.
I request you to please check the new patch file and guide me if some more
improvement is needed to be done.

Thanks and Regards
Abhinav Jain



On Tue, Feb 20, 2018 at 8:27 PM, Abhinav Jain <jainab.2009 at gmail.com> wrote:

> Sir,
>
> I will edit the current patch accordingly and will take care of this in
> the future.
>
> Thanks and Regards
> Abhinav Jain
>
> On Tue, Feb 20, 2018 at 7:31 PM, Gedare Bloom <gedare at rtems.org> wrote:
>
>> Oops, I should read my mail better. Thanks for breaking this out Chris.
>> Abhinav,
>>
>> On Tue, Feb 20, 2018 at 1:12 AM, Abhinav Jain <jainab.2009 at gmail.com>
>> wrote:
>> > Sir,
>> >
>> > I have attached the patch file with this mail. I have tried to follow
>> all
>> > the conventions that were listed in the User Git page. I request you to
>> > please check and guide me whether I have done it correctly or not or
>> whether
>> > something more is to be done.
>> >
>> I won't comment on the code, I'll let Chris do that. However, this
>> patch has a few issues that should be addressed.
>> 1. Use line breaks in your commit message. About 70-80 characters per
>> line max, please.
>> 2. The "short message" can omit "function added to", basically all
>> patches add some code, so this is a bit redundant. You can just say
>> "Check the validity ..."
>> 3. The commit message should use "Closes #nnnn." somewhere, usually in
>> the end of the commit, if it is fixing/closing a ticket.
>> 4. The commit message may be less verbose, if the extra details about
>> the change are already in the ticket.
>> 5. Avoid adding extra white spaces randomly, e.g. hunk #2 of the
>> patch, and avoid introducing white space new lines where one exists,
>> creating 2 blank lines in a row.
>> 6. You may like to try to get git-send-email to work for you. It is a
>> little nicer for submitting patches to mailing list.
>> https://devel.rtems.org/wiki/Developer/Git/Users#Configuring
>> git-send-emailtouseGMail
>>
>> -Gedare
>>
>> > Thanks and Regards
>> > Abhinav Jain
>> >
>> > On Tue, Feb 20, 2018 at 4:13 AM, Chris Johns <chrisj at rtems.org> wrote:
>> >>
>> >> On 19/02/2018 21:19, Abhinav Jain wrote:
>> >> > I have made the changes suggested by you in the code and hopefully,
>> the
>> >> > issue
>> >> > will be resolved as if the .git file is not found in the directory,
>> the
>> >> > process
>> >> > will not go ahead and hence the wrong git repository will not be
>> >> > changed.
>> >>
>> >> Excellent. Please post for review.
>> >>
>> >> > I request to please guide me whether anything more is to be done in
>> the
>> >> > code or
>> >> > should I proceed with a pull request.
>> >>
>> >> I assume you mean a github pull request. RTEMS uses patches sent to
>> this
>> >> list
>> >> for review. The top page of the Wiki has a section called RTEMS
>> Developer
>> >> Information and in that section are links to User Git access and
>> >> submitting
>> >> patches. You can also attach the patch to the ticket.
>> >>
>> >> Chris
>> >
>> >
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/devel/attachments/20180220/610a8dd4/attachment-0002.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Validity_git_repo.patch
Type: text/x-patch
Size: 2415 bytes
Desc: not available
URL: <http://lists.rtems.org/pipermail/devel/attachments/20180220/610a8dd4/attachment-0002.bin>


More information about the devel mailing list