Re: [gentoo-dev] [PATCH 1/6] eapi7-ver.eclass: Explicitly indicate that EAPI 7+ includes it

2018-03-06 Thread Michael Orlitzky
On 03/06/2018 12:25 PM, Michał Górny wrote:
> ---
>  eclass/eapi7-ver.eclass | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/eclass/eapi7-ver.eclass b/eclass/eapi7-ver.eclass
> index 7eb070c68171..6117124a90a5 100644
> --- a/eclass/eapi7-ver.eclass
> +++ b/eclass/eapi7-ver.eclass
> @@ -63,7 +63,7 @@ case ${EAPI:-0} in
>   6)
>   ;;
>   *)
> - die "${ECLASS}: EAPI=${EAPI} unknown";;
> + die "${ECLASS}: EAPI=${EAPI} includes all functions from this 
> eclass";;
>  esac
>  

That error message might be a lie; think exheres-0. I think you can put
the "6" case first, and then change the "0|1|2|3|4|5" case to "*" to
have all others die with "not supported".




[gentoo-dev] [PATCH 1/6] eapi7-ver.eclass: Explicitly indicate that EAPI 7+ includes it

2018-03-06 Thread Michał Górny
---
 eclass/eapi7-ver.eclass | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/eclass/eapi7-ver.eclass b/eclass/eapi7-ver.eclass
index 7eb070c68171..6117124a90a5 100644
--- a/eclass/eapi7-ver.eclass
+++ b/eclass/eapi7-ver.eclass
@@ -63,7 +63,7 @@ case ${EAPI:-0} in
6)
;;
*)
-   die "${ECLASS}: EAPI=${EAPI} unknown";;
+   die "${ECLASS}: EAPI=${EAPI} includes all functions from this 
eclass";;
 esac
 
 # @FUNCTION: _ver_parse_range
-- 
2.16.2