diff options
| author | spreis <spreis@yandex-team.com> | 2024-03-28 11:07:37 +0300 |
|---|---|---|
| committer | spreis <spreis@yandex-team.com> | 2024-03-28 11:17:45 +0300 |
| commit | 2de742ce97267f0469cf0c9bd384488c91442bce (patch) | |
| tree | 15daa45ea6fb7ecfd1e07759c7bbd6fba92075a6 /build/plugins/pybuild.py | |
| parent | 48212452a70da88a5e4b814979dd82f22379801f (diff) | |
| download | ydb-2de742ce97267f0469cf0c9bd384488c91442bce.tar.gz | |
Revertcommits
Эта история про второй питон подразнесла нам графогенерацию . Нам придется этот коммит откатить.
Суть проблемы в том, что генерация для 2го и 3го раньше была одной и той же командой. Ты сделал так, что команда стала разной, но аутпут этих двух команд одинаковый. Имя результата команды является ключом и сейчас берётся какая-то рандомная команда из двух. При этом команда влияет на UID и все UIDы питонячей протогенерации мигают. От фатальных проблем нас спасает только то, что protobuf-ы на самом деле одинаковые. И даже это не факт - сейчас ты патчишь какие-то рандомные либы включая таковые для 3го питона.
Мы тут конечно редиски - мы в целом контролируем клэши аутпутов, но не в мультимодулях - там часто бывает что одна и та же команда оказывается в нескольких вариантах мультимодуля (потому что оно по дефолту сейчас так себя ведёт). Мы хотим это зачинить (сделать отдельный подмодуль под кодген и отселить всякие RUN_PROGRAM туда) - но это много работы и мы не успели.
План такой:
1. мы откатываем твой фикс
2. отдельно стабилизируем раздельные команды генерации для 2го и 3го питона
3. накатываем обратно отревеченные тут правки.
916a5456c4e901ab2cda0841b4167e16397e83c4
Diffstat (limited to 'build/plugins/pybuild.py')
| -rw-r--r-- | build/plugins/pybuild.py | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/build/plugins/pybuild.py b/build/plugins/pybuild.py index ccb6d5b98f..526f98b3cf 100644 --- a/build/plugins/pybuild.py +++ b/build/plugins/pybuild.py @@ -655,8 +655,7 @@ def onpy_srcs(unit, *args): if protos: if not upath.startswith(py_runtime_path) and not upath.startswith(builtin_proto_path): - if 'protobuf_old' not in upath: - unit.onpeerdir(py_runtime_path) + unit.onpeerdir(py_runtime_path) unit.onpeerdir(unit.get("PY_PROTO_DEPS").split()) |