Re: F35 Change proposal: RPM 4.17 (System-Wide Change proposal)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 4/7/21 1:12 PM, Tomas Orsava wrote:
On 4/7/21 11:38 AM, Panu Matilainen wrote:
On 4/7/21 12:06 PM, Miro Hrončok wrote:
On 07. 04. 21 10:45, Panu Matilainen wrote:
I'm starting to think the right thing to do is to move %check to run after %build rather than %install. That would completely eliminate arguments over what is proper use and should this or that be done etc.

This is what I don't understand: The current %exclude change is backwards incompatible and breaks things. How does breaking it even more help this problem?

Because they kinda go together: the %exclude change revealed that %check is being used in ways it wasn't really intended to be used - just like %exclude was - and both "abuses" come with unwanted side-effects (note quotes, while I know the internal intention, the intended uses were never clearly documented anywhere)

 Further rationale at https://github.com/rpm-software-management/rpm/pull/1618


I understand wanting a perfect system, but I think it's important to realize that rpm is a tool for maintainers to simplify their life and make packaging other software possible.

What you're proposing now is to break thousands of packages with no advanced warning, no migration path, just because they're using rpm in a way that wasn't predicted.

Please don't do this.

If you really want to introduce such backwards incompatible changes, please first work with us maintainers on the migration path and a reasonable timeline to do so.

Isn't that exactly what we're doing here?

The %check PR may or may not go in, it was linked here so people have a chance to discuss it. Because there's this just discovered linkage between %exclude and %check uses, it probably makes sense for them to go in together. Which could mean doing both, or neither, in 4.17. As in, reverting the %exclude change in 4.17 is entirely possible if it makes sense in the grand scheme things.

We cannot possibly know what all the tens of thousands of packages out there are doing, and people will only ever wake up when the change is about to hit them. We've done a dozens of similar changes not because it's fun or because I enjoy getting yelled at for breaking their stuff but because there's no other way to fix ambiguity than making it unambiguous.

	- Panu -
_______________________________________________
devel mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/devel@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [Fedora Users]     [Fedora Kernel]     [Fedora Testing]     [Fedora Formulas]     [Fedora PHP Devel]     [Kernel Development]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Gimp]     [Yosemite News]

  Powered by Linux