1 |
В Чтв, 16/09/2010 в 16:24 +0200, Matti Bickel пишет: |
2 |
> +FOXVER=`get_version_component_range 1-2 ${FOX_PV}` |
3 |
|
4 |
It's better to prefer $() style over ``: |
5 |
http://mywiki.wooledge.org/BashFAQ/082 |
6 |
|
7 |
> if [ "${PN}" != fox ] ; then |
8 |
> FOX_COMPONENT="${FOX_COMPONENT:-${PN}}" |
9 |
> fi |
10 |
> |
11 |
> -if [ "${FOXVER}" != "1.0" ] && [ -z "${FOX_COMPONENT}" ] ; then |
12 |
> +if [ -z "${FOX_COMPONENT}" ] ; then |
13 |
> DOXYGEN_DEP="doc? ( app-doc/doxygen )" |
14 |
> fi |
15 |
|
16 |
It's better to use [[ ]] and avoid quotes since ebuilds are bash |
17 |
scripts. |
18 |
|
19 |
> - elibtoolize |
20 |
> + eautoreconf |
21 |
|
22 |
Hm, is this change necessary? |
23 |
|
24 |
> + if ( ! use doc ) && [ -d "${D}/usr/share/doc/${PF}/html" ] ; |
25 |
> then |
26 |
|
27 |
Subshell looks redundant here. |
28 |
|
29 |
> + epause |
30 |
|
31 |
It's better to avoid epause as it makes build slower at the same time |
32 |
it's most probable that nobody is looking on the screen at the moment[1] |
33 |
and portage will print all elog messages at the end of the build in any |
34 |
case. |
35 |
|
36 |
[1] while emerge output is one of those things one can observe for ages |
37 |
(like water, fire and others working) still it's possible no one |
38 |
thoughtfully staring at the screen... |
39 |
|
40 |
-- |
41 |
Peter. |