[MIR] ply, python-jsonpath-rw

Bug #1262214 reported by James Page on 2013-12-18
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ply (Ubuntu)
High
Unassigned
python-jsonpath-rw (Ubuntu)
High
Unassigned

Bug Description

python-jsonpath-rw

Availability: In universe
Rationale: New dependency for ceilometer
Security: No CVE's found.
Quality assurance: runs tests suite, packaged for openstack by Debian OpenStack maintainer.
Dependencies: aside from python-ply, all in main
Standards compliance: OK
Maintenance: OK (server team pickup in Ubuntu)

ply

Availability: In universe
Rationale: New dependency for ceilometer (via python-jsonpath-rw)
Security: No CVE's found.
Quality assurance: Currently does not run test suite; OK with python2, some failures with python3.
Dependencies: All in main
Standards compliance: OK (older cdbs packaging but tidy)
Maintenance: OK (server team pickup in Ubuntu)

James Page (james-page) wrote :
summary: - [MIR] python-ply, python-jsonpath-rw
+ [MIR] ply, python-jsonpath-rw
James Page (james-page) on 2013-12-19
description: updated
description: updated
Changed in ply (Ubuntu):
importance: Undecided → High
Changed in python-jsonpath-rw (Ubuntu):
importance: Undecided → High
Michael Terry (mterry) wrote :

python-jsonpath-rw looks good. Tests, we're in sync with Debian, good packaging, builds fine.

However, it needs a team bug subscriber.

Changed in python-jsonpath-rw (Ubuntu):
status: New → Incomplete
Michael Terry (mterry) wrote :

ply looks mostly good. However, it should run it's tests. Something like:

cd test && python ./testlex.py && python ./testyacc.py && ./cleanup.sh
cd test && python3 ./testlex.py && python3 ./testyacc.py && ./cleanup.sh

Changed in ply (Ubuntu):
status: New → Incomplete
Michael Terry (mterry) wrote :

(ply also needs a team bug subscriber)

James Page (james-page) wrote :

ubuntu-server added as subscribed for python-jsonpath-rw bugmail

Changed in python-jsonpath-rw (Ubuntu):
status: Incomplete → New
James Page (james-page) wrote :

ubuntu-server added as subscribed for ply bugmail

James Page (james-page) wrote :

Looking at enabling tests for ply; seeing some transient test failure for one set of tests on python3 (looking at upstream commits to fix).

James Page (james-page) wrote :

Test suite enabled (with some cherry picking) for ply for python2 and python3.x

Changed in ply (Ubuntu):
status: Incomplete → New
Michael Terry (mterry) on 2014-01-06
Changed in python-jsonpath-rw (Ubuntu):
status: New → Fix Committed
Michael Terry (mterry) wrote :

The way you've implemented test running in ply's debian/rules won't work. When you group multiple commands with a line-continuing backslash, only the last command run will be used to determine if the whole thing worked. You need to insert a "set -e" at the front like so:

set -e; for pyver in $(PYVERSIONS) $(PY3VERSIONS); do \

Changed in ply (Ubuntu):
status: New → Incomplete

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

On 07/01/14 16:09, Michael Terry wrote:
> The way you've implemented test running in ply's debian/rules
> won't work. When you group multiple commands with a
> line-continuing backslash, only the last command run will be used
> to determine if the whole thing worked. You need to insert a "set
> -e" at the front like so:

Sorry - I missed that - I've uploaded a fixed version which should
correctly detect failures.

Thanks!

- --
James Page
Ubuntu and Debian Developer
<email address hidden>
<email address hidden>
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.15 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQIcBAEBCAAGBQJSzUZlAAoJEL/srsug59jD3FcQAJSN57PXTeLAsn4TCaTFvKev
UNzo8i/DSnsrhSIhWRgu8DN2zKGosgRHaPyOFtxa0m5Ws88tvCD7dx0wpLRRMCkZ
VejzOwGip4+1P4uqjEPh7KNNnjW+GH7HGLxDEc+vboRKovd9glc/C/+S05ahfl28
cTscM9kBAME2CZ7EvSHdo93UbIdiQCr+g05+PP+i86DZMoy0do9a//zVIuBndPWI
iTlDbDJLf5V410RhsH5zuVsKLhqT1adqlHkqFO8aRX0QLTHRGbObcBSqJCiz7k9Y
wOwocnCNg+/2thiyUuF7FiDs4cp98ZuWVl3NOHB9xLeBT5Liof9+/947Fnq9VDQl
6E4pyAhfUKmIhpgQLFBoKAkfxhpBPryzfS6lzMW+/98iATjrr+ebngZZTBjsc9ty
VNbYfi6t1cgcMC88xHHV/a9Bhk/AHLn4ikdPydEnFtzzxMY0kIphwoPlbECl/i+5
WMqb3v31vKM7mI3/1DLpk+kwgr/5Yc6LGuxtSnw4QrTZUg/etlPvuVgakLyxiOGY
4CAjBazg8ZGo414dgTnpFGg6A/TNy3fdKbRW3WCfXJbMYk31bE2UHYBRi3BwT0Pm
X7MttfyEmdfG6kQjSQz9zVTrYUImE96HrAUM6CKrI2KDZGScg4Hc+GGxPDXXhxmv
gq7RBrQ5FVyOw+l1frul
=QpIo
-----END PGP SIGNATURE-----

Changed in ply (Ubuntu):
status: Incomplete → New
Michael Terry (mterry) on 2014-01-08
Changed in ply (Ubuntu):
status: New → Fix Committed
Matthias Klose (doko) wrote :

Override component to main
ply 3.4-3ubuntu2 in trusty: universe/python -> main
python-ply 3.4-3ubuntu2 in trusty amd64: universe/python/optional/100% -> main
python-ply 3.4-3ubuntu2 in trusty arm64: universe/python/optional/100% -> main
python-ply 3.4-3ubuntu2 in trusty armhf: universe/python/optional/100% -> main
python-ply 3.4-3ubuntu2 in trusty i386: universe/python/optional/100% -> main
python-ply 3.4-3ubuntu2 in trusty powerpc: universe/python/optional/100% -> main
python-ply 3.4-3ubuntu2 in trusty ppc64el: universe/python/optional/100% -> main
python-ply-doc 3.4-3ubuntu2 in trusty amd64: universe/doc/optional/100% -> main
python-ply-doc 3.4-3ubuntu2 in trusty arm64: universe/doc/optional/100% -> main
python-ply-doc 3.4-3ubuntu2 in trusty armhf: universe/doc/optional/100% -> main
python-ply-doc 3.4-3ubuntu2 in trusty i386: universe/doc/optional/100% -> main
python-ply-doc 3.4-3ubuntu2 in trusty powerpc: universe/doc/optional/100% -> main
python-ply-doc 3.4-3ubuntu2 in trusty ppc64el: universe/doc/optional/100% -> main
python3-ply 3.4-3ubuntu2 in trusty amd64: universe/python/optional/100% -> main
python3-ply 3.4-3ubuntu2 in trusty arm64: universe/python/optional/100% -> main
python3-ply 3.4-3ubuntu2 in trusty armhf: universe/python/optional/100% -> main
python3-ply 3.4-3ubuntu2 in trusty i386: universe/python/optional/100% -> main
python3-ply 3.4-3ubuntu2 in trusty powerpc: universe/python/optional/100% -> main
python3-ply 3.4-3ubuntu2 in trusty ppc64el: universe/python/optional/100% -> main
19 publications overridden.

Changed in python-jsonpath-rw (Ubuntu):
status: Fix Committed → Fix Released
Matthias Klose (doko) wrote :

Override component to main
python-jsonpath-rw 1.2.0-1 in trusty: universe/misc -> main
python-jsonpath-rw 1.2.0-1 in trusty amd64: universe/python/optional/100% -> main
python-jsonpath-rw 1.2.0-1 in trusty arm64: universe/python/optional/100% -> main
python-jsonpath-rw 1.2.0-1 in trusty armhf: universe/python/optional/100% -> main
python-jsonpath-rw 1.2.0-1 in trusty i386: universe/python/optional/100% -> main
python-jsonpath-rw 1.2.0-1 in trusty powerpc: universe/python/optional/100% -> main
python-jsonpath-rw 1.2.0-1 in trusty ppc64el: universe/python/optional/100% -> main
python3-jsonpath-rw 1.2.0-1 in trusty amd64: universe/python/optional/100% -> main
python3-jsonpath-rw 1.2.0-1 in trusty arm64: universe/python/optional/100% -> main
python3-jsonpath-rw 1.2.0-1 in trusty armhf: universe/python/optional/100% -> main
python3-jsonpath-rw 1.2.0-1 in trusty i386: universe/python/optional/100% -> main
python3-jsonpath-rw 1.2.0-1 in trusty powerpc: universe/python/optional/100% -> main
python3-jsonpath-rw 1.2.0-1 in trusty ppc64el: universe/python/optional/100% -> main
13 publications overridden.

Changed in ply (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Related blueprints

Bug attachments