diff options
Diffstat (limited to 'subcmds')
| -rw-r--r-- | subcmds/diff.py | 42 |
1 files changed, 38 insertions, 4 deletions
diff --git a/subcmds/diff.py b/subcmds/diff.py index c987bf23..81868176 100644 --- a/subcmds/diff.py +++ b/subcmds/diff.py | |||
| @@ -12,7 +12,11 @@ | |||
| 12 | # See the License for the specific language governing permissions and | 12 | # See the License for the specific language governing permissions and |
| 13 | # limitations under the License. | 13 | # limitations under the License. |
| 14 | 14 | ||
| 15 | from command import PagedCommand | 15 | import functools |
| 16 | import io | ||
| 17 | import multiprocessing | ||
| 18 | |||
| 19 | from command import DEFAULT_LOCAL_JOBS, PagedCommand, WORKER_BATCH_SIZE | ||
| 16 | 20 | ||
| 17 | 21 | ||
| 18 | class Diff(PagedCommand): | 22 | class Diff(PagedCommand): |
| @@ -25,15 +29,45 @@ The -u option causes '%prog' to generate diff output with file paths | |||
| 25 | relative to the repository root, so the output can be applied | 29 | relative to the repository root, so the output can be applied |
| 26 | to the Unix 'patch' command. | 30 | to the Unix 'patch' command. |
| 27 | """ | 31 | """ |
| 32 | PARALLEL_JOBS = DEFAULT_LOCAL_JOBS | ||
| 28 | 33 | ||
| 29 | def _Options(self, p): | 34 | def _Options(self, p): |
| 35 | super()._Options(p) | ||
| 30 | p.add_option('-u', '--absolute', | 36 | p.add_option('-u', '--absolute', |
| 31 | dest='absolute', action='store_true', | 37 | dest='absolute', action='store_true', |
| 32 | help='Paths are relative to the repository root') | 38 | help='Paths are relative to the repository root') |
| 33 | 39 | ||
| 40 | def _DiffHelper(self, absolute, project): | ||
| 41 | """Obtains the diff for a specific project. | ||
| 42 | |||
| 43 | Args: | ||
| 44 | absolute: Paths are relative to the root. | ||
| 45 | project: Project to get status of. | ||
| 46 | |||
| 47 | Returns: | ||
| 48 | The status of the project. | ||
| 49 | """ | ||
| 50 | buf = io.StringIO() | ||
| 51 | ret = project.PrintWorkTreeDiff(absolute, output_redir=buf) | ||
| 52 | return (ret, buf.getvalue()) | ||
| 53 | |||
| 34 | def Execute(self, opt, args): | 54 | def Execute(self, opt, args): |
| 35 | ret = 0 | 55 | ret = 0 |
| 36 | for project in self.GetProjects(args): | 56 | all_projects = self.GetProjects(args) |
| 37 | if not project.PrintWorkTreeDiff(opt.absolute): | 57 | |
| 38 | ret = 1 | 58 | # NB: Multiprocessing is heavy, so don't spin it up for one job. |
| 59 | if len(all_projects) == 1 or opt.jobs == 1: | ||
| 60 | for project in all_projects: | ||
| 61 | if not project.PrintWorkTreeDiff(opt.absolute): | ||
| 62 | ret = 1 | ||
| 63 | else: | ||
| 64 | with multiprocessing.Pool(opt.jobs) as pool: | ||
| 65 | states = pool.imap(functools.partial(self._DiffHelper, opt.absolute), | ||
| 66 | all_projects, WORKER_BATCH_SIZE) | ||
| 67 | for (state, output) in states: | ||
| 68 | if output: | ||
| 69 | print(output, end='') | ||
| 70 | if not state: | ||
| 71 | ret = 1 | ||
| 72 | |||
| 39 | return ret | 73 | return ret |
