<div dir="ltr">Hello,<br>How does the community test the rtems-tools changes?<br>In the README I can see a few steps, Is that the standard method?, It is my first patch for rtems, help is really appreciated. <br><br>Best Regards</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Mar 20, 2020 at 3:45 AM Gedare Bloom <<a href="mailto:gedare@rtems.org">gedare@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Anmol,<br>
<br>
On Thu, Mar 19, 2020 at 1:41 PM Anmol Mishra <<a href="mailto:anmol.j2020@gmail.com" target="_blank">anmol.j2020@gmail.com</a>> wrote:<br>
><br>
> Hello,<br>
> I can understand your requests, I also thought to squash/combine all changes in 1 patch. I will complete the task in a day and resend the patch. I did not know the protocol/contributing guide. I will keep a note of this. Sorry for the trouble.<br>
><br>
<br>
One more thing, can you please identify what, if any, testing of your<br>
patches that you did?<br>
<br>
It would be good to know they have been tried out :)<br>
<br>
> Regards<br>
><br>
> On Thu, Mar 19, 2020 at 9:53 PM Gedare Bloom <<a href="mailto:gedare@rtems.org" target="_blank">gedare@rtems.org</a>> wrote:<br>
>><br>
>> Hi Anmol,<br>
>><br>
>> Thanks for your patches. I have a few requests.<br>
>><br>
>> Can you please provide some details in your commit message? In<br>
>> particular, since few of us right now know how to make code compliant<br>
>> between Python 2 and 3, can you provide a brief<br>
>> justification/description for what the changes are and how they<br>
>> address this problem?<br>
>><br>
>> In general you can send your first set of patches without a -v1. That<br>
>> is implied.<br>
>><br>
>> When you send a lot of patches it is good if you also prepare a<br>
>> git-cover-letter (patch 00/xx) to explain the patch set.<br>
>><br>
>> Instead of going file-by-file with patches, you might also consider<br>
>> organizing patches by addressing "kinds" of compliance fixes. For<br>
>> example, a single patch could sweep up a bunch of "from __future__<br>
>> import ..." across several files. Either way is really OK, patches<br>
>> that touch just one file are less likely to conflict, but a patch that<br>
>> does one thing across many files is easier to review/approve. I'll<br>
>> leave it up to you how you want to proceed.<br>
>><br>
>><br>
>><br>
>> On Wed, Mar 18, 2020 at 6:19 AM Anmol mishra <<a href="mailto:anmol.j2020@gmail.com" target="_blank">anmol.j2020@gmail.com</a>> wrote:<br>
>> ><br>
>> > From: Anmol Mishra <<a href="mailto:anmol.j2020@gmail.com" target="_blank">anmol.j2020@gmail.com</a>><br>
>> ><br>
>> > ---<br>
>> >  tools/gdb/python/__init__.py | 5 ++++-<br>
>> >  1 file changed, 4 insertions(+), 1 deletion(-)<br>
>> ><br>
>> > diff --git a/tools/gdb/python/__init__.py b/tools/gdb/python/__init__.py<br>
>> > index 0ea5fb2..dfbe6ba 100644<br>
>> > --- a/tools/gdb/python/__init__.py<br>
>> > +++ b/tools/gdb/python/__init__.py<br>
>> > @@ -27,8 +27,11 @@<br>
>> >  # POSSIBILITY OF SUCH DAMAGE.<br>
>> >  #<br>
>> ><br>
>> > +from __future__ import print_function<br>
>> > +from __future__ import absolute_import<br>
>> > +<br>
>> >  import gdb<br>
>> > -import rtems<br>
>> > +from . import rtems<br>
>> ><br>
>> >  def get_architure():<br>
>> >      frame = gdb.selected_frame()<br>
>> > --<br>
>> > 2.21.0 (Apple Git-122.2)<br>
>> ><br>
>> > _______________________________________________<br>
>> > devel mailing list<br>
>> > <a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
>> > <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div>