python-service-identity: new package

PATCH New Mauro Condarelli 3 participants 16 messages Feb 11, 2016 – Feb 20, 2016

lore

Share & tools

lore

Mauro Condarelli

author

2016-02-11 10:58 UTC #0 lore raw

PATCH v5 18/20 python-service-identity: new package

Signed-off-by Mauro Condarelli <[email protected]>

---
Changes v4 -> v5
- removed spurous (runtime) depemndencies in .mk

Signed-off-by Mauro Condarelli <[email protected]>

---
 package/Config.in                                          |  1 +
 package/python-service-identity/Config.in                  | 10 ++++++++++
 .../python-service-identity/python-service-identity.hash   |  3 +++
 package/python-service-identity/python-service-identity.mk | 14 ++++++++++++++
 4 files changed, 28 insertions(+)
 create mode 100644 package/python-service-identity/Config.in
 create mode 100644 package/python-service-identity/python-service-identity.hash
 create mode 100644 package/python-service-identity/python-service-identity.mk

package/Config.in +1 -0

index 5907ad6..d1d0ec7 100644
@@ -713,6 +713,7 @@ menu "External python modules"
 	source "package/python-rtslib-fb/Config.in"
 	source "package/python-serial/Config.in"
 	source "package/python-setproctitle/Config.in"
+	source "package/python-service-identity/Config.in"
 	source "package/python-setuptools/Config.in"
 	source "package/python-shutilwhich/Config.in"
 	source "package/python-simplejson/Config.in"

package/python-service-identity/Config.in new file +10 -0

new file mode 100644
index 0000000..e2200f6
@@ -0,0 +1,10 @@
+config BR2_PACKAGE_PYTHON_SERVICE_IDENTITY
+	bool "python-service-identity"
+	select BR2_PACKAGE_PYTHON_CHARACTERISTIC # runtime
+	select BR2_PACKAGE_PYTHON_PYASN # runtime
+	select BR2_PACKAGE_PYTHON_PYASN_MODULES # runtime
+	select BR2_PACKAGE_PYTHON_PYOPENSSL # runtime
+	help
+	  Service identity verification for pyOpenSSL.
+
+	  https://pypi.python.org/pypi/service_identity

package/python-service-identity/python-service-identity.hash new file +3 -0

new file mode 100644
index 0000000..caf29dc
@@ -0,0 +1,3 @@
+# md5 from https://pypi.python.org/pypi?:action=show_md5&digest=cea0b0156d73b025ecef660fb51f0d9a, sha256 locally computed.
+md5	cea0b0156d73b025ecef660fb51f0d9a  service_identity-14.0.0.tar.gz
+sha256	3105a319a7c558490666694f599be0c377ad54824eefb404cde4ce49e74a4f5a  service_identity-14.0.0.tar.gz

package/python-service-identity/python-service-identity.mk new file +14 -0

new file mode 100644
index 0000000..9eca3c1
@@ -0,0 +1,14 @@
+################################################################################
+#
+# python-service-identity
+#
+################################################################################
+
+PYTHON_SERVICE_IDENTITY_VERSION = 14.0.0
+PYTHON_SERVICE_IDENTITY_SOURCE = service_identity-$(PYTHON_SERVICE_IDENTITY_VERSION).tar.gz
+PYTHON_SERVICE_IDENTITY_SITE = http://pypi.python.org/packages/source/s/service_identity
+PYTHON_SERVICE_IDENTITY_LICENSE = MIT
+PYTHON_SERVICE_IDENTITY_LICENSE_FILES = LICENSE
+PYTHON_SERVICE_IDENTITY_SETUP_TYPE = setuptools
+
+$(eval $(python-package))

Signature · 1 line

2.1.4

Mauro Condarelli

author

2016-02-11 10:58 UTC #1 lore raw

PATCH v5 19/20 python-autobahn: new package

Signed-off-by Mauro Condarelli <[email protected]>

---
Changes v4 -> v5
- added dependency from Python sources.

Signed-off-by Mauro Condarelli <[email protected]>

---
 package/Config.in                            |  1 +
 package/python-autobahn/Config.in            | 11 +++++++++++
 package/python-autobahn/python-autobahn.hash |  3 +++
 package/python-autobahn/python-autobahn.mk   | 14 ++++++++++++++
 4 files changed, 29 insertions(+)
 create mode 100644 package/python-autobahn/Config.in
 create mode 100644 package/python-autobahn/python-autobahn.hash
 create mode 100644 package/python-autobahn/python-autobahn.mk

package/Config.in +1 -0

index d1d0ec7..4b92a2a 100644
@@ -618,6 +618,7 @@ endif
 if BR2_PACKAGE_PYTHON || BR2_PACKAGE_PYTHON3
 menu "External python modules"
 	source "package/python-alsaaudio/Config.in"
+	source "package/python-autobahn/Config.in"
 	source "package/python-backports-abc/Config.in"
 	source "package/python-beautifulsoup4/Config.in"
 	source "package/python-bottle/Config.in"

package/python-autobahn/Config.in new file +11 -0

new file mode 100644
index 0000000..9f5da46
@@ -0,0 +1,11 @@
+config BR2_PACKAGE_PYTHON_AUTOBAHN
+	bool "python-autobahn"
+	depends on !BR2_PACKAGE_PYTHON_PYC_ONLY
+	select BR2_PACKAGE_PYTHON_SIX # runtime
+	select BR2_PACKAGE_PYTHON_TXAIO # runtime
+	help
+	  WebSocket client Hhh server library, WAMP real-time framework.
+
+	  https://pypi.python.org/pypi/autobahn
+comment "Currently Crossbar.io does NOT work without Python sources!"
+	depends on BR2_PACKAGE_PYTHON_PYC_ONLY

package/python-autobahn/python-autobahn.hash new file +3 -0

new file mode 100644
index 0000000..8f55ac1
@@ -0,0 +1,3 @@
+# md5 from https://pypi.python.org/pypi?:action=show_md5&digest=b4b5445050749c1033b9b58f694f0c64, sha256 locally computed.
+md5	b4b5445050749c1033b9b58f694f0c64  autobahn-0.12.1.tar.gz
+sha256	664223879e159c88221f42d8d1ac6b8c4268d8b9316d8ab69a02761c5744cd79  autobahn-0.12.1.tar.gz

package/python-autobahn/python-autobahn.mk new file +14 -0

new file mode 100644
index 0000000..1d02867
@@ -0,0 +1,14 @@
+################################################################################
+#
+# python-autobahn
+#
+################################################################################
+
+PYTHON_AUTOBAHN_VERSION = 0.12.1
+PYTHON_AUTOBAHN_SOURCE = autobahn-$(PYTHON_AUTOBAHN_VERSION).tar.gz
+PYTHON_AUTOBAHN_SITE = http://pypi.python.org/packages/source/a/autobahn
+PYTHON_AUTOBAHN_LICENSE = MIT
+PYTHON_AUTOBAHN_LICENSE_FILES = LICENSE
+PYTHON_AUTOBAHN_SETUP_TYPE = setuptools
+
+$(eval $(python-package))

Signature · 1 line

2.1.4

Yegor Yefremov

2016-02-11 14:10 UTC #2 lore raw

PATCH v5 19/20 python-autobahn: new package

Quoted 36 lines · Mauro Condarelli

Signed-off-by Mauro Condarelli <[email protected]>

---
Changes v4 -> v5
- added dependency from Python sources.

Signed-off-by Mauro Condarelli <[email protected]>

---
 package/Config.in                            |  1 +
 package/python-autobahn/Config.in            | 11 +++++++++++
 package/python-autobahn/python-autobahn.hash |  3 +++
 package/python-autobahn/python-autobahn.mk   | 14 ++++++++++++++
 4 files changed, 29 insertions(+)
 create mode 100644 package/python-autobahn/Config.in
 create mode 100644 package/python-autobahn/python-autobahn.hash
 create mode 100644 package/python-autobahn/python-autobahn.mk

package/Config.in +1 -0

index d1d0ec7..4b92a2a 100644
@@ -618,6 +618,7 @@ endif
 if BR2_PACKAGE_PYTHON || BR2_PACKAGE_PYTHON3
 menu "External python modules"
        source "package/python-alsaaudio/Config.in"
+       source "package/python-autobahn/Config.in"
        source "package/python-backports-abc/Config.in"
        source "package/python-beautifulsoup4/Config.in"
        source "package/python-bottle/Config.in"

package/python-autobahn/Config.in new file +1 -0

new file mode 100644
index 0000000..9f5da46
@@ -0,0 +1,11 @@
+config BR2_PACKAGE_PYTHON_AUTOBAHN

package/python-autobahn/Config.in line 11

Quoted 10 lines

package/python-autobahn/Config.in new file +10 -0

+       bool "python-autobahn"
+       depends on !BR2_PACKAGE_PYTHON_PYC_ONLY
+       select BR2_PACKAGE_PYTHON_SIX # runtime
+       select BR2_PACKAGE_PYTHON_TXAIO # runtime
+       help
+         WebSocket client Hhh server library, WAMP real-time framework.
+
+         https://pypi.python.org/pypi/autobahn
+comment "Currently Crossbar.io does NOT work without Python sources!"
+       depends on BR2_PACKAGE_PYTHON_PYC_ONLY

It is actually autobahn own problem, that has nothing to do with crossbar. It imports some python modules during setup.py and some of them try to load arm *.so files on the host. This seems to happen only when producing *.pyc files.

Quoted 36 lines

package/python-autobahn/python-autobahn.hash new file +3 -0

new file mode 100644
index 0000000..8f55ac1
@@ -0,0 +1,3 @@
+# md5 from https://pypi.python.org/pypi?:action=show_md5&digest=b4b5445050749c1033b9b58f694f0c64, sha256 locally computed.
+md5    b4b5445050749c1033b9b58f694f0c64  autobahn-0.12.1.tar.gz
+sha256 664223879e159c88221f42d8d1ac6b8c4268d8b9316d8ab69a02761c5744cd79  autobahn-0.12.1.tar.gz

package/python-autobahn/python-autobahn.mk new file +14 -1

new file mode 100644
index 0000000..1d02867
@@ -0,0 +1,14 @@
+################################################################################
+#
+# python-autobahn
+#
+################################################################################
+
+PYTHON_AUTOBAHN_VERSION = 0.12.1
+PYTHON_AUTOBAHN_SOURCE = autobahn-$(PYTHON_AUTOBAHN_VERSION).tar.gz
+PYTHON_AUTOBAHN_SITE = http://pypi.python.org/packages/source/a/autobahn
+PYTHON_AUTOBAHN_LICENSE = MIT
+PYTHON_AUTOBAHN_LICENSE_FILES = LICENSE
+PYTHON_AUTOBAHN_SETUP_TYPE = setuptools
+
+$(eval $(python-package))
--

2.1.4

_______________________________________________
buildroot mailing list
buildroot at busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot

Thomas Petazzoni

2016-02-20 14:05 UTC #3 lore raw

PATCH v5 19/20 python-autobahn: new package

In reply to Mauro Condarelli

Mauro,

package/python-autobahn/Config.in line 3

Quoted 9 lines · Mauro Condarelli

package/python-autobahn/Config.in new file +3 -0

new file mode 100644
index 0000000..9f5da46
@@ -0,0 +1,11 @@
+config BR2_PACKAGE_PYTHON_AUTOBAHN
+	bool "python-autobahn"
+	depends on !BR2_PACKAGE_PYTHON_PYC_ONLY

I did a build with BR2_PACKAGE_PYTHON_PYC_ONLY=y, and tested at runtime "import autobahn", and it worked perfectly fine.

So, I've removed this dependency, and applied your patch to the next branch. If you think the dependency is really needed, please send a follow-up patch on top of the next branch to re-add it, with a proper justification.

Thanks,

Thomas

Signature · 3 lines

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

Yegor Yefremov

2016-02-20 15:53 UTC #4 lore raw

PATCH v5 19/20 python-autobahn: new package

Thomas,

Quoted 17 lines · Thomas Petazzoni

Mauro,
package/python-autobahn/Config.in line 3
Quoted 9 lines · Mauro Condarelli
package/python-autobahn/Config.in new file +3 -0
new file mode 100644
index 0000000..9f5da46
@@ -0,0 +1,11 @@
+config BR2_PACKAGE_PYTHON_AUTOBAHN
+     bool "python-autobahn"
+     depends on !BR2_PACKAGE_PYTHON_PYC_ONLY
I did a build with BR2_PACKAGE_PYTHON_PYC_ONLY=y, and tested at runtime "import autobahn", and it worked perfectly fine.

Quoted 4 lines

So, I've removed this dependency, and applied your patch to the next branch. If you think the dependency is really needed, please send a follow-up patch on top of the next branch to re-add it, with a proper justification.

Try to compile autobahn with PYC only and twisted and python-pynacl-1.0.1 enabled. I get:

Quoted 1 line

Quoted 1 line
Quoted 1 line
python-autobahn 0.12.1 Building

(cd /home/user/Documents/versioned/buildroot/testing/build/python-autobahn-0.12.1//; PATH="/home/user/Documents/versioned/buildroot/testing/host/bin:/home/user/Documents/versioned/buildroot/testing/host/sbin:/home/user/Documents/versioned/buildroot/testing/host/usr/bin:/home/user/Documents/versioned/buildroot/testing/host/usr/sbin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games" PYTHONPATH="/home/user/Documents/versioned/buildroot/testing/target/usr/lib/python3.4/sysconfigdata/:/home/user/Documents/versioned/buildroot/testing/target/usr/lib/python3.4/site-packages/" _python_sysroot=/home/user/Documents/versioned/buildroot/testing/host/usr/arm-buildroot-linux-gnueabi/sysroot
_python_prefix=/usr _python_exec_prefix=/usr
/home/user/Documents/versioned/buildroot/testing/host/usr/bin/python
setup.py build )
running build
running build_py
creating build
creating build/lib
creating build/lib/autobahn
copying autobahn/__init__.py -> build/lib/autobahn
copying autobahn/util.py -> build/lib/autobahn
creating build/lib/autobahn/wamp
copying autobahn/wamp/__init__.py -> build/lib/autobahn/wamp
copying autobahn/wamp/keyring.py -> build/lib/autobahn/wamp
copying autobahn/wamp/component.py -> build/lib/autobahn/wamp
copying autobahn/wamp/message.py -> build/lib/autobahn/wamp
copying autobahn/wamp/websocket.py -> build/lib/autobahn/wamp
copying autobahn/wamp/serializer.py -> build/lib/autobahn/wamp
copying autobahn/wamp/auth.py -> build/lib/autobahn/wamp
copying autobahn/wamp/cryptosign.py -> build/lib/autobahn/wamp
copying autobahn/wamp/protocol.py -> build/lib/autobahn/wamp
copying autobahn/wamp/types.py -> build/lib/autobahn/wamp
copying autobahn/wamp/exception.py -> build/lib/autobahn/wamp
copying autobahn/wamp/role.py -> build/lib/autobahn/wamp
copying autobahn/wamp/uri.py -> build/lib/autobahn/wamp
copying autobahn/wamp/interfaces.py -> build/lib/autobahn/wamp
copying autobahn/wamp/request.py -> build/lib/autobahn/wamp
creating build/lib/autobahn/wamp/test
copying autobahn/wamp/test/__init__.py -> build/lib/autobahn/wamp/test
copying autobahn/wamp/test/test_auth.py -> build/lib/autobahn/wamp/test
copying autobahn/wamp/test/test_protocol.py -> build/lib/autobahn/wamp/test
copying autobahn/wamp/test/test_uri_pattern.py -> build/lib/autobahn/wamp/test
copying autobahn/wamp/test/test_component.py -> build/lib/autobahn/wamp/test
copying autobahn/wamp/test/test_message.py -> build/lib/autobahn/wamp/test
copying autobahn/wamp/test/test_runner.py -> build/lib/autobahn/wamp/test
copying autobahn/wamp/test/test_protocol_peer.py -> build/lib/autobahn/wamp/test
copying autobahn/wamp/test/test_websocket.py -> build/lib/autobahn/wamp/test
copying autobahn/wamp/test/test_serializer.py -> build/lib/autobahn/wamp/test
copying autobahn/wamp/test/test_exception.py -> build/lib/autobahn/wamp/test
copying autobahn/wamp/test/test_user_handler_errors.py ->
build/lib/autobahn/wamp/test
creating build/lib/autobahn/websocket
copying autobahn/websocket/__init__.py -> build/lib/autobahn/websocket
copying autobahn/websocket/utf8validator.py -> build/lib/autobahn/websocket
copying autobahn/websocket/compress_base.py -> build/lib/autobahn/websocket
copying autobahn/websocket/compress_bzip2.py -> build/lib/autobahn/websocket
copying autobahn/websocket/compress_deflate.py -> build/lib/autobahn/websocket
copying autobahn/websocket/compress.py -> build/lib/autobahn/websocket
copying autobahn/websocket/protocol.py -> build/lib/autobahn/websocket
copying autobahn/websocket/types.py -> build/lib/autobahn/websocket
copying autobahn/websocket/compress_snappy.py -> build/lib/autobahn/websocket
copying autobahn/websocket/interfaces.py -> build/lib/autobahn/websocket
copying autobahn/websocket/xormasker.py -> build/lib/autobahn/websocket
creating build/lib/autobahn/websocket/test
copying autobahn/websocket/test/__init__.py -> build/lib/autobahn/websocket/test
copying autobahn/websocket/test/test_protocol.py ->
build/lib/autobahn/websocket/test
copying autobahn/websocket/test/test_websocket_url.py ->
build/lib/autobahn/websocket/test
copying autobahn/websocket/test/test_websocket.py ->
build/lib/autobahn/websocket/test
creating build/lib/autobahn/asyncio
copying autobahn/asyncio/__init__.py -> build/lib/autobahn/asyncio
copying autobahn/asyncio/wamp.py -> build/lib/autobahn/asyncio
copying autobahn/asyncio/websocket.py -> build/lib/autobahn/asyncio
creating build/lib/autobahn/twisted
copying autobahn/twisted/__init__.py -> build/lib/autobahn/twisted
copying autobahn/twisted/forwarder.py -> build/lib/autobahn/twisted
copying autobahn/twisted/wamp.py -> build/lib/autobahn/twisted
copying autobahn/twisted/resource.py -> build/lib/autobahn/twisted
copying autobahn/twisted/component.py -> build/lib/autobahn/twisted
copying autobahn/twisted/websocket.py -> build/lib/autobahn/twisted
copying autobahn/twisted/choosereactor.py -> build/lib/autobahn/twisted
copying autobahn/twisted/util.py -> build/lib/autobahn/twisted
copying autobahn/twisted/rawsocket.py -> build/lib/autobahn/twisted
package init file 'twisted/plugins/__init__.py' not found (or not a
regular file)
creating build/lib/twisted
creating build/lib/twisted/plugins
copying twisted/plugins/autobahn_endpoints.py -> build/lib/twisted/plugins
copying twisted/plugins/autobahn_twistd.py -> build/lib/twisted/plugins
Twisted found (default reactor is <class
'twisted.internet.epollreactor.EPollReactor'>)
Unhandled Error
Traceback (most recent call last):
File "setup.py", line 256, in <module>
list(getPlugins(IPlugin))
File "home/user/Documents/versioned/buildroot/testing/target/usr/lib/python3.4/site-packages/twisted/plugin.py",
line 213, in getPlugins

--- <exception caught here> ---
File "home/user/Documents/versioned/buildroot/testing/target/usr/lib/python3.4/site-packages/twisted/plugin.py",
line 171, in getCache

File "home/user/Documents/versioned/buildroot/testing/target/usr/lib/python3.4/site-packages/twisted/python/modules.py",
line 389, in load

File "home/user/Documents/versioned/buildroot/testing/target/usr/lib/python3.4/site-packages/twisted/python/reflect.py",
line 303, in namedAny

File "home/user/Documents/versioned/buildroot/testing/target/usr/lib/python3.4/site-packages/twisted/python/reflect.py",
line 250, in _importAndCheckStack

File "home/user/Documents/versioned/buildroot/testing/target/usr/lib/python3.4/site-packages/twisted/python/compat.py",
line 380, in reraise

File "home/user/Documents/versioned/buildroot/testing/target/usr/lib/python3.4/site-packages/twisted/plugins/autobahn_endpoints.py",
line 46, in <module>

File "/home/user/Documents/versioned/buildroot/testing/build/python-autobahn-0.12.1/autobahn/twisted/__init__.py",
line 51, in <module>
from autobahn.twisted.wamp import ApplicationSession
File "/home/user/Documents/versioned/buildroot/testing/build/python-autobahn-0.12.1/autobahn/twisted/wamp.py",
line 35, in <module>
from autobahn.wamp import protocol
File "/home/user/Documents/versioned/buildroot/testing/build/python-autobahn-0.12.1/autobahn/wamp/protocol.py",
line 42, in <module>
from autobahn.wamp.keyring import EncryptedPayload
File "/home/user/Documents/versioned/buildroot/testing/build/python-autobahn-0.12.1/autobahn/wamp/keyring.py",
line 61, in <module>
from nacl.public import PrivateKey, PublicKey, Box
File "home/user/Documents/versioned/buildroot/testing/target/usr/lib/python3.4/site-packages/nacl/public.py",
line 18, in <module>

File "home/user/Documents/versioned/buildroot/testing/target/usr/lib/python3.4/site-packages/nacl/bindings/__init__.py",
line 17, in <module>

File "home/user/Documents/versioned/buildroot/testing/target/usr/lib/python3.4/site-packages/nacl/bindings/crypto_box.py",
line 17, in <module>

builtins.ImportError:
/home/user/Documents/versioned/buildroot/testing/target/usr/lib/python3.4/site-packages/nacl/_sodium.cpython-34m.so:
wrong ELF class: ELFCLASS32

Yegor

Thomas Petazzoni

2016-02-20 17:31 UTC #5 lore raw

PATCH v5 19/20 python-autobahn: new package

Hello,

Quoted 2 lines · Yegor Yefremov

Try to compile autobahn with PYC only and twisted and python-pynacl-1.0.1 enabled. I get:

OK. This should have been mentioned in the commit log, and if python-pynacl and twisted are optional dependencies of python-autobahn, which affect its build process, then they should be handled as such.

Can you submit follow-up patches ?

Thanks,

Thomas

Signature · 3 lines

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

Mauro Condarelli

author

2016-02-11 10:58 UTC #6 lore raw

PATCH v5 20/20 python-crossbar: new package

In reply to Mauro Condarelli

Signed-off-by Mauro Condarelli <[email protected]>

---
Changes v4 -> v5
- added dependency from Python sources.
- select python-shutilwhich only with Python2
- fixed typo BR2_PACKAGE_PYTHON_SHURILWHICH -> BR2_PACKAGE_PYTHON_SHUTILWHICH

Signed-off-by Mauro Condarelli <[email protected]>

---
 package/Config.in                            |  1 +
 package/python-crossbar/Config.in            | 38 ++++++++++++++++++++++++++++
 package/python-crossbar/python-crossbar.hash |  3 +++
 package/python-crossbar/python-crossbar.mk   | 14 ++++++++++
 4 files changed, 56 insertions(+)
 create mode 100644 package/python-crossbar/Config.in
 create mode 100644 package/python-crossbar/python-crossbar.hash
 create mode 100644 package/python-crossbar/python-crossbar.mk

package/Config.in +1 -0

index 4b92a2a..5037f12 100644
@@ -635,6 +635,7 @@ menu "External python modules"
 	source "package/python-configshell-fb/Config.in"
 	source "package/python-crc16/Config.in"
 	source "package/python-cryptography/Config.in"
+	source "package/python-crossbar/Config.in"
 	source "package/python-cssselect/Config.in"
 	source "package/python-daemon/Config.in"
 	source "package/python-dialog/Config.in"

package/python-crossbar/Config.in new file +38 -0

new file mode 100644
index 0000000..0cd3ee1
@@ -0,0 +1,38 @@
+config BR2_PACKAGE_PYTHON_CROSSBAR
+	bool "python-crossbar"
+	depends on !BR2_PACKAGE_PYTHON_PYC_ONLY
+	select BR2_PACKAGE_PYTHON_AUTOBAHN # runtime
+	select BR2_PACKAGE_PYTHON_CBOR # runtime
+	select BR2_PACKAGE_PYTHON_CLICK # runtime
+	select BR2_PACKAGE_PYTHON_CRYPTOGRAPHY # runtime
+	select BR2_PACKAGE_PYTHON_JINJA2 # runtime
+	select BR2_PACKAGE_PYTHON_MISTUNE # runtime
+	select BR2_PACKAGE_PYTHON_MSGPACK # runtime
+	select BR2_PACKAGE_PYTHON_NETADDR # runtime
+	select BR2_PACKAGE_PYTHON_PSUTIL # runtime
+	select BR2_PACKAGE_PYTHON_PYASN # runtime
+	select BR2_PACKAGE_PYTHON_PYASN_MODULES # runtime
+	select BR2_PACKAGE_PYTHON_PYGMENTS # runtime
+	select BR2_PACKAGE_PYTHON_PYINOTIFY # runtime
+	select BR2_PACKAGE_PYTHON_PYNACL # runtime
+	select BR2_PACKAGE_PYTHON_PYOPENSSL # runtime
+	select BR2_PACKAGE_PYTHON_PYTRIE # runtime
+	select BR2_PACKAGE_PYTHON_PYYAML # runtime
+	select BR2_PACKAGE_PYTHON_SETUPTOOLS # runtime
+	select BR2_PACKAGE_PYTHON_SERVICE_IDENTITY # runtime
+	select BR2_PACKAGE_PYTHON_SETPROCTITLE # runtime
+	select BR2_PACKAGE_PYTHON_SETUPTOOLS # runtime
+	select BR2_PACKAGE_PYTHON_SHUTILWHICH if BR2_PACKAGE_PYTHON # runtime
+	select BR2_PACKAGE_PYTHON_TREQ # runtime
+	select BR2_PACKAGE_PYTHON_TWISTED # runtime
+	select BR2_PACKAGE_PYTHON_UJSON # runtime
+	select BR2_PACKAGE_PYTHON_WSACCEL # runtime
+	help
+	  Crossbar.io is an open-source WAMP application router 
+	  that allows to build advanced applications from loosely-coupled
+	  components that can talk in real-time with each other.
+
+	  https://pypi.python.org/pypi/crossbar
+
+comment "Currently Crossbar.io does NOT work without Python sources!"
+        depends on BR2_PACKAGE_PYTHON_PYC_ONLY

package/python-crossbar/python-crossbar.hash new file +3 -0

new file mode 100644
index 0000000..0690e56
@@ -0,0 +1,3 @@
+# md5 from https://pypi.python.org/pypi?:action=show_md5&digest=cbe9b62a1ec2eebbc0cd0cef887a4993, sha256 locally computed.
+md5	cbe9b62a1ec2eebbc0cd0cef887a4993  crossbar-0.12.1.tar.gz
+sha256	4919e683bcfa3c633da8207f24189fd196ef8412b647f0365cf74c0bff8b4ada  crossbar-0.12.1.tar.gz

package/python-crossbar/python-crossbar.mk new file +14 -0

new file mode 100644
index 0000000..a35d54d
@@ -0,0 +1,14 @@
+################################################################################
+#
+# python-crossbar
+#
+################################################################################
+
+PYTHON_CROSSBAR_VERSION = 0.12.1
+PYTHON_CROSSBAR_SOURCE = crossbar-$(PYTHON_CROSSBAR_VERSION).tar.gz
+PYTHON_CROSSBAR_SITE = http://pypi.python.org/packages/source/c/crossbar
+PYTHON_CROSSBAR_LICENSE = AGPL3
+PYTHON_CROSSBAR_LICENSE_FILES = LICENSE
+PYTHON_CROSSBAR_SETUP_TYPE = setuptools
+
+$(eval $(python-package))

Signature · 1 line

2.1.4

Yegor Yefremov

2016-02-11 14:00 UTC #7 lore raw

PATCH v5 20/20 python-crossbar: new package

Quoted 9 lines · Mauro Condarelli

Signed-off-by Mauro Condarelli <[email protected]>
---
Changes v4 -> v5
- added dependency from Python sources.
- select python-shutilwhich only with Python2
- fixed typo BR2_PACKAGE_PYTHON_SHURILWHICH -> BR2_PACKAGE_PYTHON_SHUTILWHICH
Signed-off-by Mauro Condarelli <[email protected]>

Reviewed-by Yegor Yefremov <[email protected]>

Quoted 102 lines

---
 package/Config.in                            |  1 +
 package/python-crossbar/Config.in            | 38 ++++++++++++++++++++++++++++
 package/python-crossbar/python-crossbar.hash |  3 +++
 package/python-crossbar/python-crossbar.mk   | 14 ++++++++++
 4 files changed, 56 insertions(+)
 create mode 100644 package/python-crossbar/Config.in
 create mode 100644 package/python-crossbar/python-crossbar.hash
 create mode 100644 package/python-crossbar/python-crossbar.mk

package/Config.in +1 -0

index 4b92a2a..5037f12 100644
@@ -635,6 +635,7 @@ menu "External python modules"
        source "package/python-configshell-fb/Config.in"
        source "package/python-crc16/Config.in"
        source "package/python-cryptography/Config.in"
+       source "package/python-crossbar/Config.in"
        source "package/python-cssselect/Config.in"
        source "package/python-daemon/Config.in"
        source "package/python-dialog/Config.in"

package/python-crossbar/Config.in new file +38 -0

new file mode 100644
index 0000000..0cd3ee1
@@ -0,0 +1,38 @@
+config BR2_PACKAGE_PYTHON_CROSSBAR
+       bool "python-crossbar"
+       depends on !BR2_PACKAGE_PYTHON_PYC_ONLY
+       select BR2_PACKAGE_PYTHON_AUTOBAHN # runtime
+       select BR2_PACKAGE_PYTHON_CBOR # runtime
+       select BR2_PACKAGE_PYTHON_CLICK # runtime
+       select BR2_PACKAGE_PYTHON_CRYPTOGRAPHY # runtime
+       select BR2_PACKAGE_PYTHON_JINJA2 # runtime
+       select BR2_PACKAGE_PYTHON_MISTUNE # runtime
+       select BR2_PACKAGE_PYTHON_MSGPACK # runtime
+       select BR2_PACKAGE_PYTHON_NETADDR # runtime
+       select BR2_PACKAGE_PYTHON_PSUTIL # runtime
+       select BR2_PACKAGE_PYTHON_PYASN # runtime
+       select BR2_PACKAGE_PYTHON_PYASN_MODULES # runtime
+       select BR2_PACKAGE_PYTHON_PYGMENTS # runtime
+       select BR2_PACKAGE_PYTHON_PYINOTIFY # runtime
+       select BR2_PACKAGE_PYTHON_PYNACL # runtime
+       select BR2_PACKAGE_PYTHON_PYOPENSSL # runtime
+       select BR2_PACKAGE_PYTHON_PYTRIE # runtime
+       select BR2_PACKAGE_PYTHON_PYYAML # runtime
+       select BR2_PACKAGE_PYTHON_SETUPTOOLS # runtime
+       select BR2_PACKAGE_PYTHON_SERVICE_IDENTITY # runtime
+       select BR2_PACKAGE_PYTHON_SETPROCTITLE # runtime
+       select BR2_PACKAGE_PYTHON_SETUPTOOLS # runtime
+       select BR2_PACKAGE_PYTHON_SHUTILWHICH if BR2_PACKAGE_PYTHON # runtime
+       select BR2_PACKAGE_PYTHON_TREQ # runtime
+       select BR2_PACKAGE_PYTHON_TWISTED # runtime
+       select BR2_PACKAGE_PYTHON_UJSON # runtime
+       select BR2_PACKAGE_PYTHON_WSACCEL # runtime
+       help
+         Crossbar.io is an open-source WAMP application router
+         that allows to build advanced applications from loosely-coupled
+         components that can talk in real-time with each other.
+
+         https://pypi.python.org/pypi/crossbar
+
+comment "Currently Crossbar.io does NOT work without Python sources!"
+        depends on BR2_PACKAGE_PYTHON_PYC_ONLY

package/python-crossbar/python-crossbar.hash new file +3 -0

new file mode 100644
index 0000000..0690e56
@@ -0,0 +1,3 @@
+# md5 from https://pypi.python.org/pypi?:action=show_md5&digest=cbe9b62a1ec2eebbc0cd0cef887a4993, sha256 locally computed.
+md5    cbe9b62a1ec2eebbc0cd0cef887a4993  crossbar-0.12.1.tar.gz
+sha256 4919e683bcfa3c633da8207f24189fd196ef8412b647f0365cf74c0bff8b4ada  crossbar-0.12.1.tar.gz

package/python-crossbar/python-crossbar.mk new file +14 -1

new file mode 100644
index 0000000..a35d54d
@@ -0,0 +1,14 @@
+################################################################################
+#
+# python-crossbar
+#
+################################################################################
+
+PYTHON_CROSSBAR_VERSION = 0.12.1
+PYTHON_CROSSBAR_SOURCE = crossbar-$(PYTHON_CROSSBAR_VERSION).tar.gz
+PYTHON_CROSSBAR_SITE = http://pypi.python.org/packages/source/c/crossbar
+PYTHON_CROSSBAR_LICENSE = AGPL3
+PYTHON_CROSSBAR_LICENSE_FILES = LICENSE
+PYTHON_CROSSBAR_SETUP_TYPE = setuptools
+
+$(eval $(python-package))
--

2.1.4

_______________________________________________
buildroot mailing list
buildroot at busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot

Thomas Petazzoni

2016-02-20 14:38 UTC #8 lore raw

PATCH v5 20/20 python-crossbar: new package

In reply to Mauro Condarelli

Mauro,

package/python-crossbar/Config.in line 3

Quoted 9 lines · Mauro Condarelli

package/python-crossbar/Config.in new file +3 -0

new file mode 100644
index 0000000..0cd3ee1
@@ -0,0 +1,38 @@
+config BR2_PACKAGE_PYTHON_CROSSBAR
+	bool "python-crossbar"
+	depends on !BR2_PACKAGE_PYTHON_PYC_ONLY

I tested with BR2_PACKAGE_PYTHON_PYC_ONLY=y and it just built and worked fine for me. I was able to run "crossbar version" :

# crossbar version
Automatically choosing optimal Twisted reactor
Running on Linux and optimal reactor (epoll) was installed.
     __  __  __  __  __  __      __     __
    /  `|__)/  \/__`/__`|__) /\ |__)  |/  \
    \__,|  \\__/.__/.__/|__)/~~\|  \. |\__/

 Crossbar.io        : 0.12.1
   Autobahn         : 0.12.1 (with JSON, MessagePack, CBOR)
   Twisted          : 15.5.0-EPollReactor
   LMDB             : 0.89/lmdb-0.9.18
   Python           : 3.4.3/CPython
 OS                 : Linux-4.2.0-armv7l-with-libc1
 Machine            : armv7l

Quoted 16 lines

package/python-crossbar/Config.in new file +16 -0

+	select BR2_PACKAGE_PYTHON_AUTOBAHN # runtime
+	select BR2_PACKAGE_PYTHON_CBOR # runtime
+	select BR2_PACKAGE_PYTHON_CLICK # runtime
+	select BR2_PACKAGE_PYTHON_CRYPTOGRAPHY # runtime
+	select BR2_PACKAGE_PYTHON_JINJA2 # runtime
+	select BR2_PACKAGE_PYTHON_MISTUNE # runtime
+	select BR2_PACKAGE_PYTHON_MSGPACK # runtime
+	select BR2_PACKAGE_PYTHON_NETADDR # runtime
+	select BR2_PACKAGE_PYTHON_PSUTIL # runtime
+	select BR2_PACKAGE_PYTHON_PYASN # runtime
+	select BR2_PACKAGE_PYTHON_PYASN_MODULES # runtime
+	select BR2_PACKAGE_PYTHON_PYGMENTS # runtime
+	select BR2_PACKAGE_PYTHON_PYINOTIFY # runtime
+	select BR2_PACKAGE_PYTHON_PYNACL # runtime
+	select BR2_PACKAGE_PYTHON_PYOPENSSL # runtime
+	select BR2_PACKAGE_PYTHON_PYTRIE # runtime

package/python-crossbar/Config.in line 29

Quoted 10 lines

package/python-crossbar/Config.in new file +10 -0

+	select BR2_PACKAGE_PYTHON_PYYAML # runtime
+	select BR2_PACKAGE_PYTHON_SETUPTOOLS # runtime
+	select BR2_PACKAGE_PYTHON_SERVICE_IDENTITY # runtime
+	select BR2_PACKAGE_PYTHON_SETPROCTITLE # runtime
+	select BR2_PACKAGE_PYTHON_SETUPTOOLS # runtime
+	select BR2_PACKAGE_PYTHON_SHUTILWHICH if BR2_PACKAGE_PYTHON # runtime
+	select BR2_PACKAGE_PYTHON_TREQ # runtime
+	select BR2_PACKAGE_PYTHON_TWISTED # runtime
+	select BR2_PACKAGE_PYTHON_UJSON # runtime
+	select BR2_PACKAGE_PYTHON_WSACCEL # runtime

Mentioning "# runtime" each time was a bit annoying, so I've changed this to one single comment at the beginning saying that all dependencies are runtime dependencies.

Note however that:

* Dependency on BR2_PACKAGE_PYTHON_LMDB was missing. Without this, "crossbar version" was failing with "pkg_resources.DistributionNotFound: The 'lmdb>=0.88' distribution was not found and is required by crossbar"

* Dependency on BR2_PACKAGE_PYTHON_PYCPARSER was missing. Without this, "crossbar version" was failing with:
"pkg_resources.DistributionNotFound: The 'pycparser' distribution was not found and is required by cffi".

Also, there is still one problem with your package: when running "crossbar version", it complains that:

pkg_resources.DistributionNotFound: The 'shutilwhich>=1.1.0' distribution was not found and is required by crossbar

However, shutilwhich is only available for Python 2.x, and I'm using Python 3.x. When I force enable python-shutilwhich on Python 3.x, then I can run "crossbar version" (this is how I did my test above).

But since shutilwhich is not supposed to be enabled with Python 3.x, this was really a hack.

Can you look into this and provide a solution?

Once again, when you submit new packages, please make sure that they build and run by building a configuration that only has *this* package enabled.

package/python-crossbar/Config.in line 30

Quoted 1 line

package/python-crossbar/Config.in new file +1 -0
+PYTHON_CROSSBAR_LICENSE = AGPL3

We use AGPLv3 and not AGPL3.

I've committed to the next branch with those issues fixed, except the shutilwhich one.

Thanks,

Thomas

Signature · 3 lines

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

Yegor Yefremov

2016-02-20 15:46 UTC #9 lore raw

PATCH v5 20/20 python-crossbar: new package

Thomas,

Quoted 93 lines · Thomas Petazzoni

Mauro,
package/python-crossbar/Config.in line 3
Quoted 9 lines · Mauro Condarelli
package/python-crossbar/Config.in new file +3 -0
new file mode 100644
index 0000000..0cd3ee1
@@ -0,0 +1,38 @@
+config BR2_PACKAGE_PYTHON_CROSSBAR
+     bool "python-crossbar"
+     depends on !BR2_PACKAGE_PYTHON_PYC_ONLY
I tested with BR2_PACKAGE_PYTHON_PYC_ONLY=y and it just built and worked fine for me. I was able to run "crossbar version" :
# crossbar version
Automatically choosing optimal Twisted reactor
Running on Linux and optimal reactor (epoll) was installed.
     __  __  __  __  __  __      __     __
    /  `|__)/  \/__`/__`|__) /\ |__)  |/  \
    \__,|  \\__/.__/.__/|__)/~~\|  \. |\__/
 Crossbar.io        : 0.12.1
   Autobahn         : 0.12.1 (with JSON, MessagePack, CBOR)
   Twisted          : 15.5.0-EPollReactor
   LMDB             : 0.89/lmdb-0.9.18
   Python           : 3.4.3/CPython
 OS                 : Linux-4.2.0-armv7l-with-libc1
 Machine            : armv7l
Quoted 16 lines
package/python-crossbar/Config.in new file +16 -0
+     select BR2_PACKAGE_PYTHON_AUTOBAHN # runtime
+     select BR2_PACKAGE_PYTHON_CBOR # runtime
+     select BR2_PACKAGE_PYTHON_CLICK # runtime
+     select BR2_PACKAGE_PYTHON_CRYPTOGRAPHY # runtime
+     select BR2_PACKAGE_PYTHON_JINJA2 # runtime
+     select BR2_PACKAGE_PYTHON_MISTUNE # runtime
+     select BR2_PACKAGE_PYTHON_MSGPACK # runtime
+     select BR2_PACKAGE_PYTHON_NETADDR # runtime
+     select BR2_PACKAGE_PYTHON_PSUTIL # runtime
+     select BR2_PACKAGE_PYTHON_PYASN # runtime
+     select BR2_PACKAGE_PYTHON_PYASN_MODULES # runtime
+     select BR2_PACKAGE_PYTHON_PYGMENTS # runtime
+     select BR2_PACKAGE_PYTHON_PYINOTIFY # runtime
+     select BR2_PACKAGE_PYTHON_PYNACL # runtime
+     select BR2_PACKAGE_PYTHON_PYOPENSSL # runtime
+     select BR2_PACKAGE_PYTHON_PYTRIE # runtime
package/python-crossbar/Config.in line 29
Quoted 10 lines
package/python-crossbar/Config.in new file +10 -0
+     select BR2_PACKAGE_PYTHON_PYYAML # runtime
+     select BR2_PACKAGE_PYTHON_SETUPTOOLS # runtime
+     select BR2_PACKAGE_PYTHON_SERVICE_IDENTITY # runtime
+     select BR2_PACKAGE_PYTHON_SETPROCTITLE # runtime
+     select BR2_PACKAGE_PYTHON_SETUPTOOLS # runtime
+     select BR2_PACKAGE_PYTHON_SHUTILWHICH if BR2_PACKAGE_PYTHON # runtime
+     select BR2_PACKAGE_PYTHON_TREQ # runtime
+     select BR2_PACKAGE_PYTHON_TWISTED # runtime
+     select BR2_PACKAGE_PYTHON_UJSON # runtime
+     select BR2_PACKAGE_PYTHON_WSACCEL # runtime
Mentioning "# runtime" each time was a bit annoying, so I've changed this to one single comment at the beginning saying that all dependencies are runtime dependencies.
Note however that:
* Dependency on BR2_PACKAGE_PYTHON_LMDB was missing. Without this, "crossbar version" was failing with "pkg_resources.DistributionNotFound: The 'lmdb>=0.88' distribution was not found and is required by crossbar"
* Dependency on BR2_PACKAGE_PYTHON_PYCPARSER was missing. Without this, "crossbar version" was failing with:
"pkg_resources.DistributionNotFound: The 'pycparser' distribution was not found and is required by cffi".
Also, there is still one problem with your package: when running "crossbar version", it complains that:
pkg_resources.DistributionNotFound: The 'shutilwhich>=1.1.0' distribution was not found and is required by crossbar
However, shutilwhich is only available for Python 2.x, and I'm using Python 3.x. When I force enable python-shutilwhich on Python 3.x, then I can run "crossbar version" (this is how I did my test above).
But since shutilwhich is not supposed to be enabled with Python 3.x, this was really a hack.
Can you look into this and provide a solution?
Once again, when you submit new packages, please make sure that they build and run by building a configuration that only has *this* package enabled.

Quoted 6 lines

package/python-crossbar/Config.in line 30
Quoted 1 line
package/python-crossbar/Config.in new file +1 -0
+PYTHON_CROSSBAR_LICENSE = AGPL3
We use AGPLv3 and not AGPL3.
I've committed to the next branch with those issues fixed, except the shutilwhich one.

Have you tried this example with PYC only version?

http://crossbar.io/docs/Getting-started-with-Python/

crossbar init --template hello:python --appdir $HOME/hello crossbar start --cbdir /home/oberstet/hello/.crossbar

See my attempt to find a solution for this issue:
https://github.com/crossbario/crossbar/pull/638

Yegor

Thomas Petazzoni

2016-02-20 15:54 UTC #10 lore raw

PATCH v5 20/20 python-crossbar: new package

Hello,

Quoted 9 lines · Yegor Yefremov

Have you tried this example with PYC only version?
http://crossbar.io/docs/Getting-started-with-Python/
crossbar init --template hello:python --appdir $HOME/hello crossbar start --cbdir /home/oberstet/hello/.crossbar
See my attempt to find a solution for this issue:
https://github.com/crossbario/crossbar/pull/638

No, I only tried "crossbar version". Since the commit log did not give any details about the failing scenario, I couldn't know how to reproduce the issue.

I added a comment in your pull request, to support your claim. However, I believe it is indeed a bit horrible to add all this code. Why is the code looking for process.py in the first place rather than using the normal Python import logic, which properly works with .pyc files only?

Best regards,

Thomas

Signature · 3 lines

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

Yegor Yefremov

2016-02-11 13:59 UTC #11 lore raw

PATCH v5 18/20 python-service-identity: new package

In reply to Mauro Condarelli

Quoted 7 lines · Mauro Condarelli

Signed-off-by Mauro Condarelli <[email protected]>
---
Changes v4 -> v5
- removed spurous (runtime) depemndencies in .mk
Signed-off-by Mauro Condarelli <[email protected]>

Reviewed-by Yegor Yefremov <[email protected]>

Quoted 74 lines

---
 package/Config.in                                          |  1 +
 package/python-service-identity/Config.in                  | 10 ++++++++++
 .../python-service-identity/python-service-identity.hash   |  3 +++
 package/python-service-identity/python-service-identity.mk | 14 ++++++++++++++
 4 files changed, 28 insertions(+)
 create mode 100644 package/python-service-identity/Config.in
 create mode 100644 package/python-service-identity/python-service-identity.hash
 create mode 100644 package/python-service-identity/python-service-identity.mk

package/Config.in +1 -0

index 5907ad6..d1d0ec7 100644
@@ -713,6 +713,7 @@ menu "External python modules"
        source "package/python-rtslib-fb/Config.in"
        source "package/python-serial/Config.in"
        source "package/python-setproctitle/Config.in"
+       source "package/python-service-identity/Config.in"
        source "package/python-setuptools/Config.in"
        source "package/python-shutilwhich/Config.in"
        source "package/python-simplejson/Config.in"

package/python-service-identity/Config.in new file +10 -0

new file mode 100644
index 0000000..e2200f6
@@ -0,0 +1,10 @@
+config BR2_PACKAGE_PYTHON_SERVICE_IDENTITY
+       bool "python-service-identity"
+       select BR2_PACKAGE_PYTHON_CHARACTERISTIC # runtime
+       select BR2_PACKAGE_PYTHON_PYASN # runtime
+       select BR2_PACKAGE_PYTHON_PYASN_MODULES # runtime
+       select BR2_PACKAGE_PYTHON_PYOPENSSL # runtime
+       help
+         Service identity verification for pyOpenSSL.
+
+         https://pypi.python.org/pypi/service_identity

package/python-service-identity/python-service-identity.hash new file +3 -0

new file mode 100644
index 0000000..caf29dc
@@ -0,0 +1,3 @@
+# md5 from https://pypi.python.org/pypi?:action=show_md5&digest=cea0b0156d73b025ecef660fb51f0d9a, sha256 locally computed.
+md5    cea0b0156d73b025ecef660fb51f0d9a  service_identity-14.0.0.tar.gz
+sha256 3105a319a7c558490666694f599be0c377ad54824eefb404cde4ce49e74a4f5a  service_identity-14.0.0.tar.gz

package/python-service-identity/python-service-identity.mk new file +14 -1

new file mode 100644
index 0000000..9eca3c1
@@ -0,0 +1,14 @@
+################################################################################
+#
+# python-service-identity
+#
+################################################################################
+
+PYTHON_SERVICE_IDENTITY_VERSION = 14.0.0
+PYTHON_SERVICE_IDENTITY_SOURCE = service_identity-$(PYTHON_SERVICE_IDENTITY_VERSION).tar.gz
+PYTHON_SERVICE_IDENTITY_SITE = http://pypi.python.org/packages/source/s/service_identity
+PYTHON_SERVICE_IDENTITY_LICENSE = MIT
+PYTHON_SERVICE_IDENTITY_LICENSE_FILES = LICENSE
+PYTHON_SERVICE_IDENTITY_SETUP_TYPE = setuptools
+
+$(eval $(python-package))
--

2.1.4

_______________________________________________
buildroot mailing list
buildroot at busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot

Thomas Petazzoni

2016-02-19 21:40 UTC #12 lore raw

PATCH v5 18/20 python-service-identity: new package

In reply to Mauro Condarelli

Mauro,

Quoted 1 line · Mauro Condarelli

Signed-off-by Mauro Condarelli <[email protected]>

Just a quick comment: it is really horrible for people reviewing/applying your patches that you sent this patch separately from the rest of the series. Now I have to manually figure out that this patch labeled 18/20 in your v5 is in fact needed for patch 09/17 in your v4.

Please resend full series, so that the dependencies between patches remain explicit. Or if you don't do that, resend the new iteration of a patch as a reply to the previous patch and with the same sequence number.

Thanks,

Thomas

Signature · 3 lines

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

Mauro Condarelli

author

2016-02-20 12:32 UTC #13 lore raw

PATCH v5 18/20 python-service-identity: new package

Thomas.

Il 19/02/2016 22:40, Thomas Petazzoni ha scritto:

Quoted 9 lines

Mauro,
Quoted 1 line · Mauro Condarelli
Signed-off-by Mauro Condarelli <[email protected]>
Just a quick comment: it is really horrible for people reviewing/applying your patches that you sent this patch separately from the rest of the series. Now I have to manually figure out that this patch labeled 18/20 in your v5 is in fact needed for patch 09/17 in your v4.

Sorry about that.
I was (and perhaps am) still trying to fully understand workflow.

Quoted 4 lines

Please resend full series, so that the dependencies between patches remain explicit. Or if you don't do that, resend the new iteration of a patch as a reply to the previous patch and with the same sequence number.

Ok.
Next time I'll resend the full series and then I'll mark the previous one as "superseded" in Patchwork. Should I do something to link together the (versioned) series? They seem to appear as independent in Patchwork.

Quoted 3 lines

Thanks,
Thomas

Regards and Apologies
Mauro

Thomas Petazzoni

2016-02-20 12:57 UTC #14 lore raw

PATCH v5 18/20 python-service-identity: new package

Hello,

Quoted 2 lines · Mauro Condarelli

Next time I'll resend the full series and then I'll mark the previous one as "superseded" in Patchwork. Should I do something to link together the (versioned) series? They seem to appear as independent in Patchwork.

No, just mark the previous iteration as Superseded in patchwork.

Thanks!

Thomas

Signature · 3 lines

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

Thomas Petazzoni

Applied

2016-02-19 22:18 UTC #15 lore raw

PATCH v5 18/20 python-service-identity: new package

In reply to Mauro Condarelli

Dear Mauro Condarelli,

Quoted 5 lines · Mauro Condarelli

Signed-off-by Mauro Condarelli <[email protected]>
---
Changes v4 -> v5
- removed spurous (runtime) depemndencies in .mk

Applied to next, thanks.

Thomas

Signature · 3 lines

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