Skip to content

{pre,post}-transaction-actions: fix package files filtering regression#540

Open
kszczek wants to merge 2 commits intorpm-software-management:masterfrom
kszczek:pre-post-transaction-actions-fix
Open

{pre,post}-transaction-actions: fix package files filtering regression#540
kszczek wants to merge 2 commits intorpm-software-management:masterfrom
kszczek:pre-post-transaction-actions-fix

Conversation

@kszczek
Copy link
Copy Markdown

@kszczek kszczek commented Jun 12, 2024

According to DNF User's FAQ, starting with Fedora 40 filelists metadata became optional. This caused a regression in pre and post transaction actions plugins which rely on this information. Let's request filelists metadata whenever the pre or post transaction actions plugin is enabled.

kszczek added 2 commits June 13, 2024 10:20
According to DNF User's FAQ, starting with Fedora 40 filelists metadata
became optional. This caused a regression in pre and post transaction
actions plugins which rely on this information. Let's request filelists
metadata whenever the pre or post transaction actions plugin is enabled
and at least one action line contains a file pattern.

= changelog =
msg: Fix package files filtering in pre and post-transaction-actions plugins
type: bugfix
@kszczek kszczek force-pushed the pre-post-transaction-actions-fix branch from 658e16d to 9e893bb Compare June 13, 2024 08:24
@kszczek
Copy link
Copy Markdown
Author

kszczek commented Jun 13, 2024

Force-pushed to make this patch more robust. Instead of requesting filelists metadata whenever the plugin is enabled, it now also checks if at least one action line contains a file pattern before requesting filelists.

@kszczek
Copy link
Copy Markdown
Author

kszczek commented Jun 13, 2024

To work around this issue in the meantime, just set optional_metadata_types=filelists under the [main] section in /etc/dnf/dnf.conf.

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

Successfully merging this pull request may close these issues.

1 participant