[prev in list] [next in list] [prev in thread] [next in thread] 

List:       busybox
Subject:    Re: [PATCH 1/2] wget: add -o flag
From:       Kang-Che Sung <explorer09 () gmail ! com>
Date:       2018-12-27 0:22:49
Message-ID: CADDzAfN4eGqpFoDY-tpKwak-xdcZ58hxE+SjDbVNShvRnaROmA () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


On Wednesday, December 26, 2018, Martin Lewis <martin.lewis.x84@gmail.com>
wrote:
> [...]
> @@ -147,6 +148,7 @@
>  //usage:     "\n       -T SEC          Network read timeout is SEC
seconds"
>  //usage:       )
>  //usage:     "\n       -O FILE         Save to FILE ('-' for stdout)"
> +//usage:     "\n       -o FILE         Save output to FILE ('-' for
stdout)"

Am I the only one who think "output" is confusing for the "-o" option?

GNU wget manual already changed the name to "logfile" for what you are
implementing.

Speaking of, would you mind also change the "-O" help text to clarify it
refer to the "downloaded data"? Since both can be technically considered
wget "output" it helps to distinguish between the two.

[Attachment #5 (text/html)]

<br><br>On Wednesday, December 26, 2018, Martin Lewis &lt;<a \
href="mailto:martin.lewis.x84@gmail.com">martin.lewis.x84@gmail.com</a>&gt; \
wrote:<br>&gt; [...]<br>&gt; @@ -147,6 +148,7 @@<br>&gt;   //usage:        &quot;\n   \
-T SEC               Network read timeout is SEC seconds&quot;<br>&gt;   //usage:     \
)<br>&gt;   //usage:        &quot;\n           -O FILE              Save to FILE \
(&#39;-&#39; for stdout)&quot;<br>&gt; +//usage:        &quot;\n           -o FILE    \
Save output to FILE (&#39;-&#39; for stdout)&quot;<br><br>Am I the only one who think \
&quot;output&quot; is confusing for the &quot;-o&quot; option?<br><br>GNU wget manual \
already changed the name to &quot;logfile&quot; for what you are \
implementing.<br><br>Speaking of, would you mind also change the &quot;-O&quot; help \
text to clarify it refer to the &quot;downloaded data&quot;? Since both can be \
technically considered wget &quot;output&quot; it helps to distinguish between the \
two.



_______________________________________________
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox


[prev in list] [next in list] [prev in thread] [next in thread]