1 |
On Sun, 2019-10-20 at 20:57 +0200, Ulrich Mueller wrote: |
2 |
> > > > > > On Sun, 20 Oct 2019, David Seifert wrote: |
3 |
> > > > - [[ -n ${DOCS} ]] && { dodoc ${DOCS} || die "docs |
4 |
> > > > installation |
5 |
> > > > failed" ; } # TODO old EAPI cleanup |
6 |
> > > > + einstalldocs |
7 |
> > > > |
8 |
> > > > # install common docs |
9 |
> > > > - for commondoc in COPYRIGHT README{,.md,.txt} NEWS |
10 |
> > > > AUTHORS BUGS |
11 |
> > > > ChangeLog FONTLOG.txt; do |
12 |
> > > > + for commondoc in COPYRIGHT FONTLOG.txt; do |
13 |
> > > > [[ -s ${commondoc} ]] && dodoc ${commondoc} |
14 |
> > > > done |
15 |
> > > This changes the set of installed files, if the DOCS variable is |
16 |
> > > defined. Is that intentional? |
17 |
> > You mean if it's *not* defined? |
18 |
> |
19 |
> No, if it *is* defined. For example, if an ebuild defines DOCS but |
20 |
> doesn't include README.txt in the list, then that file was previously |
21 |
> being installed, but isn't any longer with the eclass change. |
22 |
|
23 |
Now I got it. Yes, this is fine because most fonts actually override |
24 |
DOCS and include README in there. |
25 |
|
26 |
> |
27 |
> > > This doesn't change any functionality, but it adds an overlong |
28 |
> > > line |
29 |
> > > for no good reason. |
30 |
> > The idea was to avoid if statements if you can use the more |
31 |
> > succinct |
32 |
> > form. |
33 |
> |
34 |
> Then at least wrap the long line. Still, I doubt that it will improve |
35 |
> readability, as compared to the original if statement. |
36 |
|
37 |
I'll fix it before pushing. |
38 |
|
39 |
> |
40 |
> Ulrich |