diff options
author | David Pursehouse <dpursehouse@collab.net> | 2020-02-13 13:17:03 +0900 |
---|---|---|
committer | David Pursehouse <dpursehouse@collab.net> | 2020-02-13 04:54:10 +0000 |
commit | 3cda50a41b1a950d70be8887c5efe3735586d1bb (patch) | |
tree | 35c40d0981784c2a1facb4a3fd62d99d53d8ae2f | |
parent | afbccdb11e3aad56ba9b07f4321ec4752a0edb93 (diff) | |
download | git-repo-3cda50a41b1a950d70be8887c5efe3735586d1bb.tar.gz |
pyflakes: Fix remaining "E501 line too long" warnings
We increased the max line length to 100 columns which got rid of
the majority of these warnings, but there were still a few lines
that exceeded 100 columns.
Change-Id: Ib3372868ca2297f83073a14f91c8ae3df9d0d0e6
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/254699
Tested-by: David Pursehouse <dpursehouse@collab.net>
Reviewed-by: Mike Frysinger <vapier@google.com>
-rwxr-xr-x | main.py | 3 | ||||
-rw-r--r-- | pager.py | 3 | ||||
-rw-r--r-- | platform_utils_win32.py | 3 | ||||
-rw-r--r-- | subcmds/forall.py | 3 | ||||
-rw-r--r-- | subcmds/gitc_init.py | 3 | ||||
-rw-r--r-- | tests/test_wrapper.py | 3 |
6 files changed, 12 insertions, 6 deletions
@@ -229,7 +229,8 @@ class _Repo(object): | |||
229 | if e.name: | 229 | if e.name: |
230 | print('error: project group must be enabled for project %s' % e.name, file=sys.stderr) | 230 | print('error: project group must be enabled for project %s' % e.name, file=sys.stderr) |
231 | else: | 231 | else: |
232 | print('error: project group must be enabled for the project in the current directory', file=sys.stderr) | 232 | print('error: project group must be enabled for the project in the current directory', |
233 | file=sys.stderr) | ||
233 | result = 1 | 234 | result = 1 |
234 | except SystemExit as e: | 235 | except SystemExit as e: |
235 | if e.code: | 236 | if e.code: |
@@ -59,7 +59,8 @@ def _PipePager(pager): | |||
59 | global pager_process, old_stdout, old_stderr | 59 | global pager_process, old_stdout, old_stderr |
60 | assert pager_process is None, "Only one active pager process at a time" | 60 | assert pager_process is None, "Only one active pager process at a time" |
61 | # Create pager process, piping stdout/err into its stdin | 61 | # Create pager process, piping stdout/err into its stdin |
62 | pager_process = subprocess.Popen([pager], stdin=subprocess.PIPE, stdout=sys.stdout, stderr=sys.stderr) | 62 | pager_process = subprocess.Popen([pager], stdin=subprocess.PIPE, stdout=sys.stdout, |
63 | stderr=sys.stderr) | ||
63 | old_stdout = sys.stdout | 64 | old_stdout = sys.stdout |
64 | old_stderr = sys.stderr | 65 | old_stderr = sys.stderr |
65 | sys.stdout = pager_process.stdin | 66 | sys.stdout = pager_process.stdin |
diff --git a/platform_utils_win32.py b/platform_utils_win32.py index 8eb3d2c3..c59db523 100644 --- a/platform_utils_win32.py +++ b/platform_utils_win32.py | |||
@@ -152,7 +152,8 @@ def create_dirsymlink(source, link_name): | |||
152 | 152 | ||
153 | 153 | ||
154 | def _create_symlink(source, link_name, dwFlags): | 154 | def _create_symlink(source, link_name, dwFlags): |
155 | if not CreateSymbolicLinkW(link_name, source, dwFlags | SYMBOLIC_LINK_FLAG_ALLOW_UNPRIVILEGED_CREATE): | 155 | if not CreateSymbolicLinkW(link_name, source, |
156 | dwFlags | SYMBOLIC_LINK_FLAG_ALLOW_UNPRIVILEGED_CREATE): | ||
156 | # See https://github.com/golang/go/pull/24307/files#diff-b87bc12e4da2497308f9ef746086e4f0 | 157 | # See https://github.com/golang/go/pull/24307/files#diff-b87bc12e4da2497308f9ef746086e4f0 |
157 | # "the unprivileged create flag is unsupported below Windows 10 (1703, v10.0.14972). | 158 | # "the unprivileged create flag is unsupported below Windows 10 (1703, v10.0.14972). |
158 | # retry without it." | 159 | # retry without it." |
diff --git a/subcmds/forall.py b/subcmds/forall.py index dbf26f0b..d0e51920 100644 --- a/subcmds/forall.py +++ b/subcmds/forall.py | |||
@@ -127,7 +127,8 @@ without iterating through the remaining projects. | |||
127 | help="Execute the command only on projects matching regex or wildcard expression") | 127 | help="Execute the command only on projects matching regex or wildcard expression") |
128 | p.add_option('-i', '--inverse-regex', | 128 | p.add_option('-i', '--inverse-regex', |
129 | dest='inverse_regex', action='store_true', | 129 | dest='inverse_regex', action='store_true', |
130 | help="Execute the command only on projects not matching regex or wildcard expression") | 130 | help="Execute the command only on projects not matching regex or " |
131 | "wildcard expression") | ||
131 | p.add_option('-g', '--groups', | 132 | p.add_option('-g', '--groups', |
132 | dest='groups', | 133 | dest='groups', |
133 | help="Execute the command only on projects matching the specified groups") | 134 | help="Execute the command only on projects matching the specified groups") |
diff --git a/subcmds/gitc_init.py b/subcmds/gitc_init.py index 378f9236..30a9f7a5 100644 --- a/subcmds/gitc_init.py +++ b/subcmds/gitc_init.py | |||
@@ -62,7 +62,8 @@ use for this GITC client. | |||
62 | def Execute(self, opt, args): | 62 | def Execute(self, opt, args): |
63 | gitc_client = gitc_utils.parse_clientdir(os.getcwd()) | 63 | gitc_client = gitc_utils.parse_clientdir(os.getcwd()) |
64 | if not gitc_client or (opt.gitc_client and gitc_client != opt.gitc_client): | 64 | if not gitc_client or (opt.gitc_client and gitc_client != opt.gitc_client): |
65 | print('fatal: Please update your repo command. See go/gitc for instructions.', file=sys.stderr) | 65 | print('fatal: Please update your repo command. See go/gitc for instructions.', |
66 | file=sys.stderr) | ||
66 | sys.exit(1) | 67 | sys.exit(1) |
67 | self.client_dir = os.path.join(gitc_utils.get_gitc_manifest_dir(), | 68 | self.client_dir = os.path.join(gitc_utils.get_gitc_manifest_dir(), |
68 | gitc_client) | 69 | gitc_client) |
diff --git a/tests/test_wrapper.py b/tests/test_wrapper.py index 61636b26..4d2cf3fb 100644 --- a/tests/test_wrapper.py +++ b/tests/test_wrapper.py | |||
@@ -101,7 +101,8 @@ class RepoWrapperUnitTest(RepoWrapperTestCase): | |||
101 | self.assertEqual(self.wrapper.gitc_parse_clientdir('/gitc/manifest-rw/test/extra'), 'test') | 101 | self.assertEqual(self.wrapper.gitc_parse_clientdir('/gitc/manifest-rw/test/extra'), 'test') |
102 | self.assertEqual(self.wrapper.gitc_parse_clientdir('/test/usr/local/google/gitc/test'), 'test') | 102 | self.assertEqual(self.wrapper.gitc_parse_clientdir('/test/usr/local/google/gitc/test'), 'test') |
103 | self.assertEqual(self.wrapper.gitc_parse_clientdir('/test/usr/local/google/gitc/test/'), 'test') | 103 | self.assertEqual(self.wrapper.gitc_parse_clientdir('/test/usr/local/google/gitc/test/'), 'test') |
104 | self.assertEqual(self.wrapper.gitc_parse_clientdir('/test/usr/local/google/gitc/test/extra'), 'test') | 104 | self.assertEqual(self.wrapper.gitc_parse_clientdir('/test/usr/local/google/gitc/test/extra'), |
105 | 'test') | ||
105 | self.assertEqual(self.wrapper.gitc_parse_clientdir('/gitc/manifest-rw/'), None) | 106 | self.assertEqual(self.wrapper.gitc_parse_clientdir('/gitc/manifest-rw/'), None) |
106 | self.assertEqual(self.wrapper.gitc_parse_clientdir('/test/usr/local/google/gitc/'), None) | 107 | self.assertEqual(self.wrapper.gitc_parse_clientdir('/test/usr/local/google/gitc/'), None) |
107 | 108 | ||