* [gentoo-portage-dev] [PATCH] repoman: fix erroneous LICENSE.syntax (bug 591184)
@ 2016-08-13 23:00 Zac Medico
2016-08-14 15:18 ` Brian Dolbec
0 siblings, 1 reply; 3+ messages in thread
From: Zac Medico @ 2016-08-13 23:00 UTC (permalink / raw
To: gentoo-portage-dev; +Cc: Zac Medico
X-Gentoo-bug: 591184
X-Gentoo-bug-url: https://bugs.gentoo.org/show_bug.cgi?id=591184
---
.../pym/repoman/modules/scan/depend/_depend_checks.py | 16 +++++++---------
1 file changed, 7 insertions(+), 9 deletions(-)
diff --git a/repoman/pym/repoman/modules/scan/depend/_depend_checks.py b/repoman/pym/repoman/modules/scan/depend/_depend_checks.py
index 3f6c93e..f0ae863 100644
--- a/repoman/pym/repoman/modules/scan/depend/_depend_checks.py
+++ b/repoman/pym/repoman/modules/scan/depend/_depend_checks.py
@@ -27,7 +27,7 @@ def check_slotop(depstr, is_valid_flag, badsyntax, mytype,
token_class=portage.dep.Atom)
except portage.exception.InvalidDependString as e:
my_dep_tree = None
- badsyntax.append(str(e))
+ badsyntax.append((mytype, str(e)))
def _traverse_tree(dep_tree, in_any_of):
# leaf
@@ -67,7 +67,7 @@ def _depend_checks(ebuild, pkg, portdb, qatracker, repo_metadata):
"java-pkg-opt-2" in ebuild.inherited,
inherited_wxwidgets_eclass = "wxwidgets" in ebuild.inherited
# operator_tokens = set(["||", "(", ")"])
- type_list, badsyntax = [], []
+ badsyntax = []
for mytype in Package._dep_keys + ("LICENSE", "PROPERTIES", "PROVIDE"):
mydepstr = ebuild.metadata[mytype]
@@ -83,7 +83,7 @@ def _depend_checks(ebuild, pkg, portdb, qatracker, repo_metadata):
is_valid_flag=pkg.iuse.is_valid_flag, token_class=token_class)
except portage.exception.InvalidDependString as e:
atoms = None
- badsyntax.append(str(e))
+ badsyntax.append((mytype, str(e)))
if atoms and mytype.endswith("DEPEND"):
if runtime and \
@@ -156,13 +156,11 @@ def _depend_checks(ebuild, pkg, portdb, qatracker, repo_metadata):
check_missingslot(atom, mytype, ebuild.eapi, portdb, qatracker,
ebuild.relative_path, ebuild.metadata)
- type_list.extend([mytype] * (len(badsyntax) - len(type_list)))
-
if runtime:
check_slotop(mydepstr, pkg.iuse.is_valid_flag,
badsyntax, mytype, qatracker, ebuild.relative_path)
- for m, b in zip(type_list, badsyntax):
+ for m, b in badsyntax:
if m.endswith("DEPEND"):
qacat = "dependency.syntax"
else:
@@ -171,9 +169,9 @@ def _depend_checks(ebuild, pkg, portdb, qatracker, repo_metadata):
qacat, "%s: %s: %s" % (ebuild.relative_path, m, b))
# data required for some other tests
- badlicsyntax = len([z for z in type_list if z == "LICENSE"])
- badprovsyntax = len([z for z in type_list if z == "PROVIDE"])
- baddepsyntax = len(type_list) != badlicsyntax + badprovsyntax
+ badlicsyntax = len([z for z in badsyntax if z[0] == "LICENSE"])
+ badprovsyntax = len([z for z in badsyntax if z[0] == "PROVIDE"])
+ baddepsyntax = len(badsyntax) != badlicsyntax + badprovsyntax
badlicsyntax = badlicsyntax > 0
#badprovsyntax = badprovsyntax > 0
--
2.7.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [gentoo-portage-dev] [PATCH] repoman: fix erroneous LICENSE.syntax (bug 591184)
2016-08-13 23:00 [gentoo-portage-dev] [PATCH] repoman: fix erroneous LICENSE.syntax (bug 591184) Zac Medico
@ 2016-08-14 15:18 ` Brian Dolbec
2016-08-14 18:53 ` Zac Medico
0 siblings, 1 reply; 3+ messages in thread
From: Brian Dolbec @ 2016-08-14 15:18 UTC (permalink / raw
To: gentoo-portage-dev
On Sat, 13 Aug 2016 16:00:59 -0700
Zac Medico <zmedico@gentoo.org> wrote:
> X-Gentoo-bug: 591184
> X-Gentoo-bug-url: https://bugs.gentoo.org/show_bug.cgi?id=591184
> ---
> .../pym/repoman/modules/scan/depend/_depend_checks.py | 16
> +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-)
>
> diff --git
> a/repoman/pym/repoman/modules/scan/depend/_depend_checks.py
> b/repoman/pym/repoman/modules/scan/depend/_depend_checks.py index
> 3f6c93e..f0ae863 100644 ---
> a/repoman/pym/repoman/modules/scan/depend/_depend_checks.py +++
> b/repoman/pym/repoman/modules/scan/depend/_depend_checks.py @@ -27,7
> +27,7 @@ def check_slotop(depstr, is_valid_flag, badsyntax, mytype,
> token_class=portage.dep.Atom) except
> portage.exception.InvalidDependString as e: my_dep_tree = None
> - badsyntax.append(str(e))
> + badsyntax.append((mytype, str(e)))
>
> def _traverse_tree(dep_tree, in_any_of):
> # leaf
> @@ -67,7 +67,7 @@ def _depend_checks(ebuild, pkg, portdb, qatracker,
> repo_metadata): "java-pkg-opt-2" in ebuild.inherited,
> inherited_wxwidgets_eclass = "wxwidgets" in ebuild.inherited
> # operator_tokens = set(["||", "(", ")"])
> - type_list, badsyntax = [], []
> + badsyntax = []
> for mytype in Package._dep_keys + ("LICENSE", "PROPERTIES",
> "PROVIDE"): mydepstr = ebuild.metadata[mytype]
>
> @@ -83,7 +83,7 @@ def _depend_checks(ebuild, pkg, portdb, qatracker,
> repo_metadata): is_valid_flag=pkg.iuse.is_valid_flag,
> token_class=token_class) except portage.exception.InvalidDependString
> as e: atoms = None
> - badsyntax.append(str(e))
> + badsyntax.append((mytype, str(e)))
>
> if atoms and mytype.endswith("DEPEND"):
> if runtime and \
> @@ -156,13 +156,11 @@ def _depend_checks(ebuild, pkg, portdb,
> qatracker, repo_metadata): check_missingslot(atom, mytype,
> ebuild.eapi, portdb, qatracker, ebuild.relative_path, ebuild.metadata)
>
> - type_list.extend([mytype] * (len(badsyntax) -
> len(type_list))) -
> if runtime:
> check_slotop(mydepstr,
> pkg.iuse.is_valid_flag, badsyntax, mytype, qatracker,
> ebuild.relative_path)
> - for m, b in zip(type_list, badsyntax):
> + for m, b in badsyntax:
> if m.endswith("DEPEND"):
> qacat = "dependency.syntax"
> else:
> @@ -171,9 +169,9 @@ def _depend_checks(ebuild, pkg, portdb,
> qatracker, repo_metadata): qacat, "%s: %s: %s" %
> (ebuild.relative_path, m, b))
> # data required for some other tests
> - badlicsyntax = len([z for z in type_list if z == "LICENSE"])
> - badprovsyntax = len([z for z in type_list if z == "PROVIDE"])
> - baddepsyntax = len(type_list) != badlicsyntax + badprovsyntax
> + badlicsyntax = len([z for z in badsyntax if z[0] ==
> "LICENSE"])
> + badprovsyntax = len([z for z in badsyntax if z[0] ==
> "PROVIDE"])
> + baddepsyntax = len(badsyntax) != badlicsyntax + badprovsyntax
> badlicsyntax = badlicsyntax > 0
> #badprovsyntax = badprovsyntax > 0
>
looks good, thanks for the fast fix Zac.
--
Brian Dolbec <dolsen>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [gentoo-portage-dev] [PATCH] repoman: fix erroneous LICENSE.syntax (bug 591184)
2016-08-14 15:18 ` Brian Dolbec
@ 2016-08-14 18:53 ` Zac Medico
0 siblings, 0 replies; 3+ messages in thread
From: Zac Medico @ 2016-08-14 18:53 UTC (permalink / raw
To: gentoo-portage-dev
On 08/14/2016 08:18 AM, Brian Dolbec wrote:
> On Sat, 13 Aug 2016 16:00:59 -0700
> Zac Medico <zmedico@gentoo.org> wrote:
>
>> X-Gentoo-bug: 591184
>> X-Gentoo-bug-url: https://bugs.gentoo.org/show_bug.cgi?id=591184
>> ---
>> .../pym/repoman/modules/scan/depend/_depend_checks.py | 16
>> +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-)
>
> looks good, thanks for the fast fix Zac.
>
Pushed:
https://gitweb.gentoo.org/proj/portage.git/commit/?id=99d8cdc0e9b011d39ae8537169ce2ca70a4e5f83
--
Thanks,
Zac
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-08-14 18:53 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-08-13 23:00 [gentoo-portage-dev] [PATCH] repoman: fix erroneous LICENSE.syntax (bug 591184) Zac Medico
2016-08-14 15:18 ` Brian Dolbec
2016-08-14 18:53 ` Zac Medico
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox