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

Навести порядок в скриптах #1

Open
umputun opened this issue Apr 6, 2018 · 5 comments
Open

Навести порядок в скриптах #1

umputun opened this issue Apr 6, 2018 · 5 comments

Comments

@umputun
Copy link
Member

umputun commented Apr 6, 2018

Скрипты в publish хорошо бы привести в человеческий вид. В идеале хотелось бы от всего этого зоопарка избавится, как и от мелких .py и заменить их чем-то одним и общим, делающим тоже самое, но аккуратно и красиво.

@umputun
Copy link
Member Author

umputun commented Feb 4, 2020

добавил начальную версию publisher app. Досконально не тестировал, но на вид все должно быть в порядке.

@q210 – план в том, чтоб все запускалось через нее, т.е. она должна быть основным entrypoint

От этого, есть несколько просьб к питоновой части (и не только):

  • перенести все, что относится к тагам в свой подкаталог, publiser/mp3tags
  • модифицировать Makefile (поменать вызовы в контейнер для всего, что про тэги на прямые вызовы py скриптов)
  • ту часть, что invoke видимо не надо? это будет часть конфигурации внутри mp3tags

@q210
Copy link
Contributor

q210 commented Feb 4, 2020

ок, постараюсь за сегодня сделать

@q210
Copy link
Contributor

q210 commented Feb 5, 2020

сделал в PR#129

@umputun
Copy link
Member Author

umputun commented Feb 10, 2020

мне кажется, что логика с $EPISODE в Makefile основана на предположении что этот EPISODE где-то доступен (или передается)? Если это так, то передача EPISODE это аварийный вариант, например когда подкаст уже опубликован, но по какой-то причине мы хотим повторить один из шагов.

В обычном сценарии внутри publisher он берет этот эпизод автоматически. От этого возникает вопрос/проблема - как это передавать upload-mp3 и print-mp3-tags

@q210
Copy link
Contributor

q210 commented Feb 10, 2020

да, сейчас предполагается что $EPISODE передаётся каждый раз - именно так работает скрипт upload_mp3.sh в мастере (там правда путь до файла, не номер эпизода), я в makefile эту логику лишь повторил.

Если надо, то сделать параметр опциональным не проблема - например могу оформить функцию func episode() (int, error) как самостоятельную команду и получать нужный номер из нее, если $EPISODE не передан. Сделать?

novikovSU added a commit to novikovSU/radio-t-site that referenced this issue Jul 31, 2024
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