1234313 - Typo in VirtualEnv Installation documentation

Status: VERIFIED FIXED

(Participation Infrastructure :: Phonebook, defect)

Description

[shine [:shine]]

Step 6 of http://mozillians.readthedocs.org/en/latest/installation/installation-virtualenv.html#mysql-setup says :

Install timezone info tables in mysql:
(venv)$ mysql_tzinfo_to_sql /usr/share/zoneinfo/ | mysql -uroot -proot mysql

There should be a space in between the options -u and -p. It is not a good idea (security wise) to input the root password in plain text to a command. Instead the password option should be left open with just the -p option so that mysql prompts for the password during the execution of the command.

Comment 1

[Akshay S Dinesh]

It is indeed advisable to not enter password on the command line directly. 

Just
mysql_tzinfo_to_sql /usr/share/zoneinfo/ | mysql -uroot -p mysql
must prompt for a password and work.

Note that -uroot and -u root does the same thing.

Comment 2

[[github robot]]

Commits pushed to master at https://github.com/mozilla/mozillians

https://github.com/mozilla/mozillians/commit/f009f12feed8cc4d8a4af90f39baef44714a4ce6
[fix bug 1234313] Remove password from the command line.

https://github.com/mozilla/mozillians/commit/1b7aebe9fbf047a6e4853ce05bf7c607bf69a2dd
Merge pull request #1277 from akatsoulas/1234313-update-doc

[fix bug 1234313] Remove password from the command line.

Comment 3

[John Giannelos [:nemo-yiannis]]

Verified as qa-. Although there is an issue with readthedocs builts in mozillians.org this should not affect it.