1 |
On Mon, 05/11/2007 в 10:03 -0800, Donnie Berkholz wrote: |
2 |
> On 14:23 Mon 05 Nov , Peter Volkov (pva) wrote: |
3 |
> > 1.1 net-firewall/ipset/ipset-2.3.0.20070828.ebuild |
4 |
> > |
5 |
> > src_defs() { |
6 |
> > # these are used in both of src_compile and src_install |
7 |
> > myconf="${myconf} PREFIX=" |
8 |
> > myconf="${myconf} LIBDIR=/lib" |
9 |
> |
10 |
> Should this be get_libdir()? |
11 |
|
12 |
Yes. Fixed. |
13 |
|
14 |
> Also seems like instead of having a function to do this, you could just |
15 |
> define the variables once in pkg_setup() or src_compile() as globals. |
16 |
|
17 |
Fixed too. |
18 |
|
19 |
> > pkg_config() { |
20 |
> > tar -jxf "${DISTDIR}"/iptables-${IPTVERINS}.tar.bz2 || \ |
21 |
> > die "Unable to unpack iptables-${IPTVERINS}.tar.bz2" |
22 |
> |
23 |
> Could you just call unpack() here? |
24 |
|
25 |
Ok. Done. |
26 |
|
27 |
BTW, is it possible to force portage to fetch sources if they do not |
28 |
exist in ${DISTDIR}? Also does there exist a better way to find iptables |
29 |
sources than find version without revision: |
30 |
|
31 |
IPTVERINS=`echo $(best_version net-firewall/iptables) | \ |
32 |
sed -n 's:^[^/]*/[[:alpha:]]*-\([0-9]\+\([.][0-9]\+\)*[a-z]\?\(_\(pre\|p\|beta\|alpha\|rc\)[0-9]*\)*\)\(-r[0-9]\+\)\?$:\1:p' |
33 |
|
34 |
and construct package name (iptables-${IPTVERINS}}.tar.bz2)? |
35 |
|
36 |
> > einfo "Enter path to your kernel sources, relative to ${ROOT}" |
37 |
> > echo -n "[/usr/src/linux]: " |
38 |
> > read K_DIR |
39 |
> > [ "${ROOT}${K_DIR}" == "/" ] && K_DIR=${ROOT}/usr/src/linux |
40 |
> |
41 |
> /usr/src/linux should be what's used here, and if they want to change |
42 |
> the kernel to affect, they should change where the symlink points |
43 |
> instead of having a secondary config mechanism. |
44 |
|
45 |
Agree, and fixed too. Thank you, very much, for review. |
46 |
|
47 |
-- |
48 |
Peter. |