1 |
Thomas Cort wrote: |
2 |
> What is the proper quoting style for using epatch? In the tree there |
3 |
> are about 3 different styles... |
4 |
> |
5 |
> epatch ${FILESDIR}/some-fix.patch # used by 7326 ebuilds |
6 |
> epatch "${FILESDIR}"/some-fix.patch # used by 3092 ebuilds |
7 |
> epatch "${FILESDIR}/some-fix.patch" # used by 1434 ebuilds |
8 |
> |
9 |
2 and 3 are fine. |
10 |
> What is the proper quoting style for defining the S variable? In the |
11 |
> tree there are about 3 different styles... |
12 |
> |
13 |
> S=${WORKDIR}/${MY_P} # used by 5270 ebuilds |
14 |
> S="${WORKDIR}"/${MY_P} # used by 43 ebuilds |
15 |
> S="${WORKDIR}/${MY_P}" # used by 2259 ebuilds |
16 |
> |
17 |
ditto |
18 |
> What is the purpose of setting DEPEND and RDEPEND to "" if DEPEND and |
19 |
> RDEPEND are optional[1][2]? Isn't that just a waste of disk space / |
20 |
> bandwidth? DEPEND="virtual/libc" seems like a waste too as it is an |
21 |
> implicit system dependency[3], any reason for using it? |
22 |
> |
23 |
> DEPEND="" # used by 1479 ebuilds |
24 |
> RDEPEND="" # used by 884 ebuilds |
25 |
> DEPEND="virtual/libc" # used by 809 ebuilds |
26 |
> |
27 |
If the package don't have dependencies, then don't set *DEPEND. |
28 |
virtual/libc dependency is probably futile, unless the package is part |
29 |
of the system class or is a dependency of a package which is part of the |
30 |
system class. |