diff --git a/srcpkgs/pep8/patches/revert-cli-options-only.patch b/srcpkgs/pep8/patches/revert-cli-options-only.patch new file mode 100644 index 00000000000..cf8c576e420 --- /dev/null +++ b/srcpkgs/pep8/patches/revert-cli-options-only.patch @@ -0,0 +1,73 @@ +From 435d1cbf995a659a82d1d4b42d25e3459556ef21 Mon Sep 17 00:00:00 2001 +From: Ian Lee +Date: Tue, 17 Mar 2015 21:52:23 -0700 +Subject: [PATCH] Reverted fix for #368 which had unintended repurcussions in + flake8 and other places. + +This fix reverts to the parsing of user config (~/.config/pep8), then local directory +config files, and finally overrides with cli options as was the behavior back in 1.5.7 +--- + CHANGES.txt | 6 ++++++ + pep8.py | 28 ++++++++++++++-------------- + 2 files changed, 20 insertions(+), 14 deletions(-) + +diff --git CHANGES.txt.orig CHANGES.txt +index 85eb043..5499abd 100644 +--- CHANGES.txt.orig ++++ CHANGES.txt +@@ -5,6 +5,12 @@ Changelog + 1.6.x (unreleased) + ------------------ + ++Changes: ++ ++* Reverted the fix in #368, "options passed on command line are only ones ++ accepted" feature. This has many unintended consequences in pep8 and flake8 ++ and needs to be reworked when I have more time. ++ + 1.6.2 (2015-02-15) + ------------------ + +diff --git pep8.py.orig pep8.py +index 9f40381..4d993da 100755 +--- pep8.py.orig ++++ pep8.py +@@ -1984,24 +1984,24 @@ def read_config(options, args, arglist, parser): + + local_dir = os.curdir + ++ if USER_CONFIG and os.path.isfile(USER_CONFIG): ++ if options.verbose: ++ print('user configuration: %s' % USER_CONFIG) ++ config.read(USER_CONFIG) ++ ++ parent = tail = args and os.path.abspath(os.path.commonprefix(args)) ++ while tail: ++ if config.read(os.path.join(parent, fn) for fn in PROJECT_CONFIG): ++ local_dir = parent ++ if options.verbose: ++ print('local configuration: in %s' % parent) ++ break ++ (parent, tail) = os.path.split(parent) ++ + if cli_conf and os.path.isfile(cli_conf): + if options.verbose: + print('cli configuration: %s' % cli_conf) + config.read(cli_conf) +- else: +- if USER_CONFIG and os.path.isfile(USER_CONFIG): +- if options.verbose: +- print('user configuration: %s' % USER_CONFIG) +- config.read(USER_CONFIG) +- +- parent = tail = args and os.path.abspath(os.path.commonprefix(args)) +- while tail: +- if config.read(os.path.join(parent, fn) for fn in PROJECT_CONFIG): +- local_dir = parent +- if options.verbose: +- print('local configuration: in %s' % parent) +- break +- (parent, tail) = os.path.split(parent) + + pep8_section = parser.prog + if config.has_section(pep8_section): diff --git a/srcpkgs/pep8/template b/srcpkgs/pep8/template index 5fcd550fd2f..7ad03c6f2fd 100644 --- a/srcpkgs/pep8/template +++ b/srcpkgs/pep8/template @@ -1,7 +1,7 @@ # Template file for 'pep8' pkgname=pep8 version=1.6.2 -revision=1 +revision=2 noarch=yes build_style=python-module hostmakedepends="python-setuptools"