Release v25030084 #1120
Release v25030084 #1120
Conversation
.docker/Dockerfile-unique-release
Outdated
| cd unique-chain && git switch "${CODE_HASH}" --detach && cargo update && \ | ||
| CARGO_INCREMENTAL=0 cargo build --release --features="${FEATURES}" --locked && \ | ||
| cd unique-chain && git switch "${CODE_HASH}" --detach && \ | ||
| CARGO_INCREMENTAL=0 cargo build --release --features="${FEATURES}" --offline && \ |
There was a problem hiding this comment.
Offline предполагает что все пакеты уже скачаны и находятся в кеше, нам это не подходит
И даже если бы подходило, --locked нам всё ещё надо
There was a problem hiding this comment.
Обсуждали на мите. Сделаем сначала cargo fetch, а потом билд с --frozen
There was a problem hiding this comment.
Если коротко, то зачем?)
There was a problem hiding this comment.
Особенно если cargo fetch не передавать --locked
There was a problem hiding this comment.
С --locked для fetch.
Если коротко, то зачем?)
Ну, мы же хотим предыдущий релиз таким, каким он был когда был актуален, без изменений. И с той же версией компилятора. А некоторые из внутренних зависимостей используют ещё более глубокие зависимость, разрешая minor update'ы при resolution'е.
Парочка таких глубоких зависимостей хочет edition2024, на которое ругается старый компилятор для старого релиза.
Чтобы не изменять релиз и собрать как есть, кажется, что стоит взять зависимости как есть в Cargo.lock и собрать --frozen
There was a problem hiding this comment.
--frozen емнип вовсе не то что мы хотим, но я забыл где для него нормальное описание есть
Короче с frozen не будут проверяться гит зависимости несколько я помню
There was a problem hiding this comment.
Не помню где это всё было написано, но помню что из-за frozen в nixpkgs для buildCargoPackage сделаны гитовые костыли
There was a problem hiding this comment.
Кмк тут изначально оно было правильно сделано просто с cargo build --locked, и я не понимаю чего вы хотите добиться с другими флагами/дёрганьем cargo fetch, и прочим)
There was a problem hiding this comment.
Он дополнительно ругается, что Cargo.lock хочется обновить и отказывается собирать.
Так нам это и надо, мы не хотим чтобы оно в CI пыталось резолюцию провести
Когда оно ругается надо чтобы кто-то явно обновил локфайл и закоммитил его
There was a problem hiding this comment.
Для воспроизводимости
Ну, мы же хотим предыдущий релиз таким, каким он был когда был актуален, без изменений. И с той же версией компилятора. А некоторые из внутренних зависимостей используют ещё более глубокие зависимость, разрешая minor update'ы при resolution'е.
Локфайл в репе должен работать без необходимости что-то при сборке резолвить
5 participants
Не мержить - напоминаю что master теперь обновляем через fast-forward
Гитхаб в своём UI такое не умеет, а потому только через консоль
Когда CI пройдёт и всё хорошо - кто-то с правом обновления мастера (Например я) сделает
Либо