Gentoo Archives: gentoo-dev

From: Ulrich Mueller <ulm@g.o>
To: Matt Turner <mattst88@g.o>
Cc: gentoo-dev@l.g.o
Subject: Re: [gentoo-dev] [PATCH] gnome2-utils: Drop EAPI < 5 support
Date: Sun, 06 Dec 2020 11:41:19
Message-Id: uh7ozupps@gentoo.org
In Reply to: [gentoo-dev] [PATCH] gnome2-utils: Drop EAPI < 5 support by Matt Turner
1 >>>>> On Sun, 06 Dec 2020, Matt Turner wrote:
2
3 > -[[ ${EAPI:-0} == [012345] ]] && inherit multilib
4 > +[[ ${EAPI:-0} == [5] ]] && inherit multilib
5
6 Useless brackets. Also you could drop the :-0 here.
7
8 > case "${EAPI:-0}" in
9
10 No quotes needed here.
11
12 > - 0|1|2|3|4|5|6|7) ;;
13 > - *) die "EAPI=${EAPI} is not supported" ;;
14 > + 5|6|7) ;;
15 > + 0|1|2|3|4) die "EAPI=${EAPI} is not supported" ;;
16 > esac
17
18 This should have a * instead of an explicit list of unknown EAPIs.
19
20 > case ${EAPI:-0} in
21 > -0|1|2|3|4|5|6)
22 > +5|6)
23
24 See above, :-0 could be dropped.
25
26 Ulrich

Attachments

File name MIME type
signature.asc application/pgp-signature