Re: [PATCH] intro.1: some improvements

Hi Stéphane

[I'll reorder the text a mail a little for ease of reply]

quoted

I'm not sure that we really need most of the above piece. I mean, in these 
days of GUI and laptops. Most of us use the GUI or close the lid to 
shutdown/suspend.

I use startx / shutdown, so it does not shock me. This addition and the
other I have proposed that the reader knows what to do if X does not
start.

Okay. So, some observations:

1. I've never been a huge fan of the extended intro.1 page. I just think
   it's a little out of place as man page. For the most part, tutorials
   are best put somewhere else. That said, the intro(1) page was already
   there when I arrived as maintainer, and I'm not so disturbed by
   the tutorial content that I want to eliminate it. I will even take
   reasonable additions to it.

2. I don't want this page to grow enormous.

3. I'd like it to focus on the commonly used commands. Or, where it 
   covers less commonly used commands, then a few words of explanation
   are in order. For example, one could say: "Although the X Windows
   system is commonly started automatically, and the system is shut down
   or suspended through a graphical interface, there are commands that
   can be used from the shell to perform each of these steps..."
   See what I mean?

If you do not want, no problem, but now the page is incoherent:

- Control-D was moved to the non-added portion
- Su (1), shutdown (8) in excess in SEE ALSO

Good point. So, some clean up is needed. Could you take a look?
But see comments below.

Le mardi 10 mars 2015 à 07:09:10, Michael Kerrisk (man-pages) a écrit :

quoted

I applied most of the patch, but...

On 03/09/2015 09:26 PM, Stéphane Aulery wrote:

quoted

- Add a section Logout and poweroff and change login/logout by open/close session

... not this first piece. See comments below.

So, a metacomment. This piece really would have been better as a
separate patch, since it's rather a new topi, and then we could
have started this conversation sooner, and also avoided the small
incoherent piece that we now have.

I would probably take a patch that covers this stuff, but addresses my 
idea above. But see also the comments below.

quoted

quoted

+.SS Logout and poweroff
+When you work is finished,
+you can press Control-D to close the current session.

This text is confusing. What is the "current session"?

quoted

quoted

+This does not turn off the machine.

What is "This"? I think it's confusing to bring together the
discussion of terminating a shell session and shutting down the system.

quoted

quoted

+Only superusers have access to this command.

What is "this command"?

quoted

quoted

+If you are the single user you can also not log off

What is "the single user"?

quoted

quoted

+and becoming a superuser by typing the command
+.BR su (1).

In general, I can't understand what the last sentence is trying 
to say...

quoted

quoted

+Also you must type yours password.

s/yours/your/

quoted

quoted

+Finally enter the command "shutdown -P +1",
+which will turn off the computer after one minute.
+See
+.BR shutdown (8).

So, you see pat of my problem was also that I think the text 
still needed much work. I should have explained that, but I 
left off to so, since I was not convinced we need this stuff 
in general. (But, if you want to do the work to make something,
I will probably take it.)

I'd propose this, if you want to try and add these pieces:

* Split any discussion of closing a shell session off into a 
  separate piece.
* Explain startx if you want.
* Explain shutdown if you want.

But, I think all of these pieces are more or less
orthogonal and should be explained separately.

Thanks,

Michael

-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html