s3cfg, help, pg-config #30
Conversation
There was a problem hiding this comment.
Всё ок,
табы заменил на пробелы.
Но нужен less как-то при выводе. Иначе текст быстро проскакивает.
В nancy help наверное тоже нужно, хоть там хэлп и небольшой.
Возможно это поможет: https://superuser.com/questions/775941/bash-commands-that-dont-fit-on-one-page-make-output-scrollable
| docker_exec bash -c "cat /machine_home/$PG_CONFIG_FILENAME >> /etc/postgresql/$PG_VERSION/main/postgresql.conf" | ||
| if [ -z ${TARGET_CONFIG+x} ] | ||
| then | ||
| docker_exec bash -c "sudo /etc/init.d/postgresql restart" |
There was a problem hiding this comment.
а где самое применение TARGET_CONFIG? Рестарт вижу, а применения нет..
There was a problem hiding this comment.
Если оно ниже, в строке 676, то зачем рестарт ДО, а не после? Рестарт же для того, чтобы изменения конфига применились.
There was a problem hiding this comment.
Чуть ниже, а здес применение в случае если TARGET_CONFIG не задан и больше рестарта не будет... зачем повторный рестарт...
There was a problem hiding this comment.
ну повторный не надо, да, точно достаточно одного
|
А ещё, вот тут зачем кавычки, они лишние на мой взгляд: |
s3-cfg-path renamed to s3cfg-path
help added
use of pg-config implemented