Skip to content

Commit f55e478

Browse files
committed
Trap exception directly instead of tracking with boolean.
1 parent 965af23 commit f55e478

File tree

1 file changed

+3
-12
lines changed

1 file changed

+3
-12
lines changed

setuptools/config/setupcfg.py

+3-12
Original file line numberDiff line numberDiff line change
@@ -293,18 +293,9 @@ def __setitem__(self, option_name, value):
293293
# Already inhabited. Skipping.
294294
return
295295

296-
skip_option = False
297-
parser = self.parsers.get(option_name)
298-
if parser:
299-
try:
300-
parsed = parser(value)
301-
302-
except Exception:
303-
skip_option = True
304-
if not self.ignore_option_errors:
305-
raise
306-
307-
if skip_option:
296+
try:
297+
parsed = self.parsers.get(option_name, lambda x: x)(value)
298+
except (Exception,) * self.ignore_option_errors:
308299
return
309300

310301
setter = getattr(target_obj, 'set_%s' % option_name, None)

0 commit comments

Comments
 (0)