From: | Nirbheek Chauhan <nirbheek.chauhan@×××××.com> | ||
---|---|---|---|
To: | gentoo-dev@l.g.o, patrick@g.o | ||
Subject: | Re: [gentoo-dev] Re: gentoo-x86 commit in app-forensics/memdump: memdump-1.0.1.ebuild ChangeLog | ||
Date: | Mon, 05 Jan 2009 08:25:20 | ||
Message-Id: | 8b4c83ad0901050025k2356e1e4nbf40ea3fe86b7a57@mail.gmail.com | ||
In Reply to: | [gentoo-dev] Re: gentoo-x86 commit in app-forensics/memdump: memdump-1.0.1.ebuild ChangeLog by Torsten Veller |
1 | On Mon, Jan 5, 2009 at 5:37 AM, Torsten Veller <ml-en@××××××.net> wrote: |
2 | > * "Patrick Lauer (patrick)" <patrick@g.o>: |
3 | >> file : http://sources.gentoo.org/viewcvs.py/gentoo-x86/app-forensics/memdump/memdump-1.0.1.ebuild?rev=1.1&view=markup |
4 | >> plain: http://sources.gentoo.org/viewcvs.py/gentoo-x86/app-forensics/memdump/memdump-1.0.1.ebuild?rev=1.1&content-type=text/plain |
5 | >> |
6 | >> Index: memdump-1.0.1.ebuild |
7 | >> =================================================================== |
8 | >> # Copyright 1999-2009 Gentoo Foundation |
9 | >> # Distributed under the terms of the GNU General Public License v2 |
10 | >> # $Header: /var/cvsroot/gentoo-x86/app-forensics/memdump/memdump-1.0.1.ebuild,v 1.1 2009/01/04 23:45:43 patrick Exp $ |
11 | >> |
12 | >> DESCRIPTION="Simple memory dumper for UNIX-Like systems" |
13 | >> HOMEPAGE="http://www.porcupine.org/forensics" |
14 | >> SRC_URI="http://www.porcupine.org/forensics/${PN}-1.01.tar.gz" |
15 | > ~~~~ |
16 | |
17 | Missing MY_PV I presume? |
18 | |
19 | >> LICENSE="IBM" |
20 | >> SLOT="0" |
21 | >> KEYWORDS="~amd64 ~ppc ~x86" |
22 | >> DEPEND="sys-apps/sed |
23 | >> sys-apps/grep" |
24 | >> RDEPEND="virtual/libc" |
25 | > ^^^^^^^^^^^^^^^^^^^^^^ |
26 | |
27 | Useless deps which are already in @system? |
28 | |
29 | > |
30 | >> IUSE="" |
31 | >> |
32 | >> |
33 | >> S=${WORKDIR}/${PN}-1.01 |
34 | > ~~~~ |
35 | |
36 | Quotes and ${MY_PV} missing? |
37 | |
38 | >> src_compile() { |
39 | >> cd ${S}/memdump-1.01 |
40 | > " " ~~~~ |
41 | |
42 | Quotes, and ${PN}-${MY_PV} ? |
43 | Although, should this even be required since ${S} is already set correctly? |
44 | Does the tarball have a folder inside a folder? |
45 | |
46 | >> einfo "testing" |
47 | >> if [ "`./memdump -s 344 | wc -c`" = "344" ]; |
48 | >> then |
49 | >> einfo "passed test" |
50 | >> else |
51 | >> die "failed test" |
52 | >> fi |
53 | |
54 | Here, why not use |
55 | |
56 | einfo "testing" |
57 | [ "`./memdump -s 344 | wc -c`" = "344" ] || die "failed test" |
58 | einfo "passed test" |
59 | |
60 | |
61 | -- |
62 | ~Nirbheek Chauhan |
Subject | Author |
---|---|
Re: [gentoo-dev] Re: gentoo-x86 commit in app-forensics/memdump: memdump-1.0.1.ebuild ChangeLog | Peter Volkov <pva@g.o> |