Merge ~cjwatson/ubuntu-archive-tools:flake8-W605 into ubuntu-archive-tools:main

Proposed by Colin Watson
Status: Needs review
Proposed branch: ~cjwatson/ubuntu-archive-tools:flake8-W605
Merge into: ubuntu-archive-tools:main
Diff against target: 99 lines (+17/-17)
4 files modified
kernel_workflow.py (+1/-1)
publish-image-set (+9/-9)
queuediff (+4/-4)
sru-review (+3/-3)
Reviewer Review Type Date Requested Status
Ubuntu Package Archive Administrators Pending
Review via email: mp+411081@code.launchpad.net

Commit message

Fix "W605 invalid escape sequence" flake8 warnings

To post a comment you must log in.

Unmerged commits

0dbb075... by Colin Watson

Fix "W605 invalid escape sequence" flake8 warnings

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/kernel_workflow.py b/kernel_workflow.py
index a211776..8f5a8f8 100644
--- a/kernel_workflow.py
+++ b/kernel_workflow.py
@@ -46,7 +46,7 @@ def process_sru_bug(lp, bugnum, task_callback, source_callback, context=None):
46 """46 """
47 package_re = re.compile(47 package_re = re.compile(
48 (r'^%subuntu/(?P<release>[0-9a-z.-]+)/'48 (r'^%subuntu/(?P<release>[0-9a-z.-]+)/'
49 + '\+source/(?P<package>[a-z0-9.-]+)$') % str(lp._root_uri))49 r'\+source/(?P<package>[a-z0-9.-]+)$') % str(lp._root_uri))
50 workflow_re = re.compile(50 workflow_re = re.compile(
51 r'^%skernel-sru-workflow/(?P<subtask>.*)' % str(lp._root_uri))51 r'^%skernel-sru-workflow/(?P<subtask>.*)' % str(lp._root_uri))
52 prep_re = re.compile(r'prepare-package(?P<subpackage>.*)')52 prep_re = re.compile(r'prepare-package(?P<subpackage>.*)')
diff --git a/publish-image-set b/publish-image-set
index e9669c3..4f1c3bc 100755
--- a/publish-image-set
+++ b/publish-image-set
@@ -40,21 +40,21 @@ import sys
40# See isotracker.py for setup instructions.40# See isotracker.py for setup instructions.
41from isotracker import ISOTracker41from isotracker import ISOTracker
4242
43milestone_name_re = re.compile('(Alpha|Beta|RC|Final|Pre-release)(?: (\d))?')43milestone_name_re = re.compile(r'(Alpha|Beta|RC|Final|Pre-release)(?: (\d))?')
44stable_name_re = re.compile('(Xenial|Bionic|Focal) (16|18|20)\.04\.\d+')44stable_name_re = re.compile(r'(Xenial|Bionic|Focal) (16|18|20)\.04\.\d+')
4545
46# do not warn about not being able to handle those46# do not warn about not being able to handle those
47ignore_product_re = re.compile(47ignore_product_re = re.compile(
48 'Netboot |Upgrade |Server EC2|Server Windows Azure|line-through|Ubuntu WSL')48 'Netboot |Upgrade |Server EC2|Server Windows Azure|line-through|Ubuntu WSL')
49# this identifies known builds49# this identifies known builds
50product_re = re.compile(50product_re = re.compile(
51 '((?:|u|lu|ku|edu|xu|myth)buntu(?: studio|kylin| kylin| gnome| mate| budgie| next)?) '51 r'((?:|u|lu|ku|edu|xu|myth)buntu(?: studio|kylin| kylin| gnome| mate| budgie| next)?) '
52 '(alternate|desktop|dvd|server(?: subiquity)?|mobile|base|active|wubi)(?: preinstalled)? '52 r'(alternate|desktop|dvd|server(?: subiquity)?|mobile|base|active|wubi)(?: preinstalled)? '
53 '(i386|amd64$|amd64\+mac|armel$|armel\+dove|armel\+omap$|armel\+omap4|'53 r'(i386|amd64$|amd64\+mac|armel$|armel\+dove|armel\+omap$|armel\+omap4|'
54 'armel\+ac100|armel\+mx5|armhf$|armhf\+omap$|armhf\+omap4|armhf\+ac100|'54 r'armel\+ac100|armel\+mx5|armhf$|armhf\+omap$|armhf\+omap4|armhf\+ac100|'
55 'armhf\+mx5|armhf\+nexus7|armhf\+raspi$|armhf\+raspi2|armhf\+raspi3|'55 r'armhf\+mx5|armhf\+nexus7|armhf\+raspi$|armhf\+raspi2|armhf\+raspi3|'
56 'arm64$|arm64\+raspi$|arm64\+raspi3|powerpc|ppc64el|s390x|riscv64$|'56 r'arm64$|arm64\+raspi$|arm64\+raspi3|powerpc|ppc64el|s390x|riscv64$|'
57 'riscv64\+unleashed|riscv64\+unmatched)', re.I)57 r'riscv64\+unleashed|riscv64\+unmatched)', re.I)
5858
59# map an image type from the ISO tracker to a source directory for59# map an image type from the ISO tracker to a source directory for
60# publish-release60# publish-release
diff --git a/queuediff b/queuediff
index 5719464..cc7c731 100755
--- a/queuediff
+++ b/queuediff
@@ -98,11 +98,11 @@ def from_queue(sourcepkg, release):
98 '''98 '''
99 oops_re = re.compile('class="oopsid">(OOPS[a-zA-Z0-9-]+)<')99 oops_re = re.compile('class="oopsid">(OOPS[a-zA-Z0-9-]+)<')
100 changes_re = re.compile(100 changes_re = re.compile(
101 'href="(http://launchpadlibrarian.net/\d+/%s_[^"]+_source.changes)"' %101 r'href="(http://launchpadlibrarian.net/\d+/%s_[^"]+_source.changes)"' %
102 re.escape(quote(sourcepkg)))102 re.escape(quote(sourcepkg)))
103 debdiff_re = re.compile(103 debdiff_re = re.compile(
104 'href="(http://launchpadlibrarian.net/'104 r'href="(http://launchpadlibrarian.net/'
105 '\d+/%s_[^"_]+_[^_"]+\.diff\.gz)">\s*diff from' %105 r'\d+/%s_[^"_]+_[^_"]+\.diff\.gz)">\s*diff from' %
106 re.escape(quote(sourcepkg)))106 re.escape(quote(sourcepkg)))
107107
108 queue_html = urlopen(queue_url % release).read()108 queue_html = urlopen(queue_url % release).read()
@@ -141,7 +141,7 @@ def from_ppa(sourcepkg, release, user, ppaname):
141 'href="(https://launchpad.net/[^ "]+/%s_[^"]+_source.changes)"' %141 'href="(https://launchpad.net/[^ "]+/%s_[^"]+_source.changes)"' %
142 re.escape(quote(sourcepkg)))142 re.escape(quote(sourcepkg)))
143 sourcepub_re = re.compile(143 sourcepub_re = re.compile(
144 'href="(\+sourcepub/\d+/\+listing-archive-extra)"')144 r'href="(\+sourcepub/\d+/\+listing-archive-extra)"')
145 #debdiff_re = re.compile(145 #debdiff_re = re.compile(
146 # 'href="(https://launchpad.net/.[^ "]+.diff.gz)">diff from')146 # 'href="(https://launchpad.net/.[^ "]+.diff.gz)">diff from')
147147
diff --git a/sru-review b/sru-review
index 0f55d5d..2682727 100755
--- a/sru-review
+++ b/sru-review
@@ -217,8 +217,8 @@ def from_queue(options, archive, sourcepkg, series, version=None):
217 else:217 else:
218 re_version = '[^_"]+'218 re_version = '[^_"]+'
219 debdiff_re = re.compile(219 debdiff_re = re.compile(
220 'href="(http://launchpadlibrarian.net/'220 r'href="(http://launchpadlibrarian.net/'
221 '\d+/%s_[^"_]+_%s\.diff\.gz)">\s*diff from' %221 r'\d+/%s_[^"_]+_%s\.diff\.gz)">\s*diff from' %
222 (re.escape(sourcepkg), re_version))222 (re.escape(sourcepkg), re_version))
223223
224 queue_html = urlopen(queue_url).read().decode('utf-8')224 queue_html = urlopen(queue_url).read().decode('utf-8')
@@ -250,7 +250,7 @@ def from_ppa(options, sourcepkg, user, ppaname):
250 'href="(https://launchpad.net/[^ "]+/%s_[^"]+_source.changes)"' %250 'href="(https://launchpad.net/[^ "]+/%s_[^"]+_source.changes)"' %
251 re.escape(sourcepkg))251 re.escape(sourcepkg))
252 sourcepub_re = re.compile(252 sourcepub_re = re.compile(
253 'href="(\+sourcepub/\d+/\+listing-archive-extra)"')253 r'href="(\+sourcepub/\d+/\+listing-archive-extra)"')
254 #debdiff_re = re.compile(254 #debdiff_re = re.compile(
255 # 'href="(https://launchpad.net/.[^ "]+.diff.gz)">diff from')255 # 'href="(https://launchpad.net/.[^ "]+.diff.gz)">diff from')
256256

Subscribers

People subscribed via source and target branches