Skip to content

Commit

Permalink
games-fps/chocolate-doom: [QA] unconditionally install completion files
Browse files Browse the repository at this point in the history
QA policy [0] says that we don't conditionalise installation of
small files. It's a wasteful rebuild and inconsistent across packages
for when users desire completions to be available.

[0] https://projects.gentoo.org/qa/policy-guide/installed-files.html#pg0301
Signed-off-by: Sam James <[email protected]>
  • Loading branch information
thesamesam committed Jul 31, 2021
1 parent be8b011 commit b04be91
Showing 1 changed file with 6 additions and 5 deletions.
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
# Copyright 1999-2020 Gentoo Authors
# Copyright 1999-2021 Gentoo Authors
# Distributed under the terms of the GNU General Public License v2

EAPI=7

PYTHON_COMPAT=( python3_{7,8,9} )
PYTHON_COMPAT=( python3_{8,9} )

inherit autotools prefix python-any-r1 xdg

Expand All @@ -15,7 +15,7 @@ SRC_URI="https://github.com/${PN}/${PN}/archive/${P}.tar.gz
LICENSE="BSD GPL-2+"
SLOT="0"
KEYWORDS="~amd64 ~x86"
IUSE="bash-completion doc libsamplerate +midi png vorbis"
IUSE="doc libsamplerate +midi png vorbis"

DEPEND="
media-libs/libsdl2[video]
Expand All @@ -24,8 +24,9 @@ DEPEND="
libsamplerate? ( media-libs/libsamplerate )
png? ( media-libs/libpng:= )"
RDEPEND="${DEPEND}"
# ${PYTHON_DEPS} for bash-completion
BDEPEND="
bash-completion? ( ${PYTHON_DEPS} )
${PYTHON_DEPS}
doc? ( ${PYTHON_DEPS} )"

S="${WORKDIR}/${PN}-${P}"
Expand Down Expand Up @@ -69,7 +70,7 @@ src_prepare() {

src_configure() {
econf \
$(use_enable bash-completion) \
--enable-bash-completion \
$(use_enable doc) \
--disable-fonts \
--disable-icons \
Expand Down

0 comments on commit b04be91

Please sign in to comment.