Change 23456
of the wrong dir.
Bypassing pre-commit review as this is an obvious typo fix.
#review-23457
|
|
rmarin (on ) commented
7 years ago
Hey @tom_tyler Are you sure this is a fix? I just ran into an error when falling into this logic while setting up a replica, the exact error i get is: "stderr": "/depotdata/sdp/Server/Unix/setup/mkdirs.sh: line 220: LD: unbound variable", I can't see that LD is declared or assigned in this file or in the cfg file. Am I missing something? |
| ·0 | |
|
|
|
|
|
C. Thomas Tyler commented
7 years ago
Good catch! Thanks! I just shelved the change I intended to make the first time! The fix was intended to be to change the variable from $LG -> $DD, but it was done wrong as $LG -> $LD (where $LD is bogus and undefined). See 2 lines above this line where it creates the directory with $DD in the path. That's because the checkpoints being created should be created on the /hxdepots volume (the one that must be backed up). The directory is created correctly, but the chown misses adjusting the ownership due to the original bug. This this fix, the directory is created correctly and chown'd correctly. |
| ·0 | |
|
|