-
Notifications
You must be signed in to change notification settings - Fork 228
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
package_state output doesn't count with %generate_buildrequires #1429
Labels
Comments
praiskup
changed the title
package_state doesn't provide complete info with dynamic buildrequires
package_state output doesn't count with %generate_buildrequires
Aug 21, 2024
praiskup
added a commit
to praiskup/mock
that referenced
this issue
Aug 22, 2024
This change required us to create a new "postdeps" hook with post-%generate_buildrequires timing. Fixes: rpm-software-management#1429 Relates: rpm-software-management#1393
praiskup
added a commit
to praiskup/mock
that referenced
this issue
Aug 22, 2024
This change required us to create a new "postdeps" hook with post-%generate_buildrequires timing. Fixes: rpm-software-management#1429 Relates: rpm-software-management#1393
praiskup
added a commit
to praiskup/mock
that referenced
this issue
Aug 22, 2024
This change required us to create a new "postdeps" hook with post-%generate_buildrequires timing. Fixes: rpm-software-management#1429 Relates: rpm-software-management#1393
praiskup
added a commit
to praiskup/mock
that referenced
this issue
Aug 22, 2024
This change required us to create a new "postdeps" hook with post-%generate_buildrequires timing. Fixes: rpm-software-management#1429 Relates: rpm-software-management#1393
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The package state output,
installed_pkgs.log
, is supposed to provide a list of installed needed packages for the package to build. But we provide a list of packages after installing static build requires, before calculating the dynamic dependencies (so we do not include them). CC @tkopecekThe text was updated successfully, but these errors were encountered: