search

YoVVassup / MOVision_Builder

Сборщик фанатского аддона MO Vision для Mental Omega
GNU General Public License v3.0
0 stars 0 forks source link

Использование GitHub Action для сборки проекта #2

Closed MahBoiDeveloper closed 1 month ago

MahBoiDeveloper commented 1 month ago

Добавлено:

Пример реализации.

YoVVassup commented 1 month ago

Штука несомненно полезная, однако из-за проблем с размером катсцен (некоторые > 100мб), файлы за них отвечающие лежат особняком на Ядиске, что ломает концепцию автоматической сборки проекта.

MahBoiDeveloper commented 1 month ago

Нет, это не ломает концепцию сборки. Этот позволяет народу скачивать постоянно актуальную версию без необходимости качать весь репозиторий и собирать его ручками. Если есть возможность выгрузить на FTP-сервер кат-сцены, то можно вообще скачивать их при каждой автоматической сборке.

Многие проекты используют GitHub Actions, и ничего не мешает использовать тебе.

MahBoiDeveloper commented 1 month ago

Важно отметить, что это билд между релизами. Этим логично было бы объяснить отсутствие кат-сцен. Ну да ладно, моё дело предложить.

YoVVassup commented 1 month ago

Халявного FTP нет, а так было бы оптимальным решением. А вот про катсцены информация не верна, т.к. в описании сборки информация о этом есть + само определение в настройках для их выбора изменено на список по сравнению с последним релизом, где был простой чекбокс. И логически следует, что если проигнорить отсутствие файлов катсцен при сборке, то при попытке выбрать что-нибудь кроме "Disabled" из списка "Cutscenes" в настройках игры вывалит ошибку по отсутствию expandmo11_XXXp.mix.

MahBoiDeveloper commented 1 month ago

Халявного FTP нет, а так было бы оптимальным решением. А вот про катсцены информация не верна, т.к. в описании сборки информация о этом есть + само определение в настройках для их выбора изменено на список по сравнению с последним релизом, где был простой чекбокс. И логически следует, что если проигнорить отсутствие файлов катсцен при сборке, то при попытке выбрать что-нибудь кроме "Disabled" из списка "Cutscenes" в настройках игры вывалит ошибку по отсутствию expandmo11_XXXp.mix.

Можно сами кат-сцены выгрузить в репозиторий, на крайний случай. Сколько они весят суммарно?

YoVVassup commented 1 month ago

3,2Гб - оригинал + 768p + 1080p, и это только по RA2 и RA2YR, а в планах еще перенос компании RA1 с катсценами из Remaster версии

MahBoiDeveloper commented 1 month ago

Ок, тем не менее, предлагаю смержить ПР, т.к. он поможет хотя бы часть миксов не формировать, а просто скачать для проверки. Видео можно в отдельных миксах держать, чтобы их не было необходимости компилировать. Также я считаю ошибочной идеей делать чекбокс на включение кат-сцен, т.к. если их нет, игра не должна жаловаться и должна запускать миссию.

YoVVassup commented 1 month ago

Часть скомпилить\часть оставить - это противоречит вообще принципу сборки, но вопрос оставляю открытым. По поводу чекбокса катсцен написано выше, нет такого больше. Есть только список по дефолту возвращающий null, что не дает ошибки и не противоречит описанию, другое дело если пользователь захочет выбрать что-либо из списка, при отсутствии нужных файлов mix катсцен. Если они не нужны тебе и ты не хочешь париться с возможным вылетами при случайном выборе можно создать пустые mix, ,чтобы DTA не ломался при попытке скопировать в корень отсутствующего файла. Пожалуй, можно добавить подобную проверку и создание фейковых mix катсцен на последних этапах при сборке builda.

MahBoiDeveloper commented 1 month ago

Часть скомпилить\часть оставить - это противоречит вообще принципу сборки, но вопрос оставляю открытым. По поводу чекбокса катсцен написано выше, нет такого больше. Есть только список по дефолту возвращающий null, что не дает ошибки и не противоречит описанию, другое дело если пользователь захочет выбрать что-либо из списка, при отсутствии нужных файлов mix катсцен. Если они не нужны тебе и ты не хочешь париться с возможным вылетами при случайном выборе можно создать пустые mix, ,чтобы DTA не ломался при попытке скопировать в корень отсутствующего файла. Пожалуй, можно добавить подобную проверку и создание фейковых mix катсцен на последних этапах при сборке builda.

Если будет собираться только часть проекта (без кат-сцен), то это не будет противоречить принципу сборки. Для конечных пользователей есть релизные билды, где всё вылизано и вычищено и будут кат-сцены. Если же человеку интересны изменения в коммите, где исправляется его ошибка — он скачает собранные артефакты и закинет поверх релизного мода, чтобы протестировать. Задача ПР — дать удобство пользователю, который ищет решение своей проблемы, а не предоставить всё в одном.

Использование чекбокса для "включения" кат-сцен — это ошибка проектирования проекта.

YoVVassup commented 1 month ago

Твое решение резонно и оптимально для командной разработки. На данном этапе будет mix-заглушка и от автосборки я, пожалуй, откажусь т.к. репозиторий не резиновый) Не вижу очереди пользователей с проблемами, хоть убей) Про чекбокс я уже тебе НЕ РАЗ писал, ну нет его, все кастомные выборки сейчас на списках.