cucumber-sp / yandex-music-linux

Linux client of Yandex Music
Other
235 stars 5 forks source link

Switch to patching files #15

Closed cucumber-sp closed 5 months ago

cucumber-sp commented 5 months ago

Patch files instead of using sed

Originally posted by @ein-shved in https://github.com/cucumber-sp/yandex-music-linux/issues/11#issuecomment-1919577048

cucumber-sp commented 5 months ago

@ein-shved как вы предлагаете патчить файлы?

ein-shved commented 5 months ago

@cucumber-sp предлагаю использовать собственно утилиту patch. Патчи можно генерировать гитом. Думаю процесс можно пока вручную проводить тут ничего сложного. Главное прикрутить build-валидацию, чтобы хоть частично быть уверенным, что патчи накладываются успешно при перещелкивании на новую версию.

cucumber-sp commented 5 months ago

@cucumber-sp предлагаю использовать собственно утилиту patch. Патчи можно генерировать гитом. Думаю процесс можно пока вручную проводить тут ничего сложного. Главное прикрутить build-валидацию, чтобы хоть частично быть уверенным, что патчи накладываются успешно при перещелкивании на новую версию.

хм, а как должна build валидация работать? Сейчас я все пакеты собираю на локальной машине. Хотите это в облако перенести?

ein-shved commented 5 months ago

Есть github actions. Не пользовался, и не знаю, как это работает, но вроде это как раз таки местный CI. По крайней мере на него ссылается cachix

cucumber-sp commented 5 months ago

Есть github actions. Не пользовался, и не знаю, как это работает, но вроде это как раз таки местный CI. По крайней мере на него ссылается cachix

да, самый популярный вариант, если бесплатного тарифа хватит то можно использовать. но это уже к другому issue относится. по поводу патчинга файлов - нужно занятся. sed для фикса secretKey продолжу использовать т.к. там каждый раз меняется файлы, а вот для основных хендлеров и тд можно использовать патчинг я думаю без проблем

cucumber-sp commented 5 months ago

https://github.com/cucumber-sp/yandex-music-linux/commit/13a6e87a40d934127c7d24f8aefdb570c26820d5