Bug 2299818
Summary: | regression in %setup | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Dan Horák <dan> |
Component: | rpm | Assignee: | Packaging Maintenance Team <packaging-team-maint> |
Status: | CLOSED NOTABUG | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | unspecified | ||
Version: | rawhide | CC: | igor.raits, mdomonko, packaging-team-maint, pbrobinson, pmatilai |
Target Milestone: | --- | Keywords: | Regression |
Target Release: | --- | ||
Hardware: | Unspecified | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | If docs needed, set a value | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2024-08-01 07:15:14 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 2300612 |
Description
Dan Horák
2024-07-25 10:21:04 UTC
Most likely this:
> %global builddir %{_builddir}/%{srcdir}
%builddir is a rpm-defined macro in >= 4.20 and redefining it to something else will break stuff. Rename that to "mybuilddir" or something.
yes, renaming builddir solves the issue, thanks for the hint Ack, thanks for confirming. Ideally you'd get a meaningful warning/error message when trying to override system macros of course - and you actually do for built-in macros (eg you can't override %define), but there's nothing like that for others at the moment. Thanks, I was going to suggest something like that about emitting a warning in such cases, like for example the C/GCC preprocessor does about redefined macros. Yup. Unlike C preprocessor macros, rpm macros are by design stackable and tons of functionality depends on that behavior, so we can't just error/warn on any macro redefinition. There'd need to be some "read-only" (or non-stackable) attribute that can be applied to some carefully selected macros. And now I remember there actually is a related ticket already: https://github.com/rpm-software-management/rpm/issues/2314 |