<div dir="ltr"><br><div class="gmail_extra"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div>Hi, all !</div></div></div></div></div></div>Pabs, Thanks for suggestions :) </div><div class="gmail_extra">I made the changes as you mentioned, please find an attached patches. </div><div class="gmail_extra"><br></div><div class="gmail_extra"><div class="gmail_quote">On Sun, Jun 12, 2016 at 1:31 PM, Paul Wise <span dir="ltr"><<a href="mailto:pabs@debian.org" target="_blank">pabs@debian.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">On Sun, 2016-06-12 at 11:50 +0530, Satyam Zode wrote:<br>
<br>
> Tags: patch<br>
<br>
That only works in mails to <a href="mailto:submit@bugs.debian.org">submit@bugs.debian.org</a>, for mails to the<br>
bug report address, you want this instead (-1 means the current bug):<br>
<br>
Control: tags -1 + patch<br>
<span class=""><br>
> I have updated the patch as per suggestions please review it.<br>
<br>
</span>Review below...<br>
<br>
>  override_dh_clean:<br>
> +     debian/diffoscope.bash-completion<br>
<br>
This causes the package to fail to build. I think override_dh_clean<br>
should be removed and both the bash completion script and<br>
debian/diffoscope.1 added to debian/clean. Moving the existing things<br>
to the debian/clean file should be done in a separate commit.<br>
<br>
>       dh_python3 --recommends=python-debian --recommends=rpm-python --recommends=tlsh --recommends=guestfs<br>
<br>
This line needs to add --recommends=argcomplete<br>
<br>
> +override_dh_auto_build:<br>
> +     register-python-argcomplete diffoscope > debian/diffoscope.bash-completion<br>
<br>
This disables the upstream build system, you should call dh_auto_build<br>
after calling register-python-argcomplete, I think this is a better way<br>
to do it than what is there now:<br>
<br>
override_dh_auto_build: debian/diffoscope.bash-completion debian/diffoscope.1<br>
<br>
debian/diffoscope.bash-completion:<br>
        register-python-argcomplete diffoscope > $@<br></blockquote><div>I made above changes and here is build log : <a href="http://paste.debian.net/740041/">http://paste.debian.net/740041/</a></div><div>I am not sure if anything else is needed or not in debian/* files.</div><div>   </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
You also need to add generated files to .gitignore so they don't get<br>
accidentally committed to the repository.<br>
<br>
> +* ``python-argcomplete`` is used for argument completion.<br>
> +  Available on Debian and Fedora as<br>
> +  ``python-argcomplete``.<br>
> +  ``python-argcomplete`` is also available on `PyPI ` as<br>
> +  ``argcomplete``.<br>
<br>
I think I would write that like this:<br>
<br>
* ``argcomplete`` is used for argument completion.<br>
  Available on Debian as ``python3-argcomplete``.<br>
  Available on Fedora as ``python-argcomplete``.<br>
  Available on `PyPI <<a href="https://pypi.python.org/pypi/argcomplete/" rel="noreferrer" target="_blank">https://pypi.python.org/pypi/argcomplete/</a>>`_.<br>
<br></blockquote><div><br></div><div>I have made these changes too. </div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
> -                        default=Config.general.max_report_size)<br>
> +                        default=Config.general.max_report_size).completer=RangeCompleter(0,<br>
> +                        Config.general.max_report_size, 200000)<br>
<br>
I think this prevents people from completing values above the<br>
default max_report_size? Same for the other RangeCompleters.<br>
<br>
Interestingly, in the range completers, 1000000 appears to sort before<br>
800000, so it is doing string sorting not numeric sorting.<br></blockquote><div> </div><div>I have removed this temporarily! we already had discussion regarding this on IRC. </div><div>Looking forward to your response! </div><div><br></div><div><br></div><div>Thanks! </div><div>Satyam Zode</div></div><br></div></div>