Created attachment 290621 [details, diff] less-444-regcmp-libs.patch We apply a patch to less to avoid its configure linking needlessly against libgen and libintl. Instead of this patch which requires an autoreconf, we can also export two ac_cv_* variables to achieve the same effect, conditionally for Solaris: # Solaris includes regcmp in it's libc, but configure wrongly checks -lgen # and -lintl, which obviously both "provide" regcmp. if [[ ${CHOST} == *-solaris* ]] ; then export ac_cv_lib_gen_regcmp=no export ac_cv_lib_intl_regcmp=no fi regcmp-libs patch is attached to this comment. @@ -2,7 +2,7 @@ # Distributed under the terms of the GNU General Public License v2 # $Header: /var/cvsroot/gentoo-x86/sys-apps/less/less-444.ebuild,v 1.1 2011/06/09 18:37:30 vapier Exp $ -inherit eutils +inherit eutils autotools DESCRIPTION="Excellent text file viewer" HOMEPAGE="http://www.greenwoodsoftware.com/less/" @@ -21,6 +21,8 @@ cd "${S}" cp "${DISTDIR}"/code2color "${S}"/ epatch "${FILESDIR}"/code2color.patch + epatch "${FILESDIR}"/${P}-regcmp-libs.patch + eautoreconf } yesno() { use $1 && echo yes || echo no ; } Please review and tell your preference for either the patch as sent upstream, or the workaround which doesn't need eautoreconf.
configure patch is fine. upstream tends to be semi-responsive, so it should def be sent there as well.
I already did. Got no response yet, but will update the patch header as soon as I get response.
patch is no longer necessary/upstreamed with 445
