Skip to content
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

Переводит python скрипты из ./publisher на работу с go-паблишером #129

Merged
merged 3 commits into from
Feb 6, 2020

Conversation

q210
Copy link
Contributor

@q210 q210 commented Feb 5, 2020

Правки скриптов publisher, заказанные @umputun в #1

Что сделано:

  • python скрипты перенесы в папку ./publisher/mp3tags
  • python скрипты теперь работают с теми же переменными окружения, что и go код (в частности LOCATION)
  • python скрипты переведены с invoke, который раньше был использован ради вызовов shell команд, на click, в связи с изменениями требований
  • python скрипты запускаются с помощью скрипта /usr/local/bin/mp3tags для удобства указания entrypoint'а
  • бинарник publisher, собираемый из go кода, перенесен в контейнере в /usr/local/bin/publisher (раньше он копировался в /srv/publisher и затирался при подключении volume в docker-compose)
  • WORKDIR в Dockerfile сменен на /srv/hugo, чтобы работал go-код
  • ENTRYPOINT в Dockerfile сменен на /usr/local/bin/publisher, т.к. точкой входа стал этот бинарь
  • поправлены README файлы

@umputun
Copy link
Member

umputun commented Feb 5, 2020

было бы замечательно, если бы фиксы к go части были бы отдельным pr

@q210
Copy link
Contributor Author

q210 commented Feb 5, 2020

@umputun готово, вынес изменения go кода в отдельный PR#130, сделав его на основе последних правок ветки publisher-go. Сначала надо принять его, потом этот.

Copy link
Member

@umputun umputun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thx

@umputun umputun merged commit 54dd9f0 into radio-t:publisher-go Feb 6, 2020
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

Successfully merging this pull request may close these issues.

2 participants