1 |
On 02/26/10 22:01, Samuli Suominen wrote: |
2 |
> On 02/26/2010 10:49 PM, Patrick Lauer (patrick) wrote: |
3 |
>> patrick 10/02/26 20:49:19 |
4 |
>> |
5 |
>> Modified: ChangeLog |
6 |
>> Added: samhain-2.6.2.ebuild |
7 |
>> Log: |
8 |
>> Bump |
9 |
>> (Portage version: 2.2_rc63/cvs/Linux x86_64) |
10 |
>> Index: samhain-2.6.2.ebuild |
11 |
>> =================================================================== |
12 |
>> # Copyright 1999-2010 Gentoo Foundation |
13 |
>> # Distributed under the terms of the GNU General Public License v2 |
14 |
>> # $Header: /var/cvsroot/gentoo-x86/app-forensics/samhain/samhain-2.6.2.ebuild,v 1.1 2010/02/26 20:49:18 patrick Exp $ |
15 |
>> |
16 |
>> KEYWORDS="~amd64 ~x86" |
17 |
>> DESCRIPTION="Advanced file integrity and intrusion detection tool." |
18 |
>> HOMEPAGE="http://la-samhna.de/samhain/" |
19 |
>> SRC_URI="http://la-samhna.de/archive/samhain_signed-${PV}.tar.gz" |
20 |
>> LICENSE="GPL-2" |
21 |
>> SLOT="0" |
22 |
>> IUSE="crypt debug login-watch mounts-check mysql netclient netserver postgres prelude static suidcheck userfiles xml" |
23 |
>> |
24 |
>> RESTRICT="strip" |
25 |
> |
26 |
> Why restricting strip on source based package? It's wrong, at least if |
27 |
> it's to supress QA warnings from Portage. It only hides them. |
28 |
|
29 |
If that's your largest criticism ... ;) |
30 |
This ebuild is in a rather sorry state, and no one has found the |
31 |
motivation to just package.mask it. |
32 |
|
33 |
Of course you're right, that RESTRICT shouldn't be there (and I have no |
34 |
idea how that came to happen) - ChangeLog mentions: |
35 |
|
36 |
02 Jul 2007; Piotr Jaroszy<C5><84>ski <peper@g.o> |
37 |
samhain-2.1.3.ebuild, |
38 |
samhain-2.2.0.ebuild: |
39 |
(QA) RESTRICT clean up. |
40 |
|
41 |
where this happened: |
42 |
-RESTRICT="nostrip" |
43 |
+RESTRICT="strip" |
44 |
|
45 |
Impressive. And will be fixed very soonish. I hope. Sigh :) |
46 |
|
47 |
>> make || die "compile failed" |
48 |
>> } |
49 |
>> |
50 |
>> src_install() { |
51 |
>> make DESTDIR="${D}" install || die "make install failed" |
52 |
> |
53 |
> Use emake instead of make. |
54 |
> |