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

rpm-config-SUSE changed %requires_eq #185

Open
bnavigator opened this issue Nov 4, 2024 · 1 comment
Open

rpm-config-SUSE changed %requires_eq #185

bnavigator opened this issue Nov 4, 2024 · 1 comment

Comments

@bnavigator
Copy link
Collaborator

openSUSE/rpm-config-SUSE#64, openSUSE/rpm-config-SUSE#79

This breaks the delayed copy and replaced prefix here:

"%requires_eq", "%requires_ge",

elseif PROPERTY_COPY_MODIFIED[property] then
-- specifically handle %name macro before expansion
if not subpackage_only then
line = line:gsub("%%{?name}?", current_flavor .. "-" .. modname)
end
local function print_property_copy_modified(value)
value = replace_prefix(value, flavor)
-- rely on print_altered to perform expansion on the result
print_altered(string.format("%s %s", property, value))
end
if PROPERTY_COPY_DEFAULT_PROVIDER[property] then
-- print renamed lines for all flavors which the current_flavor provides.
for iflavor in string.gmatch(rpm.expand("%{?" .. current_flavor .. "_provides}") .. " " .. current_flavor, "%S+" ) do
current_flavor = iflavor -- make sure to process the main current_flavor last for final reset.
print_property_copy_modified(value)
end
else
print_property_copy_modified(value)
end
end

bmwiedemann pushed a commit to bmwiedemann/openSUSE that referenced this issue Nov 8, 2024
https://build.opensuse.org/request/show/1221370
by user bnavigator + anag+factory
- Avoid using requires_eq, which after the last modifications
  conflicts with python singlespec (order of expansion).
  * gh#openSUSE/python-rpm-macros#185
bmwiedemann pushed a commit to bmwiedemann/openSUSE that referenced this issue Nov 8, 2024
bmwiedemann pushed a commit to bmwiedemann/openSUSE that referenced this issue Nov 8, 2024
bmwiedemann pushed a commit to bmwiedemann/openSUSE that referenced this issue Nov 8, 2024
@mcepl
Copy link
Contributor

mcepl commented Dec 17, 2024

@bnavigator Ben, what is needed to be done here? I thought that we have removed all %requires_* macros in python-* packages, didn’t we?

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

No branches or pull requests

2 participants