Re: [cdrom-detect] Change template proposal

2019-12-10 Thread Justin B Rye
Holger Wansing wrote:
> Summarizing your comments into one patch, would this be ok?
> 
> diff --git a/debian/cdrom-detect.templates b/debian/cdrom-detect.templates
> index 2058b0b..ba50e89 100644
> --- a/debian/cdrom-detect.templates
> +++ b/debian/cdrom-detect.templates
> @@ -55,9 +55,10 @@ _Description: Device file for accessing the installation 
> media:
>   the device file that should be used. Non-standard CD-ROM drives use
>   non-standard device files (such as /dev/mcdx).
>   .
> - You may switch to the shell on the second terminal (ALT+F2) to check the
> + You may want switch to the shell on the second terminal (CTRL+ALT+F2 in the

You've dropped a word - "You may want to switch..."

> + graphical installer, ALT+F2 for the text-based variant) to check the
>   available devices in /dev with "ls /dev". You can return to this screen
> - by pressing ALT+F1.
> + by pressing CTRL+ALT+F5 or ALT+F1 respectively.

I wasn't even going to bother with the "respectively" - all it means
is "and I haven't maliciously sorted this list into a different order
from the last one", and I take that as a default assumption.  But it
doesn't hurt to include it.
-- 
JBR with qualifications in linguistics, experience as a Debian
sysadmin, and probably no clue about this particular package



Re: [cdrom-detect] Change template proposal

2019-12-10 Thread Holger Wansing
Hi,

Justin B Rye  wrote:
> > "You may want to switch to the shell on the second terminal (CLTR+ALT+F2 in 
> > the "
> > "graphical installer, ALT+F2 for the text-based variant) to check the "
> > "available devices in /dev with \"ls /dev\". You can return to this screen "
> > "by pressing CLTR+ALT+F5 respective ALT+F1."
>^^
> > 
> > 
> > Comments?
> 
> "Respective" is never ever a conjunction in English.  Just say "or".
> 
> And it's CTRL, not CLTR.



john doe  wrote:
> > "You may want to switch to the shell on the second terminal (CLTR+ALT+F2 in 
> > the "
> > "graphical installer, ALT+F2 for the text-based variant) to check the "
> > "available devices in /dev with \"ls /dev\". You can return to this screen "
> > "by pressing CLTR+ALT+F5 respective ALT+F1."
> >
> 
> "> by pressing CLTR+ALT+F5 respective ALT+F1."
> 
> At the very least the above line should be:
> 
> "by pressing CLTR+ALT+F5 and ALT+F1 respectively."


Summarizing your comments into one patch, would this be ok?



diff --git a/debian/cdrom-detect.templates b/debian/cdrom-detect.templates
index 2058b0b..ba50e89 100644
--- a/debian/cdrom-detect.templates
+++ b/debian/cdrom-detect.templates
@@ -55,9 +55,10 @@ _Description: Device file for accessing the installation 
media:
  the device file that should be used. Non-standard CD-ROM drives use
  non-standard device files (such as /dev/mcdx).
  .
- You may switch to the shell on the second terminal (ALT+F2) to check the
+ You may want switch to the shell on the second terminal (CTRL+ALT+F2 in the
+ graphical installer, ALT+F2 for the text-based variant) to check the
  available devices in /dev with "ls /dev". You can return to this screen
- by pressing ALT+F1.
+ by pressing CTRL+ALT+F5 or ALT+F1 respectively.
 
 Template: cdrom-detect/cdrom_fs
 Type: string





-- 
Holger Wansing 
PGP-Fingerprint: 496A C6E8 1442 4B34 8508  3529 59F1 87CA 156E B076



Re: [cdrom-detect] Change template proposal

2019-12-08 Thread Justin B Rye
Holger Wansing wrote:
> 
> Template: cdrom-detect/cdrom_device
> Type: string
> Default: /dev/cdrom
> # :sl2:
> _Description: Device file for accessing the installation media:
>  In order to access your installation media (like your CD-ROM), please enter
>  the device file that should be used. Non-standard CD-ROM drives use
>  non-standard device files (such as /dev/mcdx).
>  .
>  You may switch to the shell on the second terminal (ALT+F2) to check the
>  available devices in /dev with "ls /dev". You can return to this screen
>  by pressing ALT+F1.
> 
> 
> 
> This template is from the time where we only had the text-based installer, 
> and it does not work exactly for the graphical one.
> Therefore, I would like to change the second part like this:
> 
> 
> "You may want to switch to the shell on the second terminal (CLTR+ALT+F2 in 
> the "
> "graphical installer, ALT+F2 for the text-based variant) to check the "
> "available devices in /dev with \"ls /dev\". You can return to this screen "
> "by pressing CLTR+ALT+F5 respective ALT+F1."
   ^^
> 
> 
> Comments?

"Respective" is never ever a conjunction in English.  Just say "or".

And it's CTRL, not CLTR.
-- 
JBR with qualifications in linguistics, experience as a Debian
sysadmin, and probably no clue about this particular package