| 1 |
>>>>> On Tue, 20 Aug 2013, Micha³ Górny wrote: |
| 2 |
|
| 3 |
> 1. Name |
| 4 |
|
| 5 |
> - einstalldocs |
| 6 |
|
| 7 |
> -- probably the least confusing one, consistent with emake |
| 8 |
|
| 9 |
Go for it. |
| 10 |
|
| 11 |
> 2. Support for HTML_DOCS |
| 12 |
|
| 13 |
I've no strong opinion about this one. |
| 14 |
|
| 15 |
But if you are going to add it, then please support both array and |
| 16 |
whitespace separated list, so that it's consistent with DOCS. |
| 17 |
|
| 18 |
> 3. Support DOCS=() / DOCS='' to disable dodoc |
| 19 |
|
| 20 |
Yes, please. The EAPI 4 default_src_install is broken that it doesn't |
| 21 |
handle this case. |
| 22 |
|
| 23 |
> For example, hasufell has suggested checking ${DOCS// /} to support |
| 24 |
> DOCS=' ' as well. |
| 25 |
|
| 26 |
Looks like over-engineering to me. Are there any use cases for it? |
| 27 |
|
| 28 |
> 4. Globbing support in DOCS array |
| 29 |
> 5. Appending to DOCS |
| 30 |
|
| 31 |
Creeping featurism. Define an explicit src_install if you need any of |
| 32 |
these, and I'm sure it will improve readability of your ebuild. |
| 33 |
|
| 34 |
Ulrich |
Archives