<div dir="ltr">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>Regards</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">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></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>
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>