In our TAG, the way we are presenting command lines is not fully consistent: In some cases, we are documenting that you should use "sudo". In many cases, we are not. We should use the same style everywhere.
All examples should now follow the same style. As a bonus they also got some basic highlighting.
It seems we have used both `.. code:: console` and `.. code-block:: console` tagging for these types of cases. We should probably standardize on one of them to avoid future problems with styling etc.
Everything has now been standardised on code::.
Some sudo's are missing from the "Tip" for tl-ssh-all in commands.html: > First, setup the SSH key-pair as described below: > > # > # First time / One time procedure > # > # Generate a private and public key-pair for SSH with SSH keygen. > # When prompted pick a secret password for the key-pair. > # > ssh-keygen -t dsa > > # Copy the public key to SSH authorized_keys > cp /root/.ssh/id_dsa.pub /root/.ssh/authorized_keys > > # Make sure the authorized key has the right permissions > chmod 600 /root/.ssh/authorized_keys > > # Copy the authorized key to all ThinLinc Agents > tl-rsync-all /root/.ssh/authorized_keys > > Next, before using tl-ssh-all, do as follows > > eval `ssh-agent` > ssh-add > > # Run your commands > tl-ssh-all rpm -Uvh /root/kdelibs-3.5.1-1.fc4.i386.rpm Also, we should probably replace "dsa" with "rsa" there :)
I think that tip should probably be removed. It doesn't really fit well with the sudo principle of avoiding switching user to root, since that is effectively what you do if you set up a SSH agent with a key that gives you root access.
Section is now removed.
Looks good: * All commands now use sudo where appropriate. Note that the instructions for running "install-server" and "tl-setup" don't need sudo in this case since these programs have a built in prompt asking for that. * There are no "code-block" left * The command styling look good both in the HTML and the PDF * The commit removing the tl-ssh-all "Tip" looks good
