Skip to content

Commit

Permalink
dev-games/libnw: remove highly confusing sed to "respect CC"
Browse files Browse the repository at this point in the history
But it's a generated automake Makefile.in, and the sed is removing every
line containing "CC = @cc@", so it's puzzling that anyone ever thought
to do this.

It was originally added in gentoo-historical-2:

  29 Jan 2009; Michael Sterrett <[email protected]> libnw-1.30.02.ebuild:
  EAPI=2; respect CC

And 15 years later it's the same (2003) tarball and the sed is decidedly
not needed (nor ever was).

Remove this bloaty junk.

Signed-off-by: Eli Schwartz <[email protected]>
Signed-off-by: Sam James <[email protected]>
  • Loading branch information
eli-schwartz authored and thesamesam committed May 6, 2024
1 parent f02e824 commit fd511ce
Showing 1 changed file with 1 addition and 11 deletions.
12 changes: 1 addition & 11 deletions dev-games/libnw/libnw-1.30.02.ebuild
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@

EAPI=7

inherit autotools toolchain-funcs
inherit autotools

DESCRIPTION="Tools and libraries for NWN file manipulation"
HOMEPAGE="https://sourceforge.net/projects/openknights"
Expand All @@ -23,19 +23,9 @@ DOCS=( AUTHORS ChangeLog NEWS README README.tech TODO )
src_prepare() {
default
eautoreconf

local f
while IFS="" read -d $'\0' -r f; do
einfo "Removing hardcoded CC/CXX from ${f}"
sed -i \
-e '/^CC =/d' \
-e '/^CXX =/d' \
"${f}" || die
done < <(find "${S}" -name Makefile.in -type f -print0)
}

src_configure() {
tc-export CC CXX
econf --disable-static
}

Expand Down

0 comments on commit fd511ce

Please sign in to comment.