package/python-service-identity: the attrs distribution is required

PATCH New Charles Hardin 4 participants 4 messages Jul 26, 2017 – Sep 4, 2017

lore

Share & tools

lore

Charles Hardin

author

2017-07-26 18:59 UTC #0 lore raw

PATCH 1/1 package/python-service-identity: the attrs distribution is required

Apparently the service identify code requires the python attrs to be availabe:

Traceback (most recent call last):
File "/opt/exablox/bin/configsrv", line 5, in <module>
from pkg_resources import load_entry_point
File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 3138, in <module>
@_call_aside
File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 3124, in _call_aside

    f(*args, **kwargs)
  File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 3151, in _initialize_master_working_set
    working_set = WorkingSet._build_master()
  File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 661, in _build_master
    ws.require(__requires__)

File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 962, in require
needed = self.resolve(parse_requirements(requirements))
File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 849, in resolve
raise DistributionNotFound(req, requirers)
pkg_resources.DistributionNotFound: The 'attrs' distribution was not found and is required by service-identity

Signed-off-by Charles Hardin <[email protected]>

---
 package/python-service-identity/Config.in | 1 +
 1 file changed, 1 insertion(+)

package/python-service-identity/Config.in +1 -0

index 304b38acde..73593b0d61 100644
@@ -1,6 +1,7 @@
 config BR2_PACKAGE_PYTHON_SERVICE_IDENTITY
 	bool "python-service-identity"
 	depends on BR2_INSTALL_LIBSTDCPP # python-pyopenssl, python-pyasn
+	select BR2_PACKAGE_PYTHON_ATTRS # runtime
 	select BR2_PACKAGE_PYTHON_CHARACTERISTIC # runtime
 	select BR2_PACKAGE_PYTHON_PYASN # runtime
 	select BR2_PACKAGE_PYTHON_PYASN_MODULES # runtime

Signature · 1 line

2.11.0 (Apple Git-81)

Yegor Yefremov

2017-07-26 19:11 UTC #1 lore raw

PATCH 1/1 package/python-service-identity: the attrs distribution is required

Quoted 14 lines · Charles Hardin

Apparently the service identify code requires the python attrs to be availabe:
Traceback (most recent call last):
File "/opt/exablox/bin/configsrv", line 5, in <module>
from pkg_resources import load_entry_point
File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 3138, in <module>
@_call_aside
File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 3124, in _call_aside
    f(*args, **kwargs)
  File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 3151, in _initialize_master_working_set
    working_set = WorkingSet._build_master()
  File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 661, in _build_master
    ws.require(__requires__)

Quoted 7 lines

File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 962, in require
needed = self.resolve(parse_requirements(requirements))
File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 849, in resolve
raise DistributionNotFound(req, requirers)
pkg_resources.DistributionNotFound: The 'attrs' distribution was not found and is required by service-identity
Signed-off-by Charles Hardin <[email protected]>

Reviewed-by Yegor Yefremov <[email protected]>

Quoted 19 lines

---
 package/python-service-identity/Config.in | 1 +
 1 file changed, 1 insertion(+)

package/python-service-identity/Config.in +1 -1

index 304b38acde..73593b0d61 100644
@@ -1,6 +1,7 @@
 config BR2_PACKAGE_PYTHON_SERVICE_IDENTITY
        bool "python-service-identity"
        depends on BR2_INSTALL_LIBSTDCPP # python-pyopenssl, python-pyasn
+       select BR2_PACKAGE_PYTHON_ATTRS # runtime
        select BR2_PACKAGE_PYTHON_CHARACTERISTIC # runtime
        select BR2_PACKAGE_PYTHON_PYASN # runtime
        select BR2_PACKAGE_PYTHON_PYASN_MODULES # runtime
--

2.11.0 (Apple Git-81)

Thomas Petazzoni

Applied

2017-07-29 13:40 UTC #2 lore raw

PATCH 1/1 package/python-service-identity: the attrs distribution is required

In reply to Charles Hardin

Hello,

Quoted 20 lines · Charles Hardin

Apparently the service identify code requires the python attrs to be availabe:
Traceback (most recent call last):
File "/opt/exablox/bin/configsrv", line 5, in <module>
from pkg_resources import load_entry_point
File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 3138, in <module>
@_call_aside
File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 3124, in _call_aside
    f(*args, **kwargs)
  File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 3151, in _initialize_master_working_set
    working_set = WorkingSet._build_master()
  File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 661, in _build_master
    ws.require(__requires__)
File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 962, in require
needed = self.resolve(parse_requirements(requirements))
File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 849, in resolve
raise DistributionNotFound(req, requirers)
pkg_resources.DistributionNotFound: The 'attrs' distribution was not found and is required by service-identity

Quoted 4 lines

Signed-off-by Charles Hardin <[email protected]>
---
 package/python-service-identity/Config.in | 1 +
 1 file changed, 1 insertion(+)

Applied to master, thanks.

Thomas

Signature · 3 lines

Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

Peter Korsgaard

2017-09-04 20:42 UTC #3 lore raw

PATCH 1/1 package/python-service-identity: the attrs distribution is required

Quoted 24 lines

Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line
"Thomas" == Thomas Petazzoni <[email protected]> writes:
Hello,
Quoted 20 lines · Charles Hardin
Apparently the service identify code requires the python attrs to be availabe:
Traceback (most recent call last):
File "/opt/exablox/bin/configsrv", line 5, in <module>
from pkg_resources import load_entry_point
File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 3138, in <module> @_call_aside
File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 3124, in _call_aside f(*args, **kwargs)
File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 3151, in _initialize_master_working_set working_set = WorkingSet._build_master()
File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 661, in _build_master ws.require(__requires__)
File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 962, in require needed = self.resolve(parse_requirements(requirements))
File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 849, in resolve
raise DistributionNotFound(req, requirers)
pkg_resources.DistributionNotFound: The 'attrs' distribution was not found and is required by service-identity

Quoted 6 lines

Quoted 4 lines
Signed-off-by Charles Hardin <[email protected]>
---
package/python-service-identity/Config.in | 1 +
1 file changed, 1 insertion(+)
Applied to master, thanks.

While looking at this commit for backporting to 2017.02.x, it seems like the select of BR2_PACKAGE_PYTHON_CHARACTERISTIC should be dropped after the bump to 16.0.0, as it is isn't listed in INSTALL_REQUIRES. PKG-INFO states:

Changes:
- Port from ``characteristic`` to its spiritual successor attrs <https://attrs.readthedocs.org/>_.

Care to verify and send a patch to drop the select if OK?

In the mean time I've applied this patch to the 2017.02.x series, thanks.

Signature · 1 line

Bye, Peter Korsgaard