Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 39019 - flabels eBuild (tetex)
Summary: flabels eBuild (tetex)
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High enhancement
Assignee: Text-Markup Team (OBSOLETE)
URL:
Whiteboard:
Keywords: EBUILD
Depends on:
Blocks:
 
Reported: 2004-01-22 03:50 UTC by Andreas Volz
Modified: 2004-11-07 06:10 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Andreas Volz 2004-01-22 03:50:33 UTC
# Copyright 1999-2004 Gentoo Technologies, Inc.
# Distributed under the terms of the GNU General Public License v2
# $Header: $

DESCRIPTION=" Macros for typesetting pretty labels (optionally colored) for the back of files or binders."
HOMEPAGE="http://www.ctan.org/tex-archive/help/Catalogue/entries/flabels.html"
SRC_URI="ftp://ftp.dante.de/tex-archive/macros/latex/contrib/flabels.tar.gz"
LICENSE="LaTeX Project Public License"
SLOT="0"
KEYWORDS="~x86"
IUSE=""
DEPEND="tetex"

S=${WORKDIR}/flabels

src_compile() {
    cd ${S}
    latex flabels.ins
    ./makedoc
}

src_install() {
    mkdir -p ${D}/usr/share/texmf/tex/latex/flabels
    cp ${S}/flabels.sty ${D}/usr/share/texmf/tex/latex/flabels
}

pkg_postinst() {
    texhash
}
Comment 1 Mamoru KOMACHI (RETIRED) gentoo-dev 2004-02-28 14:44:28 UTC
It would be greatly appreciated if you add comment or information
about the package. Also, please make a separate file (not just
cut and paste your ebuild into the form) and attach it. I'll
give you some comments about it, so would you rewrite it accordingly
and attach it?

* LICENSE must be one of /usr/portage/licenses. In this case,
  LPPL-1.2 is the right value for LICENSE.
* DEPEND should be what we call "DEPEND atom" (see `man 5 ebuild`).
  That is, you cannot write a name of a package directly. 
  You should at least include category along with the package name.
  So it should look like DEPEND="virtual/tetex". (virtual/tetex
  is a virtual package for app-text/tetex, app-text/ptex and
  app-text/cstetex)
* `cd ${S}` in src_compile() is not necessary. The initial working
  directory is automatically set to ${S}. See man 5 ebuild for
  working directory of each function.
* It would be better to use

insinto /usr/share/texmf/tex/latex/${PN}
doins flabels.sty

  instead of writing mkdir -p and cp the filename (You need to write
  ugly ${D} if you choose to not use insinto/doins). Also, if you
  don't install flabels.dvi in src_install(), what's the point of
  creating it in src_compile() by `./makedoc` ?
* It looks nice if you add pkg_postrm() { texhash } as well as
  pkg_postinst(). 

If in doubt, feel free to ask.
Comment 2 Mamoru KOMACHI (RETIRED) gentoo-dev 2004-11-07 06:10:26 UTC
ok, I rewrote it with latex-package.eclass and committed.
Please remember to check what I wrote in comment #2
next time you submit an ebuild.
Thanks for your contribution.