Skip to content

ошибка в README.md - установка репы в Ubuntu #379

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
dld-r00f opened this issue May 3, 2021 · 3 comments
Closed

ошибка в README.md - установка репы в Ubuntu #379

dld-r00f opened this issue May 3, 2021 · 3 comments

Comments

@dld-r00f
Copy link
Contributor

dld-r00f commented May 3, 2021

Строка sudo echo "deb [arch=amd64] https://repo.postgrespro.ru/pg_probackup/deb/ $(lsb_release -cs) main-$(lsb_release -cs)" > /etc/apt/sources.list.d/pg_probackup.list не работает от обычного пользователя. Так как sudo распространяется только на команду echo, и не даёт прав перенаправлению на запись в директорию /etc/apt/sources.list.d/
чтобы работало нужно что-то вроде:
sudo sh -c 'echo "deb [arch=amd64] https://repo.postgrespro.ru/pg_probackup/deb/ $(lsb_release -cs) main-$(lsb_release -cs)" > /etc/apt/sources.list.d/pg_probackup.list'

я сделал пул реквест в мастер с исправлением этой строчки в ридми, но не уверен, что поступил правильно :)
#378

@gsmolk
Copy link
Contributor

gsmolk commented May 4, 2021

Добрый день!
Вы поступили правильно, ведь вы самостоятельно взялись за решение проблемы =)
Оставил предложения по улучшению вашего патча в тредике ваше пулл-реквеста.

@dld-r00f
Copy link
Contributor Author

dld-r00f commented May 4, 2021

Спасибо, я, просто, не был уверен, что в мастер стоит пулреквест делать.
Я добавил изменения для секции PostgresPro.

@gsmolk
Copy link
Contributor

gsmolk commented May 11, 2021

Смержено

@gsmolk gsmolk closed this as completed May 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants