Size: a a a

2017 September 27

KT

Konstantin Tskhovrebov in GitFox
Ваш рефакторинг ci.gradle мне понравился. Оставьте его (кроме вычисления номера сборки)
источник

KT

Konstantin Tskhovrebov in GitFox
Но вы делали экран эбаут, а затронули скрипт сборки
источник

KT

Konstantin Tskhovrebov in GitFox
Вот это плохо
источник

VD

Vitalii Dmitriev in GitFox
Затронут был, потому что я использовал hash коммита, а без изменения скриптов сборки я бы не смог засабмитить чтение этого хэша.
Ну либо двумя MR пришлось бы, было бы правильнее.

Я отреверчу рефакторинг метода тогда.

Но я всё ещё в упор не понимаю, как вообще технически наши методы могут отличаться, ибо ведут себя они абсолютно одинаково, ну и

» Текущий способ работает корректно только если не менять историю в гите.
и
» А мой метод подсчёта версии сборки только для статичной истории. В этом его большой минус

как эти отличия возможны — тоже не могу понять.
Мы буквально одно и то же написали.
источник

VD

Vitalii Dmitriev in GitFox
Но поскольку

» Но этот метод - временное решение, которое будет изменено

В общем-то, не важно тогда уже :)
источник

KT

Konstantin Tskhovrebov in GitFox
Нет. Вы не поняли. Я согласен: ваш метод = мой. Вернет одно и то же.
Но мой вариант описан в yml скрипте, который относится к гитлабу.
А ваш в gradle, который принадлежит больше проекту.
Понимаете?
источник

KT

Konstantin Tskhovrebov in GitFox
Теперь стало яснее?)
источник

VD

Vitalii Dmitriev in GitFox
Ну, вот сейчас читаю .yml скрипт, но нет, не понимаю 😔
Ваше тоже было написано в build.gradle, называется так же.
Внутри скрипта .yml в переменную окружения задаётся результат выполнения команды git rev-list --count ${CI_BUILD_REF}, после чего задаёт права выполнения для gradlew, после чего...
...после чего в итоге дёргает просто ./gradlew assembleDebug в итоге.
Потом составляет имя артифакта из переменных окружения CI-системы.

Собственно, и отличие только в том, что скрипт использует переменные окружения CI-системы gitlab'a.

Зато теперь я вижу в этом скрипте, почему именно я сам теперь не буду поднимать версию Android :D
источник

KT

Konstantin Tskhovrebov in GitFox
Vitalii Dmitriev
Ну, вот сейчас читаю .yml скрипт, но нет, не понимаю 😔
Ваше тоже было написано в build.gradle, называется так же.
Внутри скрипта .yml в переменную окружения задаётся результат выполнения команды git rev-list --count ${CI_BUILD_REF}, после чего задаёт права выполнения для gradlew, после чего...
...после чего в итоге дёргает просто ./gradlew assembleDebug в итоге.
Потом составляет имя артифакта из переменных окружения CI-системы.

Собственно, и отличие только в том, что скрипт использует переменные окружения CI-системы gitlab'a.

Зато теперь я вижу в этом скрипте, почему именно я сам теперь не буду поднимать версию Android :D
Вот предпоследний параграф, это и есть причина
источник

KT

Konstantin Tskhovrebov in GitFox
Ладно, в целом я думаю вы поняли, что поменяли шило на мыло.
источник

KT

Konstantin Tskhovrebov in GitFox
Так что едем дальше
источник

KT

Konstantin Tskhovrebov in GitFox
источник

VD

Vitalii Dmitriev in GitFox
Konstantin Tskhovrebov
Вот предпоследний параграф, это и есть причина
Так отличие не между вашим и моим вариантом, а между тем, как я проверял и как оно будет проверяться на CI.
Ток у меня там 3 строчки и коротко, на groovy, gradle-way.
источник

KT

Konstantin Tskhovrebov in GitFox
все. я добрался до рабочего места и проверил. мы с вами разошлись из-за моего общения на ходу. ваше получени номера версии - все ОК.

я был против другого. и именно об этом писал на гитлабе. про получение хеша коммита сборки. вы его получаете через запуск команды, а я прошу экспортнуть его из переменной гитлаба. и дал вам ссылку на список этих переменных
источник

KT

Konstantin Tskhovrebov in GitFox
прошу меня понять - я часто отвечаю, пока нахожусь в дороге
источник

AK

Andrey Khitryy in GitFox
По поводу версионирования - androidGitVersion плагин для гредла есть. Если нужно, скину инфу. Мы в проде юзаем на всех проектах.
источник

KT

Konstantin Tskhovrebov in GitFox
то есть сделать в ямл так:
 - export BUILD_COMMIT_SHA=$CI_COMMIT_SHA

как-то так, надо проверить синтаксис
источник

KT

Konstantin Tskhovrebov in GitFox
Andrey Khitryy
По поводу версионирования - androidGitVersion плагин для гредла есть. Если нужно, скину инфу. Мы в проде юзаем на всех проектах.
нет, у гитлаба уже практически готов правильный номер сборки
источник

KT

Konstantin Tskhovrebov in GitFox
номер сборки должен идти именно из CI
источник

KT

Konstantin Tskhovrebov in GitFox
просто сейчас это не совсем то, что хочется
источник