Re: [gentoo-portage-dev] [PATCH] repoman: check for ebuilds not using slot operator against multi-slot packages.

[Brian Dolbec]

On Mon,  4 Aug 2014 11:11:25 +0200
Michał Górny <mgorny@g.o> wrote:

> The idea if that a particular dependency atom matches more than one
> slot of a package, you are supposed to either use := or :* operator
> (or a specific :slot dependency), whichever is appropriate.
>
> This will help catching mistakes (when packages become slotted) and
> make cross-slot behavior clear (it is undefined with no slot
> operator). I will estimate the amount of new warnings later.
> ---
>  bin/repoman | 28 +++++++++++++++++++++++-----
>  1 file changed, 23 insertions(+), 5 deletions(-)
>
> diff --git a/bin/repoman b/bin/repoman
> index 71fc7f0..b169393 100755
> --- a/bin/repoman
> +++ b/bin/repoman
> @@ -78,7 +78,8 @@ from portage.output import ConsoleStyleFile,
> StyleWriter from portage.util import writemsg_level
>  from portage.util._argparse import ArgumentParser
>  from portage.package.ebuild.digestgen import digestgen
> -from portage.eapi import eapi_has_iuse_defaults, eapi_has_required_use
> +from portage.eapi import eapi_has_iuse_defaults, eapi_has_required_use, \
> +	eapi_has_slot_operator
>

+from portage.eapi import (eapi_has_iuse_defaults, eapi_has_required_use,
+	eapi_has_slot_operator)

Please use brackets for long imports instead of using line continuation backslashes
-- 
Brian Dolbec <dolsen>