Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 123578 - mail-filter/clapf-0.3.23.ebuild (New package)
Summary: mail-filter/clapf-0.3.23.ebuild (New package)
Status: RESOLVED NEEDINFO
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Andrej Kacian (RETIRED)
URL: http://clapf.acts.hu/
Whiteboard:
Keywords: EBUILD
Depends on: 123576
Blocks:
  Show dependency tree
 
Reported: 2006-02-20 23:32 UTC by Anatoly Shipitsin
Modified: 2007-08-22 09:02 UTC (History)
1 user (show)

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


Attachments
clapf-0.3.23.ebuild (clapf-0.3.23.ebuild,2.46 KB, text/plain)
2006-02-20 23:34 UTC, Anatoly Shipitsin
Details
fix tinycdb depends (gentoo-tinycdb.patch,3.80 KB, patch)
2006-02-20 23:35 UTC, Anatoly Shipitsin
Details | Diff
change config file location (gentoo-config.patch,3.01 KB, patch)
2006-02-20 23:36 UTC, Anatoly Shipitsin
Details | Diff
clapf init script (clapf,519 bytes, text/plain)
2006-02-20 23:36 UTC, Anatoly Shipitsin
Details
confd file (clapf,64 bytes, text/plain)
2006-02-20 23:38 UTC, Anatoly Shipitsin
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Anatoly Shipitsin 2006-02-20 23:32:18 UTC
Clapf is a modular network filter for postfix. It includes a Bayesian antispam module, the blackhole feature and antivirus support (clamav, AVG Linux and avast!) preventing virus infection as well as a cgi and a command line interface to easily train the token database. 

I'm think this shuld working faster amavisd-new.
Comment 1 Anatoly Shipitsin 2006-02-20 23:34:32 UTC
Created attachment 80348 [details]
clapf-0.3.23.ebuild
Comment 2 Anatoly Shipitsin 2006-02-20 23:35:20 UTC
Created attachment 80349 [details, diff]
fix tinycdb depends
Comment 3 Anatoly Shipitsin 2006-02-20 23:36:32 UTC
Created attachment 80350 [details, diff]
change config file location
Comment 4 Anatoly Shipitsin 2006-02-20 23:36:50 UTC
Created attachment 80351 [details]
clapf init script
Comment 5 Anatoly Shipitsin 2006-02-20 23:38:11 UTC
Created attachment 80352 [details]
confd file

If i'm can add env variable TMP in init script, it's not need ;)
Comment 6 Andrej Kacian (RETIRED) gentoo-dev 2006-02-21 02:49:42 UTC
Let's give user the choice where does he want temporary files to be stored, as long as default value works.
Comment 7 Anatoly Shipitsin 2006-02-21 03:02:39 UTC
> Let's give user the choice where does he want temporary files to be stored, as
> long as default value works.
> 
Okey.
Comment 8 Andrej Kacian (RETIRED) gentoo-dev 2006-02-27 03:09:06 UTC
Ok, some QA points:

Saying "${PN}-${PV}" is the same as saying "${P}"

S=${WORKDIR}/${P} is the default value of ${S}, thus can be omitted

Instead of gentoo-config.patch, you could just use sed to tweak the config file to your needs - for example, see app-antivirus/clamav ebuild.

Do not use spaces to indent lines (pkg_preinst() and pkg_postinst()), use tab(s) instead.

It's better to fix configure.in file to add required flags to LDFLAGS and CPPFLAGS and send the patch upstream, instead of overriding portage's LDD/CPPFLAGS.

You probably forgot to say $(use_enable mysql blackhole) in econf call. (I assume "blackhole" is a mysql-dependant feature.)

Do not get discouraged by this criticism, it's a common QA practice. If you need help with any of these issues, just say so, I'll be glad to help.
Comment 9 Anatoly Shipitsin 2006-02-27 05:14:08 UTC
> Do not get discouraged by this criticism, it's a common QA practice. If you
> need help with any of these issues, just say so, I'll be glad to help.

Where i'm can read QA ?

Comment 10 Andrej Kacian (RETIRED) gentoo-dev 2006-02-27 07:15:47 UTC
There is currently a motion to have this kind of documentation in one central place. Meanwhile, try these URLs:

Official:
Development-related documentation at www.gentoo.org, most notably
http://www.gentoo.org/proj/en/devrel/handbook/hb-guide-common-mistakes.xml

Unofficial:
http://dev.gentoo.org/~plasmaroo/devmanual/
http://dev.gentoo.org/~ciaranm/docs/
Comment 11 Anatoly Shipitsin 2006-02-28 02:44:49 UTC
Thanks. I'm read this and recommit ebuild.

Comment 12 Andrej Kacian (RETIRED) gentoo-dev 2007-01-02 18:38:09 UTC
Anatoly, are you planning to submit a rewritten ebuild?
Comment 13 Andrej Kacian (RETIRED) gentoo-dev 2007-08-22 09:02:05 UTC
Closing due to inactivity.