Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Side by Side Diff: abp/filters/sources.py

Issue 29968569: Issue 4014 - Publish python-abp on PyPI (Closed) Base URL: https://hg.adblockplus.org/python-abp/
Patch Set: Add MANIFEST.in Created Dec. 29, 2018, 2:01 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
OLDNEW
1 # This file is part of Adblock Plus <https://adblockplus.org/>, 1 # This file is part of Adblock Plus <https://adblockplus.org/>,
2 # Copyright (C) 2006-present eyeo GmbH 2 # Copyright (C) 2006-present eyeo GmbH
3 # 3 #
4 # Adblock Plus is free software: you can redistribute it and/or modify 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 5 # it under the terms of the GNU General Public License version 3 as
6 # published by the Free Software Foundation. 6 # published by the Free Software Foundation.
7 # 7 #
8 # Adblock Plus is distributed in the hope that it will be useful, 8 # Adblock Plus is distributed in the hope that it will be useful,
9 # but WITHOUT ANY WARRANTY; without even the implied warranty of 9 # but WITHOUT ANY WARRANTY; without even the implied warranty of
10 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the 10 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
11 # GNU General Public License for more details. 11 # GNU General Public License for more details.
12 # 12 #
13 # You should have received a copy of the GNU General Public License 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/>. 14 # along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>.
15 15
16 """Helper classes that handle IO for filter list parsing and rendering.""" 16 """Helper classes that handle IO for filter list parsing and rendering."""
17 17
18 import io 18 import io
19 from os import path 19 from os import path
20 import sys 20 import sys
21 21
22 try: 22 try:
23 from urllib2 import urlopen, HTTPError 23 from urllib2 import urlopen, HTTPError
24 except ImportError: # The module was renamed in Python 3. 24 except ImportError: # pragma: no py2 cover
25 # The module was renamed in Python 3
Sebastian Noack 2018/12/29 03:24:56 Nit: Would you agree that this comment is somewhat
Vasily Kuznetsov 2019/01/02 18:32:28 I would agree, the pragma comment seems informativ
rhowell 2019/01/03 04:42:27 Done.
25 from urllib.request import urlopen 26 from urllib.request import urlopen
26 from urllib.error import HTTPError 27 from urllib.error import HTTPError
27 28
28 __all__ = ['NotFound', 'FSSource', 'TopSource', 'WebSource'] 29 __all__ = ['NotFound', 'FSSource', 'TopSource', 'WebSource']
29 30
30 31
31 class NotFound(Exception): 32 class NotFound(Exception):
32 """Requested file doesn't exist in this source. 33 """Requested file doesn't exist in this source.
33 34
34 The file with requested name doesn't exist. If this results from an 35 The file with requested name doesn't exist. If this results from an
(...skipping 137 matching lines...) Expand 10 before | Expand all | Expand 10 after
172 get_param = (getattr(info, 'get_param', None) 173 get_param = (getattr(info, 'get_param', None)
173 or getattr(info, 'getparam', None)) 174 or getattr(info, 'getparam', None))
174 encoding = get_param('charset') or self.default_encoding 175 encoding = get_param('charset') or self.default_encoding
175 for line in response: 176 for line in response:
176 yield line.decode(encoding).rstrip() 177 yield line.decode(encoding).rstrip()
177 except HTTPError as err: 178 except HTTPError as err:
178 if err.code == 404: 179 if err.code == 404:
179 raise NotFound("HTTP 404 Not found: '{}:{}'" 180 raise NotFound("HTTP 404 Not found: '{}:{}'"
180 .format(self.protocol, path_in_source)) 181 .format(self.protocol, path_in_source))
181 raise err 182 raise err
OLDNEW

Powered by Google App Engine
This is Rietveld