1 |
>>>>> On Fri, 8 Jun 2018, Michał Górny wrote: |
2 |
|
3 |
> My previous fix to insopts turns out twice wrong: firstly, because I did |
4 |
> not really type what I meant, and secondly because package managers have |
5 |
> inconsistent behavior there. Portage uses the default mode (specified |
6 |
> by PMS) when insopts was called with no arguments or arguments forming |
7 |
> an empty string; PkgCore and Paludis literally pass empty argument list |
8 |
> to install. Mark the behavior as undefined to match that. |
9 |
> --- |
10 |
> pkg-mgr-commands.tex | 5 ++--- |
11 |
> 1 file changed, 2 insertions(+), 3 deletions(-) |
12 |
|
13 |
> diff --git a/pkg-mgr-commands.tex b/pkg-mgr-commands.tex |
14 |
> index 01c0c6d..031638a 100644 |
15 |
> --- a/pkg-mgr-commands.tex |
16 |
> +++ b/pkg-mgr-commands.tex |
17 |
> @@ -725,9 +725,8 @@ function has returned. |
18 |
|
19 |
> \item[docinto] As \t{into}, for install subdirectory of \t{dodoc} et al. |
20 |
> |
21 |
> -\item[insopts] Takes zero or more arguments, and sets the options passed by \t{doins} et al.\ |
22 |
> - to the \t{install} command to them. When called with no arguments, resets the option list |
23 |
> - to empty. |
24 |
> +\item[insopts] Takes one or more arguments, and sets the options passed by \t{doins} et al.\ |
25 |
> + to the \t{install} command to them. Behavior upon encountering empty arguments is undefined. |
26 |
> |
27 |
> \item[diropts] As \t{insopts}, for \t{dodir} et al. |
28 |
|
29 |
LGTM, except for s/Behavior/Behaviour/ because we use the English |
30 |
spelling throughout (and already fixed my local repo). |
31 |
|
32 |
Ulrich |