Re: [gentoo-dev] [PATCH] python-single-r1.eclass: don't crash Portage with invalid USEDEP syntax

[Mike Gilbert]

On Wed, Feb 12, 2020 at 1:31 PM Michał Górny <mgorny@g.o> wrote:
>
> On Wed, 2020-02-12 at 13:25 -0500, Mike Gilbert wrote:
> > This should still serve the purpose of alerting overlay maintainers
> > without making emerge completely unusable in the interim.
> >
>
> I don't understand what's the gain.  In both cases emerge won't proceed.
> However, with the original syntax the message is clearer:

A pentoo user reported an actual crash in portage when it was unable
to parse the dependency atom. See the backlog in #gentoo-qa for the
discussion.

However, I have been unable to reproduce this crash myself, so I'm
going to hold off on merging this patch until the problem can be
further diagnosed.

The portage output is below.

Calculating dependencies  ... ..... done!
Traceback (most recent call last):
  File "/usr/lib64/python3.6/site-packages/portage/dep/__init__.py",
line 739, in use_reduce
    is_valid_flag=is_valid_flag)
  File "/usr/lib64/python3.6/site-packages/portage/dep/__init__.py",
line 1411, in __init__
    use = _use_dep(use_str[1:-1].split(","), eapi_attrs)
  File "/usr/lib64/python3.6/site-packages/portage/dep/__init__.py",
line 897, in __init__
    raise InvalidAtom(_("Invalid use dep: '%s'") % (x,))
portage.exception.InvalidAtom: Invalid use dep:
'%PYTHON_USEDEP-HAS-BEEN-REMOVED%'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python-exec/python3.6/emerge", line 53, in <module>
    retval = emerge_main()
  File "/usr/lib64/python3.6/site-packages/_emerge/main.py", line
1309, in emerge_main
    return run_action(emerge_config)
  File "/usr/lib64/python3.6/site-packages/_emerge/actions.py", line
3358, in run_action
    retval = action_build(emerge_config, spinner=spinner)
  File "/usr/lib64/python3.6/site-packages/_emerge/actions.py", line
357, in action_build
    settings, trees, myopts, myparams, myaction, myfiles, spinner)
  File "/usr/lib64/python3.6/site-packages/_emerge/depgraph.py", line
9891, in backtrack_depgraph
    myaction, myfiles, spinner)
  File "/usr/lib64/python3.6/site-packages/_emerge/depgraph.py", line
9928, in _backtrack_depgraph
    success, favorites = mydepgraph.select_files(myfiles)
  File "/usr/lib64/python3.6/site-packages/_emerge/depgraph.py", line
3990, in select_files
    return self._select_files(args)
  File "/usr/lib64/python3.6/site-packages/_emerge/depgraph.py", line
4333, in _select_files
    return self._resolve(myfavorites)
  File "/usr/lib64/python3.6/site-packages/_emerge/depgraph.py", line
4484, in _resolve
    if not self._create_graph():
  File "/usr/lib64/python3.6/site-packages/_emerge/depgraph.py", line
2722, in _create_graph
    allow_unsatisfied=allow_unsatisfied):
  File "/usr/lib64/python3.6/site-packages/_emerge/depgraph.py", line
3480, in _add_pkg_deps
    allow_unsatisfied):
  File "/usr/lib64/python3.6/site-packages/_emerge/depgraph.py", line
3496, in _add_pkg_dep_string
    allow_unsatisfied)
  File "/usr/lib64/python3.6/site-packages/_emerge/depgraph.py", line
3576, in _wrapped_add_pkg_dep_string
    pkg, dep_priority, root_config, selected_atoms[pkg]):
  File "/usr/lib64/python3.6/site-packages/_emerge/depgraph.py", line
3775, in _minimize_children
    root_config.root, atom, parent=parent)
  File "/usr/lib64/python3.6/site-packages/_emerge/depgraph.py", line
5775, in _select_pkg_highest_available
    ret = self._select_pkg_highest_available_imp(root, atom,
onlydeps=onlydeps, parent=parent)
  File "/usr/lib64/python3.6/site-packages/_emerge/depgraph.py", line
6003, in _select_pkg_highest_available_imp
    root, atom, onlydeps=onlydeps, parent=parent)
  File "/usr/lib64/python3.6/site-packages/_emerge/depgraph.py", line
6600, in _wrapped_select_pkg_highest_available_imp
    self._changed_deps(pkg))):
  File "/usr/lib64/python3.6/site-packages/_emerge/depgraph.py", line
2697, in _changed_deps
    eapi=ebuild.eapi, token_class=Atom)
  File "/usr/lib64/python3.6/site-packages/portage/dep/__init__.py",
line 744, in use_reduce
    % (e, pos+1), errors=(e,))
portage.exception.InvalidDependString: Invalid atom (Invalid use dep:
'%PYTHON_USEDEP-HAS-BEEN-REMOVED%'), token 6