Left: | ||
Right: |
OLD | NEW |
---|---|
(Empty) | |
1 # This file is part of Adblock Plus <https://adblockplus.org/>, | |
2 # Copyright (C) 2006-present eyeo GmbH | |
3 # | |
4 # Adblock Plus is free software: you can redistribute it and/or modify | |
5 # it under the terms of the GNU General Public License version 3 as | |
6 # published by the Free Software Foundation. | |
7 # | |
8 # Adblock Plus is distributed in the hope that it will be useful, | |
9 # but WITHOUT ANY WARRANTY; without even the implied warranty of | |
10 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | |
11 # GNU General Public License for more details. | |
12 # | |
13 # You should have received a copy of the GNU General Public License | |
14 # along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>. | |
15 | |
16 """Command line script for rendering Adblock Plus filter list diffs.""" | |
17 | |
18 from __future__ import print_function | |
19 import argparse | |
Sebastian Noack
2018/09/21 11:51:21
Nit: __future__ imports are special, so I would se
rhowell
2018/09/24 22:05:21
Done.
| |
20 import io | |
21 import sys | |
22 | |
23 from .sources import NotFound | |
24 from .renderer import render_diff | |
25 __all__ = ['main'] | |
Sebastian Noack
2018/09/21 11:51:21
Nit: There should be blank link above, here as wel
rhowell
2018/09/24 22:05:21
Done.
| |
26 | |
27 | |
28 def parse_args(): | |
29 parser = argparse.ArgumentParser(description='Render a filter list diff.') | |
30 parser.add_argument( | |
31 'base', help='the older filter list that needs to be updated', | |
32 nargs='?') | |
33 parser.add_argument( | |
34 'latest', help='the most recent version of the filter list', | |
35 nargs='?') | |
36 parser.add_argument( | |
37 'outfile', help='output file for filter list diff', | |
38 default='-', nargs='?') | |
39 return parser.parse_args() | |
40 | |
41 | |
42 def main(args_in=None): | |
Sebastian Noack
2018/09/21 11:51:20
The args_in argument seems to be unused, and I don
rhowell
2018/09/24 22:05:21
Oops, I should've removed this when removing the e
| |
43 """Entry point for the diff rendering script (diffrender).""" | |
44 args = parse_args() | |
45 | |
46 with io.open(args.base, 'r') as base, io.open(args.latest, 'r') as latest: | |
Sebastian Noack
2018/09/21 11:51:21
Unlike below, you don't specify an encoding when c
rhowell
2018/09/24 22:05:21
Done.
| |
47 try: | |
48 lines = render_diff(base, latest) | |
49 if args.outfile == '-': | |
50 outfile = io.open(sys.stdout.fileno(), 'w', | |
51 closefd=False, | |
52 encoding=sys.stdout.encoding or 'utf-8') | |
53 else: | |
54 outfile = io.open(args.outfile, 'w', encoding='utf-8') | |
55 | |
56 with outfile: | |
57 for line in lines: | |
58 print(line, file=outfile) | |
59 except NotFound as exc: | |
Sebastian Noack
2018/09/21 11:51:20
Can we even run into this exception since we neith
rhowell
2018/09/24 22:05:21
I guess not. Removed it.
| |
60 sys.exit(exc) | |
OLD | NEW |