[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: debootstrap: manpage neglects complete path for



Hideki Yamane wrote:
>> If I was doing a pedantic review of every bit of debootstrap's
>> documentation and output I'd suggest replacing "http(s)" with
>> "HTTP(S)" throughout, but there's no point doing that just here.
> 
>  I can find only one "via http" word, as below. Is it okay?

Well, I was saying "don't bother"!  But it looks easier than I was
assuming.

> diff --git a/debootstrap.8 b/debootstrap.8
> index e802003..87e2ae1 100644
> --- a/debootstrap.8
> +++ b/debootstrap.8
> @@ -137,7 +137,7 @@ Don't delete the /debootstrap directory in the target after completing the
>  installation.
>  .IP
>  .IP "\fB\-\-unpack\-tarball=FILE\fP"
> -Acquire .debs from tarball FILE instead of downloading via http.
> +Acquire .debs from tarball FILE instead of downloading via HTTP(S).

Yes, this is an improvement if it might use HTTPS, though not if it
will always use HTTP.

But you missed the equivalent text in the debootstrap usage message:

      --unpack-tarball=T     acquire .debs from a tarball instead of http
                                                                     ^^^^
(Elsewhere in the usage output it already uses uppercase.)

I notice one other mention of wrong-case "https" in the debootstrap
sources (apart from changelog mentions): "functions" has a "keyring()"
function with this output:

         info KEYRING "Keyring file not available at %s; switching to https mirror %s" "$1" "$DEF_HTTPS_MIRROR"
                                                                      ^^^^^
-- 
JBR	with qualifications in linguistics, experience as a Debian
	sysadmin, and probably no clue about this particular package


Reply to: