<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html;
      charset=windows-1252">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <p><br>
    </p>
    <br>
    <div class="moz-cite-prefix">El 22/11/17 a las 12:51, Hugo Lefeuvre
      escribió:<br>
    </div>
    <blockquote type="cite"
      cite="mid:20171122115157.teeoev2sdqjhkzrx@hle-laptop.local">
      <pre wrap="">Hi Marcos,

</pre>
      <blockquote type="cite">
        <pre wrap="">That was a stupid mistake on my side. Its fixed now :-))
</pre>
      </blockquote>
      <pre wrap="">
Aww, I suspect you didn't test the package ;)</pre>
    </blockquote>
    Right :-( I had to admit that i followed blindly your recommendation
    to migrate to python3.<br>
    <br>
    I wrongly supposed that it should work fine.
    <blockquote type="cite"
      cite="mid:20171122115157.teeoev2sdqjhkzrx@hle-laptop.local">
      <pre wrap="">
When installing the resulting binary package I get the following error:

</pre>
      <blockquote type="cite">
        <pre wrap="">[...]
Setting up dnsrecon (0.8.11-1) ...
File "/usr/share/dnsrecon/tools/parser.py", line 96
  print elem.attrib['address']
           ^
SyntaxError: Missing parentheses in call to 'print'.
Did you mean print(print elem.attrib['address'])?
[...]
</pre>
      </blockquote>
      <pre wrap="">
Upstream bug. You can patch it easily by adding the missing parentheses[0].

... but ...

actually it looks like dnsrecon's Python 3 support is still a bit shaky

For example, the args parser seems to be halfway broken, the -h/--help option
isn't recognized anymore. And, quoting upstream[1]:

"Looks like I will have to refactor the code to work with 3.x"

"not abandone, just need to set time to figure how to keep it compatible
with python 2.7 and 3.x and python breaking stuff in minor changes does
not help, specially since Python is not a strength of mine"

So, even if Python 3 is officially supported[2], I think we'll have to
keep building for Python 2 for the moment.

So, here's what I propose:
 1) Revert the Python 3 changes. After that, everything should be fine
    and I'll upload.</pre>
    </blockquote>
    Agree, i reverted the switch to python3. Please check again now.<br>
    <blockquote type="cite"
      cite="mid:20171122115157.teeoev2sdqjhkzrx@hle-laptop.local">
      <pre wrap="">
 2) Work together with upstream towards a better Python 3 support.

    o issue #62: port code to argparse should fix our "-h" problem
    o issue #66: needs some testing, this should really be fixed
    o testing, testing ! :)</pre>
    </blockquote>
    I am not a python coder. But i'll try to help with betatesting,
    eventually.
    <blockquote type="cite"
      cite="mid:20171122115157.teeoev2sdqjhkzrx@hle-laptop.local">
      <pre wrap="">
 3) Prepare a Debian release building dnsrecon for Python 3 !</pre>
    </blockquote>
    Yes, i will drop python2 package when possible.<br>
    <blockquote type="cite"
      cite="mid:20171122115157.teeoev2sdqjhkzrx@hle-laptop.local">
      <pre wrap="">

Also, a little hint for your future packaging work: Always test your packages !</pre>
    </blockquote>
    I usually do it.
    <blockquote type="cite"
      cite="mid:20171122115157.teeoev2sdqjhkzrx@hle-laptop.local">
      <pre wrap="">
At least:
    * build in a clean chroot (+bonus points for running piuparts!)</pre>
    </blockquote>
    I always build package using git-pbuilder and test with piuparts
    (with -p option)
    <blockquote type="cite"
      cite="mid:20171122115157.teeoev2sdqjhkzrx@hle-laptop.local">
      <pre wrap="">    * lintian</pre>
    </blockquote>
    I use it and try to fix even pedantic tags.<br>
    <br>
    <blockquote type="cite"
      cite="mid:20171122115157.teeoev2sdqjhkzrx@hle-laptop.local">
      <pre wrap="">
    * debc (make sure content of the package is still what you want)
    * debdiff (check your changes carefully)
    * install (in a VM if you don't want to test your packages in your dev system)
    * test ! You should have a set of test files for each of your
      packages so you can make sure nothing gets broken by your changes
</pre>
    </blockquote>
    I will consider those recommendations. Thanks for your steering!<br>
    <blockquote type="cite"
      cite="mid:20171122115157.teeoev2sdqjhkzrx@hle-laptop.local">
      <pre wrap="">

Cheers,
 Hugo

[0] Could you submit this issue to upstream's bug tracker/Pull Request a fix ? :-)
[1] <a class="moz-txt-link-freetext" href="https://github.com/darkoperator/dnsrecon/issues/66">https://github.com/darkoperator/dnsrecon/issues/66</a>
[2] changelog 0.6.5

</pre>
    </blockquote>
    <blockquote type="cite"
      cite="mid:20171122115157.teeoev2sdqjhkzrx@hle-laptop.local">
    </blockquote>
    <br>
  </body>
</html>