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
_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint

Reply via email to