[Python-modules-team] Bug#963020: impacket breaks smbmap autopkgtest: No module named 'pkg_resources'

2020-06-18 Thread Raphael Hertzog
Hi,

On Thu, 18 Jun 2020, Raphael Hertzog wrote:
> Actually your fix does not work because pkg_resources is not documented in
> setup.py or requirements.txt and thus it will not magically appear in
> ${python3:Depends} just by adding it in Build-Depends.
> 
> I opened https://github.com/SecureAuthCorp/impacket/issues/885 upstream to
> get this fixed but in the meantime we have to hardcode the dependency in
> Depends. I'll do that for now.

Actually, I'm not sure that anything needs to happen at the upstream
level... pkg_resources is part of setuptools, we have packaged it
separately but I believe that there's no way for us to track its usage
based on upstream dependencies so we must always manually add such a
depency (this really smells like a big deficiency in our packaging tools).

Am I correct?

Should upstream list setuptools in their requirements.txt or setup.py
because they use "pkg_resources"?

Cheers,
-- 
  ⢀⣴⠾⠻⢶⣦⠀   Raphaël Hertzog 
  ⣾⠁⢠⠒⠀⣿⡁
  ⢿⡄⠘⠷⠚⠋The Debian Handbook: https://debian-handbook.info/get/
  ⠈⠳⣄   Debian Long Term Support: https://deb.li/LTS

___
Python-modules-team mailing list
Python-modules-team@alioth-lists.debian.net
https://alioth-lists.debian.net/cgi-bin/mailman/listinfo/python-modules-team

[Python-modules-team] Bug#963020: impacket breaks smbmap autopkgtest: No module named 'pkg_resources'

2020-06-18 Thread Raphael Hertzog
On Wed, 17 Jun 2020, Emmanuel Arias wrote:
> I've just pushed to salsa the fix, could anyone review it and sponsor
> it, please?

Actually your fix does not work because pkg_resources is not documented in
setup.py or requirements.txt and thus it will not magically appear in
${python3:Depends} just by adding it in Build-Depends.

I opened https://github.com/SecureAuthCorp/impacket/issues/885 upstream to
get this fixed but in the meantime we have to hardcode the dependency in
Depends. I'll do that for now.

Cheers,
> El mié., 17 de jun. de 2020 a la(s) 19:18, Emmanuel Arias
> (emmanuelaria...@gmail.com) escribió:
> >
> > Hi,
> >
> > I will fix it today
> >
> > Cheers,
> > Arias Emmanuel
> > @eamanu
> > http://eamanu.com
> >
> > El mié., 17 de jun. de 2020 a la(s) 19:18, Scott Kitterman
> > (deb...@kitterman.com) escribió:
> > >
> > > On Wed, 17 Jun 2020 21:12:40 +0200 Paul Gevers  wrote:
> > > > Source: impacket, smbmap
> > > > Control: found -1 impacket/0.9.21-1
> > > > Control: found -1 smbmap/1.8.2-1
> > > > Severity: serious
> > > > Tags: sid bullseye
> > > > X-Debbugs-CC: debian...@lists.debian.org
> > > > User: debian...@lists.debian.org
> > > > Usertags: breaks needs-update
> > > >
> > > > Dear maintainer(s),
> > > >
> > > > With a recent upload of impacket the autopkgtest of smbmap fails in
> > > > testing when that autopkgtest is run with the binary packages of
> > > > impacket from unstable. It passes when run with only packages from
> > > > testing. In tabular form:
> > > >
> > > >passfail
> > > > impacket   from testing0.9.21-1
> > > > smbmap from testing1.8.2-1
> > > > all others from testingfrom testing
> > > >
> > > > I copied some of the output at the bottom of this report. Is the latest
> > > > impacket missing a dependency or is the version function not supported
> > > > anymore?
> > > >
> > > > Currently this regression is blocking the migration of impacket to
> > > > testing [1]. Due to the nature of this issue, I filed this bug report
> > > > against both packages. Can you please investigate the situation and
> > > > reassign the bug to the right package?
> > > >
> > > > More information about this bug and the reason for filing it can be 
> > > > found on
> > > > https://wiki.debian.org/ContinuousIntegration/RegressionEmailInformation
> > > >
> > > > Paul
> > > >
> > > > [1] https://qa.debian.org/excuses.php?package=impacket
> > > >
> > > > https://ci.debian.net/data/autopkgtest/testing/amd64/s/smbmap/5914948/log.gz
> > > >
> > > > autopkgtest [07:08:34]: test command1: smbmap -h
> > > > autopkgtest [07:08:34]: test command1: [---
> > > > Traceback (most recent call last):
> > > >   File "/usr/bin/smbmap", line 19, in 
> > > > from impacket import version, smbserver
> > > >   File "/usr/lib/python3/dist-packages/impacket/version.py", line 7, in
> > > > 
> > > > import pkg_resources
> > > > ModuleNotFoundError: No module named 'pkg_resources'
> > > > autopkgtest [07:08:35]: test command1: ---]
> > > >
> > > Looks like a missing depends on python3-pkg-resources for impacket since
> > > that's where it's imported.
> > >
> > > Scott K
> 

-- 
  ⢀⣴⠾⠻⢶⣦⠀   Raphaël Hertzog 
  ⣾⠁⢠⠒⠀⣿⡁
  ⢿⡄⠘⠷⠚⠋The Debian Handbook: https://debian-handbook.info/get/
  ⠈⠳⣄   Debian Long Term Support: https://deb.li/LTS

___
Python-modules-team mailing list
Python-modules-team@alioth-lists.debian.net
https://alioth-lists.debian.net/cgi-bin/mailman/listinfo/python-modules-team

[Python-modules-team] Bug#963020: impacket breaks smbmap autopkgtest: No module named 'pkg_resources'

2020-06-18 Thread Raphael Hertzog
Hi,

On Wed, 17 Jun 2020, Emmanuel Arias wrote:
> I've just pushed to salsa the fix, could anyone review it and sponsor
> it, please?

I'll take care of it right now.

Cheers,
-- 
  ⢀⣴⠾⠻⢶⣦⠀   Raphaël Hertzog 
  ⣾⠁⢠⠒⠀⣿⡁
  ⢿⡄⠘⠷⠚⠋The Debian Handbook: https://debian-handbook.info/get/
  ⠈⠳⣄   Debian Long Term Support: https://deb.li/LTS

___
Python-modules-team mailing list
Python-modules-team@alioth-lists.debian.net
https://alioth-lists.debian.net/cgi-bin/mailman/listinfo/python-modules-team

[Python-modules-team] Bug#963020: impacket breaks smbmap autopkgtest: No module named 'pkg_resources'

2020-06-17 Thread Emmanuel Arias
I've just pushed to salsa the fix, could anyone review it and sponsor
it, please?

Cheers,
Arias Emmanuel
@eamanu
http://eamanu.com

El mié., 17 de jun. de 2020 a la(s) 19:18, Emmanuel Arias
(emmanuelaria...@gmail.com) escribió:
>
> Hi,
>
> I will fix it today
>
> Cheers,
> Arias Emmanuel
> @eamanu
> http://eamanu.com
>
> El mié., 17 de jun. de 2020 a la(s) 19:18, Scott Kitterman
> (deb...@kitterman.com) escribió:
> >
> > On Wed, 17 Jun 2020 21:12:40 +0200 Paul Gevers  wrote:
> > > Source: impacket, smbmap
> > > Control: found -1 impacket/0.9.21-1
> > > Control: found -1 smbmap/1.8.2-1
> > > Severity: serious
> > > Tags: sid bullseye
> > > X-Debbugs-CC: debian...@lists.debian.org
> > > User: debian...@lists.debian.org
> > > Usertags: breaks needs-update
> > >
> > > Dear maintainer(s),
> > >
> > > With a recent upload of impacket the autopkgtest of smbmap fails in
> > > testing when that autopkgtest is run with the binary packages of
> > > impacket from unstable. It passes when run with only packages from
> > > testing. In tabular form:
> > >
> > >passfail
> > > impacket   from testing0.9.21-1
> > > smbmap from testing1.8.2-1
> > > all others from testingfrom testing
> > >
> > > I copied some of the output at the bottom of this report. Is the latest
> > > impacket missing a dependency or is the version function not supported
> > > anymore?
> > >
> > > Currently this regression is blocking the migration of impacket to
> > > testing [1]. Due to the nature of this issue, I filed this bug report
> > > against both packages. Can you please investigate the situation and
> > > reassign the bug to the right package?
> > >
> > > More information about this bug and the reason for filing it can be found 
> > > on
> > > https://wiki.debian.org/ContinuousIntegration/RegressionEmailInformation
> > >
> > > Paul
> > >
> > > [1] https://qa.debian.org/excuses.php?package=impacket
> > >
> > > https://ci.debian.net/data/autopkgtest/testing/amd64/s/smbmap/5914948/log.gz
> > >
> > > autopkgtest [07:08:34]: test command1: smbmap -h
> > > autopkgtest [07:08:34]: test command1: [---
> > > Traceback (most recent call last):
> > >   File "/usr/bin/smbmap", line 19, in 
> > > from impacket import version, smbserver
> > >   File "/usr/lib/python3/dist-packages/impacket/version.py", line 7, in
> > > 
> > > import pkg_resources
> > > ModuleNotFoundError: No module named 'pkg_resources'
> > > autopkgtest [07:08:35]: test command1: ---]
> > >
> > Looks like a missing depends on python3-pkg-resources for impacket since
> > that's where it's imported.
> >
> > Scott K

___
Python-modules-team mailing list
Python-modules-team@alioth-lists.debian.net
https://alioth-lists.debian.net/cgi-bin/mailman/listinfo/python-modules-team

[Python-modules-team] Bug#963020: impacket breaks smbmap autopkgtest: No module named 'pkg_resources'

2020-06-17 Thread Emmanuel Arias
Hi,

I will fix it today

Cheers,
Arias Emmanuel
@eamanu
http://eamanu.com

El mié., 17 de jun. de 2020 a la(s) 19:18, Scott Kitterman
(deb...@kitterman.com) escribió:
>
> On Wed, 17 Jun 2020 21:12:40 +0200 Paul Gevers  wrote:
> > Source: impacket, smbmap
> > Control: found -1 impacket/0.9.21-1
> > Control: found -1 smbmap/1.8.2-1
> > Severity: serious
> > Tags: sid bullseye
> > X-Debbugs-CC: debian...@lists.debian.org
> > User: debian...@lists.debian.org
> > Usertags: breaks needs-update
> >
> > Dear maintainer(s),
> >
> > With a recent upload of impacket the autopkgtest of smbmap fails in
> > testing when that autopkgtest is run with the binary packages of
> > impacket from unstable. It passes when run with only packages from
> > testing. In tabular form:
> >
> >passfail
> > impacket   from testing0.9.21-1
> > smbmap from testing1.8.2-1
> > all others from testingfrom testing
> >
> > I copied some of the output at the bottom of this report. Is the latest
> > impacket missing a dependency or is the version function not supported
> > anymore?
> >
> > Currently this regression is blocking the migration of impacket to
> > testing [1]. Due to the nature of this issue, I filed this bug report
> > against both packages. Can you please investigate the situation and
> > reassign the bug to the right package?
> >
> > More information about this bug and the reason for filing it can be found on
> > https://wiki.debian.org/ContinuousIntegration/RegressionEmailInformation
> >
> > Paul
> >
> > [1] https://qa.debian.org/excuses.php?package=impacket
> >
> > https://ci.debian.net/data/autopkgtest/testing/amd64/s/smbmap/5914948/log.gz
> >
> > autopkgtest [07:08:34]: test command1: smbmap -h
> > autopkgtest [07:08:34]: test command1: [---
> > Traceback (most recent call last):
> >   File "/usr/bin/smbmap", line 19, in 
> > from impacket import version, smbserver
> >   File "/usr/lib/python3/dist-packages/impacket/version.py", line 7, in
> > 
> > import pkg_resources
> > ModuleNotFoundError: No module named 'pkg_resources'
> > autopkgtest [07:08:35]: test command1: ---]
> >
> Looks like a missing depends on python3-pkg-resources for impacket since
> that's where it's imported.
>
> Scott K

___
Python-modules-team mailing list
Python-modules-team@alioth-lists.debian.net
https://alioth-lists.debian.net/cgi-bin/mailman/listinfo/python-modules-team