Typo fix #1069

Merged
merged 2 commits into from Feb 19, 2013

Projects

None yet

3 participants

@Zoubi
Zoubi commented Jan 31, 2013

No description provided.

Michael Delva Typo fix 021265c
@rhelmer rhelmer was assigned Jan 31, 2013
@adngdb
Member
adngdb commented Jan 31, 2013

lgtm but I'd like @rhelmer to review it as well.

@rhelmer rhelmer commented on the diff Jan 31, 2013
docs/prodinstall.rst
@@ -23,8 +23,8 @@ Set up directories and permissions
sudo mkdir -p /data/socorro
sudo useradd socorro
sudo chown socorro:socorro /var/log/socorro
- sudo mkdir /home/socorro/primaryCrashStore /home/socorro/fallback /home/socorro/persistent
- sudo chown www-data-socorro /home/socorro/primaryCrashStore /home/socorro/fallback
+ sudo mkdir -p /home/socorro/primaryCrashStore /home/socorro/fallback /home/socorro/persistent
@rhelmer
rhelmer Jan 31, 2013 Member

hmm useradd should create /home/socorro so this should not be necessary... let me take a look at this before we merge.

@rhelmer rhelmer commented on the diff Jan 31, 2013
docs/prodinstall.rst
@@ -23,8 +23,8 @@ Set up directories and permissions
sudo mkdir -p /data/socorro
sudo useradd socorro
sudo chown socorro:socorro /var/log/socorro
- sudo mkdir /home/socorro/primaryCrashStore /home/socorro/fallback /home/socorro/persistent
- sudo chown www-data-socorro /home/socorro/primaryCrashStore /home/socorro/fallback
+ sudo mkdir -p /home/socorro/primaryCrashStore /home/socorro/fallback /home/socorro/persistent
+ sudo chown www-data:socorro /home/socorro/primaryCrashStore /home/socorro/fallback
@rhelmer
rhelmer Jan 31, 2013 Member

This is definitely a typo thanks :)

Michael Delva Typo fix f53dcac
@rhelmer
Member
rhelmer commented Feb 19, 2013

r+

@rhelmer rhelmer merged commit b7731d0 into mozilla:master Feb 19, 2013

1 check passed

default Jenkins build 'socorro-github' #432 has succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment