Fix CI building - Githubissues

cucumber-sp commented 5 months ago

@Maks1mS https://github.com/cucumber-sp/yandex-music-linux/blob/2a6702b05e17daee47712e8f58cf9155a59675e5/.github/workflows/build.sh#L17 устанавливаем два раза уже jq но не помогает) видимо в чем-то другом проблема

cucumber-sp commented 5 months ago

@Maks1mS я бы перенес сборку deb да и rpm в будущем в арч тоже, нет особо смысла от нескольких контейнеров мне кажется, можно в одном все собрать, для deb пакетов только понадобится dpkg установить через pacman и все

Maks1mS commented 5 months ago

@Maks1mS я бы перенес сборку deb да и rpm в будущем в арч тоже, нет особо смысла от нескольких контейнеров мне кажется, можно в одном все собрать, для deb пакетов только понадобится dpkg установить через pacman и все

Да, можно сделать так

Maks1mS commented 5 months ago

Похоже, этим комитом всё исправлено cd5016dc013495a19e4ac1fa4d7ffd99972f6e37. Еще можно сделать релиз на workflow_dispatch или on push tags.

Можно добавить что-то вроде этого.

cucumber-sp commented 5 months ago

@Maks1mS да, я бы просто заменял файлы в релизе для текущей версии новыми. То есть пока 5.0.8 то в релизе 5.0.8 заменял существующие новыми. Если сможете реализовать то будет отлично

Maks1mS commented 5 months ago

@cucumber-sp

https://github.com/cucumber-sp/yandex-music-linux/blob/42c3cf52a33ed2bb914b02873e363865ab92573f/.github/workflows/update_packages.yml#L10

А должно быть

runs-on: ubuntu-latest
container: 
  image: archlinux:latest

Поэтому не запускается

cucumber-sp commented 5 months ago

@Maks1mS вроде бы все сделали, единственное о чем вас могу попросить это добавить следующий функционал. Перед сборкой коммита проверять не обновилось ли exe яндекс музыки. Если обновилось то прекращать текущий workflow, вызвать Update Packages и он уже создав коммит с обновлениями пакетов автоматически триггернет еще одну сборку но уже с новыми пакетами. Ну и из самой сборки можно убрать обновление пакетов через generate_packages.sh

Maks1mS commented 5 months ago

Могу сделать, только в настройках нужно будет установить токен https://docs.github.com/en/actions/using-workflows/triggering-a-workflow#triggering-a-workflow-from-a-workflow

Как раз из-за этого и не сработало:

cucumber-sp commented 5 months ago

Могу сделать, только в настройках нужно будет установить токен https://docs.github.com/en/actions/using-workflows/triggering-a-workflow#triggering-a-workflow-from-a-workflow

Как раз из-за этого и не сработало:

а если вручную вызвать то должно сработать? С токенами возиться мне кажется целая история просто

Maks1mS commented 5 months ago

Могу сделать, только в настройках нужно будет установить токен https://docs.github.com/en/actions/using-workflows/triggering-a-workflow#triggering-a-workflow-from-a-workflow Как раз из-за этого и не сработало:

а если вручную вызвать то должно сработать? С токенами возиться мне кажется целая история просто

Если вручную вызвать, то должно сработать.

С токеном это так будет примерно выглядеть: https://github.com/Maks1mS/xmrig-zero-donation/blob/main/.github/workflows/update-upstream.yml В таком варианте вызывается workflow build.yml:

Maks1mS commented 5 months ago

Вручную пушить комит с "Update packages to Beta", а не вызывать через workflow_dispatch

cucumber-sp commented 5 months ago

Вручную пушить комит с "Update packages to Beta", а не вызывать через workflow_dispatch

я имею ввиду коммит делать через action а потом из этого же action вызвать другой workflow вручную если он не триггерится с коммита

Maks1mS commented 5 months ago

Вручную пушить комит с "Update packages to Beta", а не вызывать через workflow_dispatch

я имею ввиду коммит делать через action а потом из этого же action вызвать другой workflow вручную если он не триггерится с коммита

Я понял, так тоже можно. Завтра займусь этим

hrueschwein commented 5 months ago

Прошу прощения что, возможно, не в тему — а куда делась сборка RPM?) Сейчас на выходе только .deb, nix flake и PKGBUILD

cucumber-sp commented 5 months ago

Прошу прощения что, возможно, не в тему — а куда делась сборка RPM?) Сейчас на выходе только .deb, nix flake и PKGBUILD

отказались от electron forge чтобы вручную все собирать, откройте новый issue, в течение пары дней посмотрю структуру пакетов и добавлю сборку

cucumber-sp / yandex-music-linux

Fix CI building #22