Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

epm: fix add_unirequires if the package contains files of different architectures (eterbug #17610) #231

Open
wants to merge 2 commits into
base: devel
Choose a base branch
from

Conversation

Boria138
Copy link
Collaborator

@Boria138 Boria138 commented Aug 17, 2024

No description provided.

@Boria138 Boria138 changed the title epm: fix add_unirequires if the package contains files of different a… epm: fix add_unirequires if the package contains files of different architectures (eterbug #17610) Aug 17, 2024
@vitlav
Copy link
Collaborator

vitlav commented Aug 18, 2024

Этот коммит исправляет
__get_binary_requires()
которая не имеет отношения к add_unirequires

@Boria138
Copy link
Collaborator Author

Этот коммит исправляет __get_binary_requires() которая не имеет отношения к add_unirequires

Основные зависимости ищются через add_libs_requires которая добавляет в add_unirequires всё что получает от get_libs_requires которая в свою очередь получает зависимости от __get_binary_requires, проблема в том что __get_binary_requires возвращает библиотеки без архитектуры, раньше add_unirequires смотрел по бинарникам если в пакете есть 32-х битные бинарники то и библиотеки 32-х битные что ломало bitwig потому что там нужны библиотеки обоих архитектур вот я и изменил __get_binary_requires что бы он сразу возращал всё с архитектурами и изменил add_libs_requires так что бы он запускал add_unirequires с новым аргументом auto который подрузмевает что add_unirequires просто прочитает список библиотек и всё, что бы сохранить старое поведение нужно писать без аргументов, но я это я описал в баге так что не буду дублировать здесь

@Boria138
Copy link
Collaborator Author

Могу коммит переименовать в fix add_libs_requred если это прям критически важно, или могу переименовать в fix bitwig dependency check, сам по себе баг с bitwig о котором мне написал @VanoMajukin я исправил

@Boria138
Copy link
Collaborator Author

Или мне на каждую изменённую функцию делать свой коммит ? Получается на add_unirequires, add_libs_requeres и get_binary_requres ?

@Boria138 Boria138 force-pushed the fix-add_unirequires branch 2 times, most recently from 85c15a8 to 29ff5ef Compare August 20, 2024 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants