Donnie Berkholz <[EMAIL PROTECTED]> wrote: > On 13:56 Mon 13 Oct , Petteri Räty wrote: > > Could you also send a diff next time. > > Or this time, even. +1 on that.
Here you are. It's attached. > One easy thing to do is move what comments exist to the eclass-manpages > format. I also included some eclass-manpage foo now, thanks for the hint. -- Regards, Matti Bickel Signed/Encrypted email preferred (key 4849EC6C)
--- gentoo-x86/eclass/fox.eclass 2008-10-12 14:31:36.000000000 +0200 +++ fox-proposed.eclass 2008-10-13 20:27:05.000000000 +0200 @@ -1,8 +1,12 @@ # Copyright 1999-2005 Gentoo Foundation # Distributed under the terms of the GNU General Public License v2 -# $Header: /var/cvsroot/gentoo-x86/eclass/fox.eclass,v 1.8 2008/10/12 12:31:36 mabi Exp $ +# $Header: /var/cvsroot/gentoo-x86/eclass/fox.eclass,v 1.7 2007/01/15 20:27:06 mabi Exp $ -# fox eclass +# @ECLASS: fox.eclass +# @MAINTAINER: [EMAIL PROTECTED] +# @BLURB: Common build and install functions for fox-related apps and library +# @DESCRIPTION: Used by the x11-libs/fox library and all applications that come +# with the upstream source tarball. # # This eclass allows building SLOT-able FOX Toolkit installations # (x11-libs/fox: headers, libs, and docs), which are by design @@ -19,26 +23,18 @@ # are API unstable; changes are made to the apps, and likely need to be # bumped together with the library. # -# Here are sample [R]DEPENDs for the fox apps -# fox versions that do not use this eclass are blocked in INCOMPAT_DEP below -# 1.0: '=x11-libs/fox-1.0*' -# 1.2: '=x11-libs/fox-1.2*' +# Some concepts borrowed from gst-plugins and gtk-sharp-component eclasses +# +# @EXAMPLE: Here are sample [R]DEPENDs for the fox apps # 1.4: '=x11-libs/fox-1.4*' # 1.5: '~x11-libs/fox-${PV}' # 1.6: '=x11-libs/fox-${FOXVER}*' -# -# Some concepts borrowed from gst-plugins and gtk-sharp-component eclasses - -inherit eutils libtool versionator +inherit autotools eutils libtool versionator FOX_PV="${FOX_PV:-${PV}}" -PVP=(${FOX_PV//[-\._]/ }) -FOXVER="${PVP[0]}.${PVP[1]}" - -if [ "${FOXVER}" != "1.0" ] ; then - FOXVER_SUFFIX="-${FOXVER}" -fi +FOXVER=$(get_version_component_range 1-2) +FOXVER_SUFFIX="-${FOXVER}" DESCRIPTION="C++ based Toolkit for developing Graphical User Interfaces easily and effectively" HOMEPAGE="http://www.fox-toolkit.org/" @@ -46,44 +42,28 @@ IUSE="debug doc profile" -# from fox-1.0 -FOX_APPS="adie calculator pathfinder" -# from fox-1.2+ -if [ "${FOXVER}" != "1.0" ] ; then - FOX_APPS="${FOX_APPS} shutterbug" - FOX_CHART="chart" -fi +# @ECLASS-VARIABLE: FOX_APPS +# @DESCRIPTION: all applications that come with the fox toolkit source +FOX_APPS="adie calculator pathfinder shutterbug chart" if [ "${PN}" != fox ] ; then FOX_COMPONENT="${FOX_COMPONENT:-${PN}}" fi -if [ "${FOXVER}" != "1.0" ] && [ -z "${FOX_COMPONENT}" ] ; then - DOXYGEN_DEP="doc? ( app-doc/doxygen )" -fi - if [ "${PN}" != reswrap ] ; then RESWRAP_DEP="dev-util/reswrap" fi -# These versions are not compatible with new fox layout -# and will cause collissions - we need to block them -INCOMPAT_DEP="!<x11-libs/fox-1.0.53 - !=x11-libs/fox-1.2.4 - !~x11-libs/fox-1.2.6 - !=x11-libs/fox-1.4.11" - -DEPEND="${INCOMPAT_DEP} - ${DOXYGEN_DEP} - ${RESWRAP_DEP} - =sys-devel/automake-1.4* +DEPEND="${RESWRAP_DEP} + doc? ( app-doc/doxygen ) + >=sys-devel/automake-1.4 >=sys-apps/sed-4" S="${WORKDIR}/fox-${FOX_PV}" fox_src_unpack() { unpack ${A} - cd ${S} + cd "${S}" ebegin "Fixing configure" @@ -103,14 +83,14 @@ done # use the installed reswrap for everything else - for d in ${FOX_APPS} ${FOX_CHART} tests ; do + for d in ${FOX_APPS} tests ; do sed -i -e 's:$(top_builddir)/utils/reswrap:reswrap:' \ ${d}/Makefile.am || die "sed ${d}/Makefile.am error" done # use the installed headers and library for apps for d in ${FOX_APPS} ; do - if version_is_at_least "1.6.34" ${PV} ; then + if version_is_at_least "1.6.34" ${PV}; then sed -i \ -e "s:-I\$(top_srcdir)/include -I\$(top_builddir)/include:-I\$(includedir)/fox${FOXVER_SUFFIX}:" \ -e 's:$(top_builddir)/src/libFOX:-lFOX:' \ @@ -124,19 +104,13 @@ ${d}/Makefile.am || die "sed ${d}/Makefile.am error" fi done - - # Upstream often has trouble with version number transitions - if [ "${FOXVER}" == "1.5" ] ; then - sed -i -e 's:1.4:1.5:g' chart/Makefile.am - fi - eend ebegin "Running automake" - automake-1.4 -a -c || die "automake error" + eautomake || die "automake error" eend - elibtoolize + #elibtoolize } fox_src_compile() { @@ -150,21 +124,21 @@ $(use_with profile profiling) \ || die "configure error" - cd ${S}/${FOX_COMPONENT} + cd "${S}/${FOX_COMPONENT}" emake || die "compile error" # build class reference docs (FOXVER >= 1.2) - if use doc && [ "${FOXVER}" != "1.0" ] && [ -z "${FOX_COMPONENT}" ] ; then - cd ${S}/doc - make docs || die "doxygen error" + if use doc && [ -z "${FOX_COMPONENT}" ] ; then + cd "${S}/doc" + emake docs || die "doxygen error" fi } fox_src_install () { - cd ${S}/${FOX_COMPONENT} + cd "${S}/${FOX_COMPONENT}" - make install \ - DESTDIR=${D} \ + emake install \ + DESTDIR="${D}" \ htmldir=/usr/share/doc/${PF}/html \ artdir=/usr/share/doc/${PF}/html/art \ screenshotsdir=/usr/share/doc/${PF}/html/screenshots \ @@ -195,41 +169,37 @@ done # remove documentation if USE=-doc - if ( ! use doc ) && [ -d ${D}/usr/share/doc/${PF}/html ] ; then - rm -fr ${D}/usr/share/doc/${PF}/html + if ( ! use doc ) && [ -d "${D}"/usr/share/doc/${PF}/html ] ; then + rm -fr "${D}"/usr/share/doc/${PF}/html fi # install class reference docs (FOXVER >= 1.2) if USE=doc - if use doc && [ "${FOXVER}" != "1.0" ] && [ -z "${FOX_COMPONENT}" ] ; then - dohtml -r ${S}/doc/ref + if use doc && [ -z "${FOX_COMPONENT}" ] ; then + dohtml -r "${S}"/doc/ref fi # slot fox-config where present (FOXVER >= 1.2) - if [ -f ${D}/usr/bin/fox-config ] ; then - mv ${D}/usr/bin/fox-config ${D}/usr/bin/fox-${FOXVER}-config + if [ -f "${D}"/usr/bin/fox-config ] ; then + mv "${D}"/usr/bin/fox-config "${D}"/usr/bin/fox-${FOXVER}-config fi } fox_pkg_postinst() { if [ -z "${FOX_COMPONENT}" ] ; then + elog "Multiple versions of the FOX Toolkit library may now be installed" + elog "in parallel SLOTs on the same system." + elog + elog "The reswrap utility and the applications included in the FOX Toolkit" + elog "(adie, calculator, pathfinder, shutterbug) are now available as" + elog "separate ebuilds." echo - einfo "Multiple versions of the FOX Toolkit library may now be installed" - einfo "in parallel SLOTs on the same system." - einfo - einfo "The reswrap utility and the applications included in the FOX Toolkit" - einfo "(adie, calculator, pathfinder, shutterbug) are now available as" - einfo "separate ebuilds." - echo - if [ "${FOXVER}" != "1.0" ] ; then - einfo "The fox-config script has been installed as fox-${FOXVER}-config." - einfo "The fox-wrapper package is used to direct calls to fox-config" - einfo "to the correct versioned script, based on the WANT_FOX variable." - einfo "For example:" - einfo - einfo " WANT_FOX=\"${FOXVER}\" fox-config <options>" - einfo - epause - fi + elog "The fox-config script has been installed as fox-${FOXVER}-config." + elog "The fox-wrapper package is used to direct calls to fox-config" + elog "to the correct versioned script, based on the WANT_FOX variable." + elog "For example:" + elog + elog " WANT_FOX=\"${FOXVER}\" fox-config <options>" + elog fi }
pgpsTmDVn9XHM.pgp
Description: PGP signature