Merge lp:~jelmer/brz/byov-no-py3 into lp:brz

Proposed by Jelmer Vernooij
Status: Merged
Approved by: Jelmer Vernooij
Approved revision: 7505
Merge reported by: The Breezy Bot
Merged at revision: not available
Proposed branch: lp:~jelmer/brz/byov-no-py3
Merge into: lp:brz
Diff against target: 93 lines (+18/-20)
2 files modified
breezy/tests/test_source.py (+5/-1)
byov.conf (+13/-19)
To merge this branch: bzr merge lp:~jelmer/brz/byov-no-py3
Reviewer Review Type Date Requested Status
Jelmer Vernooij Approve
Review via email: mp+384841@code.launchpad.net

Commit message

Drop python 2 dependencies in byoci.

Description of the change

Drop python 2 dependencies.

To post a comment you must log in.
Revision history for this message
Jelmer Vernooij (jelmer) :
review: Approve
Revision history for this message
The Breezy Bot (the-breezy-bot) wrote :
lp:~jelmer/brz/byov-no-py3 updated
7504. By Jelmer Vernooij

Drop '3' suffix.

7505. By Jelmer Vernooij

Skip flake8 where broken.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'breezy/tests/test_source.py'
2--- breezy/tests/test_source.py 2020-02-18 01:57:45 +0000
3+++ breezy/tests/test_source.py 2020-05-29 17:47:46 +0000
4@@ -332,6 +332,7 @@
5 new_path.insert(
6 0, os.path.join(os.path.dirname(__file__), '..', '..', 'tools'))
7 self.overrideAttr(sys, 'path', new_path)
8+ import argparse
9 from flake8.main.application import Application
10 from flake8.formatting.base import BaseFormatter
11 app = Application()
12@@ -352,8 +353,11 @@
13 def handle(self, error):
14 self.errors.append(error)
15
16+ try:
17+ app.initialize([])
18+ except argparse.ArgumentError as e:
19+ self.skipTest('broken flake8: %r' % e)
20 app.formatter = Formatter()
21- app.initialize([])
22 app.run_checks()
23 app.report()
24 self.assertEqual(app.formatter.errors, [])
25
26=== modified file 'byov.conf'
27--- byov.conf 2019-10-13 22:53:02 +0000
28+++ byov.conf 2020-05-29 17:47:46 +0000
29@@ -3,20 +3,14 @@
30 vm.update = True
31 # External sources dependencies, packages are not recent enough
32 dulwich.clone = (git clone git://jelmer.uk/dulwich ../dulwich.git)
33-dulwich.install = (cd ../dulwich.git && ./setup.py install --user)
34-dulwich.install3 = (cd ../dulwich.git && python3 ./setup.py install --user)
35+dulwich.install = (cd ../dulwich.git && python3 ./setup.py install --user)
36 subunit.clone = (git clone https://github.com/testing-cabal/subunit.git ../subunit)
37 fastimport.clone = (git clone git://jelmer.uk/python-fastimport.git ../fastimport.git)
38-fastimport.install = (cd ../fastimport.git && ./setup.py install --user)
39-fastimport.install3 = (cd ../fastimport.git && python3 ./setup.py install --user)
40-sphinx_epytext.install = (pip install sphinx==1.8.5 sphinx_epytext)
41-sphinx_epytext.install3 = (pip3 install sphinx_epytext)
42-flake8.install = (pip install flake8)
43-flake8.install3 = (pip3 install flake8)
44-patiencediff.install = (pip install patiencediff)
45-patiencediff.install3 = (pip3 install patiencediff)
46-cython.install = (pip install cython)
47-cython.install3 = (pip3 install cython)
48+fastimport.install = (cd ../fastimport.git && python3 ./setup.py install --user)
49+sphinx_epytext.install = (pip3 install sphinx_epytext)
50+flake8.install = (pip3 install flake8)
51+patiencediff.install = (pip3 install patiencediff)
52+cython.install = (pip3 install cython)
53
54 [brz]
55 # FIXME: we're stuck on xenial
56@@ -29,12 +23,12 @@
57
58 # FIXME: Arguably this should be vm.build_deps=brz but it requires either an
59 # available package or at least a debian/ dir ? -- vila 2018-02-23
60-brz.build_deps = gcc, debhelper, python, python-all-dev, python3-all-dev, python-configobj, python3-configobj, python-docutils, python3-docutils, python-paramiko, python3-paramiko, python-subunit, python3-subunit, python-testtools, python3-testtools, subunit, python-pip, python3-pip, python-setuptools, python3-setuptools, python-flake8, python3-flake8, python-sphinx, python3-sphinx, python-launchpadlib, python3-launchpadlib
61-subunit.build_deps = python-testscenarios, python3-testscenarios, python-testtools, python3-testtools, cython, cython3, quilt
62+brz.build_deps = gcc, debhelper, python3, python3-all-dev, python3-configobj, python3-docutils, python3-paramiko, python3-subunit, python3-testtools, subunit, python3-pip, python3-setuptools, python3-flake8, python3-sphinx, python3-launchpadlib
63+subunit.build_deps = python3-testscenarios, python3-testtools, cython, cython3, quilt
64 vm.packages = {brz.build_deps}, {subunit.build_deps}, bzr, git, python-junitxml
65 [brz-xenial]
66 vm.release = xenial
67-byoci.setup.command = ({dulwich.clone} && {dulwich.install} && {dulwich.install3} && {fastimport.clone} && {fastimport.install} && {fastimport.install3} && {subunit.clone} && {flake8.install} && {flake8.install3} && {patiencediff.install} && {patiencediff.install3} && {sphinx_epytext.install} && {sphinx_epytext.install3} && {cython.install} && {cython.install3})
68+byoci.setup.command = ({dulwich.clone} && {dulwich.install} && {fastimport.clone} && {fastimport.install} && {subunit.clone} && {flake8.install} && {patiencediff.install} && {sphinx_epytext.install} && {cython.install})
69 # FIXME: bzr log -l2 should be by default -- vila 2018-03-09
70 byoci.tests.command = bash -o pipefail -c "bzr log -l2 && PYTHONPATH=../subunit/python:$PYTHONPATH PATH=../subunit/filters:$HOME/.local/bin:$PATH make check-ci | subunit2junitxml -o ../results.xml -f | subunit2pyunit"
71 [brz-bionic]
72@@ -43,8 +37,8 @@
73 vm.release = cosmic
74 [brz-disco]
75 vm.release = disco
76-vm.packages = {brz.build_deps}, {subunit.build_deps}, bzr, git, python-dulwich, python3-dulwich, python-junitxml
77-byoci.setup.command = ({subunit.clone} && {sphinx_epytext.install} && {sphinx_epytext.install3} )
78+vm.packages = {brz.build_deps}, {subunit.build_deps}, bzr, git, python3-dulwich, python-junitxml
79+byoci.setup.command = ({subunit.clone} && {sphinx_epytext.install} )
80 byoci.tests.command = bash -o pipefail -c "bzr log -l2 && (PYTHONPATH=../subunit/python:$PYTHONPATH PATH=../subunit/filters:$HOME/.local/bin:$PATH make check-ci | subunit2junitxml -o ../results.xml -f | subunit2pyunit)"
81 [brz-eoan]
82 vm.release = eoan
83@@ -52,8 +46,8 @@
84 [brz-buster]
85 vm.distribution = debian
86 vm.release = buster
87-vm.packages = {brz.build_deps}, {subunit.build_deps}, bzr, git, python-dulwich, python3-dulwich, python-junitxml
88-byoci.setup.command = ({subunit.clone} && {sphinx_epytext.install} && {sphinx_epytext.install3} )
89+vm.packages = {brz.build_deps}, {subunit.build_deps}, bzr, git, python3-dulwich, python-junitxml
90+byoci.setup.command = ({subunit.clone} && {sphinx_epytext.install} )
91 byoci.tests.command = bash -o pipefail -c "bzr log -l2 && (PYTHONPATH=../subunit/python:$PYTHONPATH PATH=../subunit/filters:$HOME/.local/bin:$PATH make check-ci | subunit2junitxml -o ../results.xml -f | subunit2pyunit)"
92 [brz-sid]
93 vm.distribution = debian

Subscribers

People subscribed via source and target branches