Re: [gentoo-portage-dev] [PATCH] emerge: auto-enable --with-bdeps if --usepkg is not enabled (bug 598444)

[Zac Medico]

On 03/04/2017 02:30 AM, Zac Medico wrote:
> On 03/04/2017 01:59 AM, Michał Górny wrote:
>> W dniu 04.03.2017, sob o godzinie 01∶34 -0800, użytkownik Zac Medico
>> napisał:
>>> It's useful to enable --with-bdeps by default so that @world updates
>>> will update all packages that are not eligible for removal by
>>> emerge --depclean. However, many users of binary packages do not want
>>> unnecessary build time dependencies installed, therefore do not
>>> auto-enable --with-bdeps for installation actions when the --usepkg
>>> option is enabled.
>>>
>>> X-Gentoo-bug: 598444
>>> X-Gentoo-bug-url: https://bugs.gentoo.org/show_bug.cgi?id=598444
>>> ---
>>>  man/emerge.1                                     |  18 ++-
>>>  pym/_emerge/create_depgraph_params.py            |   4 +
>>>  pym/_emerge/depgraph.py                          |   4 +-
>>>  pym/portage/tests/resolver/ResolverPlayground.py |   5 +
>>>  pym/portage/tests/resolver/test_bdeps.py         | 197 +++++++++++++++++++++++
>>>  5 files changed, 223 insertions(+), 5 deletions(-)
>>>  create mode 100644 pym/portage/tests/resolver/test_bdeps.py
>>
>> Just to be clear, this will break stuff for people who have more than
>> one slot of LLVM enabled since different slots build-depend on colliding
>> packages.
>
> Hmm, that's unfortunate, because putting --with-bdeps=n in
> EMERGE_DEFAULT_OPTS is not good for --depclean. I'll have to think about
> a better way to handle this case. We've already got a bug open for this
> sort of situation here:
>
> https://bugs.gentoo.org/show_bug.cgi?id=427938

For people who want --with-bdeps disabled for installation actions, and
--with-bdeps enabled for removal actions like --depclean, putting
--with-bdeps=n in EMERGE_DEFAULT_OPTS is going to do the wrong thing for
removal actions. So, we'll need to introduce some kind of flag that can
be used to disable the automatic --with-bdeps behavior. For example,
when can introduce --with-bdeps-auto=n as a means to disable the
automatic --with-bdeps behavior.
-- 
Thanks,
Zac