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.

Sign up for GitHub

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

Jump to bottom

Restore %requires_eq #1452

Closed
darix opened this issue Dec 1, 2020 · 3 comments
Closed

Restore %requires_eq #1452

darix opened this issue Dec 1, 2020 · 3 comments

Comments

@darix
Copy link

darix commented Dec 1, 2020

it was killed by accident in ba85c95

and I would be glad if it could also be backported to the fedora 33 rpm version again. so i can remove the copy of %requires_eq in the spec files.

Thank you in advance.
@pmatilai
Copy link
Member

pmatilai commented Dec 1, 2020

Removing it was no accident...

@darix
Copy link
Author

darix commented Dec 1, 2020

the commit message was talking about the perl and python macros.

%requires_eq is not related to either of those.

so what was the rational to remove it?

@pmatilai
Copy link
Member

The perl-macros were using it, and were in fact the only user in rpm.
The other reason for wanting to get rid of it was the implementation (calling rpm from within rpm, discussed eg in #541 )

No plans to bring it back.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@darix @pmatilai