Summary: | Ebuild for DTC - ISP Web Control Panel | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Mark <Mark> |
Component: | New packages | Assignee: | Default Assignee for New Packages <maintainer-wanted> |
Status: | RESOLVED WONTFIX | ||
Severity: | enhancement | CC: | jakub |
Priority: | High | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: | draft - ebuild - wip |
Description
Mark
2005-07-09 04:42:40 UTC
Created attachment 63009 [details]
draft - ebuild - wip
1/ There are tarballs and CVS snapshots for this app, please use them in the ebuild. See ftp://ftp.gplhost.com/pub/dtc/ and fix the empty SRC_URI. We try to avoid live CVS ebuilds as much as possible. 2/ This looks like a webapp, so should use webapp eclass. 3/ Any keywords missing. :=) Put ~x86 or other arch which you are testing this on there. 4/ IUSE has apache2 and php in there, but it's unused anywhere else. See other ebuilds in portage how to handle apache-1 vs. apache-2 dependencies; php should be removed from IUSE, it's most probably not optional dependency and does not do anything useful in the ebuild. 5/ The ebuild depends on dev-php/mod_php, this is wrong. It needs to depend on virtual/httpd-php. 6/ Generally check DEPEND vs. RDEPEND and fix as needed. 7/ src_install() is plain wrong, if it depends on some perl module, it needs to depend on an ebuild. If it's not in portage, you need to create an ebuild for that perl module as well. 8/ As for the rest, the needs to be adopted to webapp eclass, as already said above. etc. etc. Some docs which could help you: http://www.gentoo.org/proj/en/devrel/handbook/handbook.xml?part=2&chap=3 http://dev.gentoo.org/~plasmaroo/devmanual/ Meanwhile, closing WONTFIX. Please reopen after you submit an improved ebuild. Thanks. |