[Rpm-maint] [rpm-software-management/rpm] elfdeps: Add full multiarch deps support (#360)

[Panu Matilainen]

Okay so those two cases seem to be intentional for creating backwards compatible provides, but for requires one obviously doesn't need to create compat entries. Clearly there needs to be a way to handle the compat situation, but if somebody was starting from a distro from scratch you probably wouldn't want that. And anybody migrating to multiarch deps will want to get rid of the compat provides sooner or later too.

My suggestion: instead of an option to enable multiarch, make it an option that takes a biarch/multiarch/compat style argument, controllable with a single macro that defaults to biarch. Pass the whole elfInfo structure into addDeps() and handle all the logic inside that. Oh and drop -deps from the name anyhow, it doesn't add any value since *all* elfdeps arguments are about dependencies, because ... well, elf*deps*.


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/360#issuecomment-368789887
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20180227/48c38df0/attachment.html>