From 2be5fc096480d3e3b2fa511c1b4b3ed45933d515 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Petr=20P=C3=ADsa=C5=99?= Date: Wed, 12 Jun 2024 15:51:35 +0200 Subject: [PATCH] Fix dnf remove-n, install-n, and autoremove-n commands to only match package names Resolves: RHEL-5747 --- ...-nevra-forms-when-provided-by-comman.patch | 135 ++++++++++++++++++ ...de-of-spec-definition-for-repoquery-.patch | 31 ++++ dnf.spec | 4 + 3 files changed, 170 insertions(+) create mode 100644 0002-Limit-queries-to-nevra-forms-when-provided-by-comman.patch create mode 100644 0003-doc-Remove-provide-of-spec-definition-for-repoquery-.patch diff --git a/0002-Limit-queries-to-nevra-forms-when-provided-by-comman.patch b/0002-Limit-queries-to-nevra-forms-when-provided-by-comman.patch new file mode 100644 index 0000000..3a80ff6 --- /dev/null +++ b/0002-Limit-queries-to-nevra-forms-when-provided-by-comman.patch @@ -0,0 +1,135 @@ +From b3b9b3a48ca5efd9935f3f12bb65530a51ade09c Mon Sep 17 00:00:00 2001 +From: Jaroslav Mracek +Date: Wed, 22 May 2024 11:09:34 +0200 +Subject: [PATCH] Limit queries to nevra forms when provided by command +MIME-Version: 1.0 +Content-Type: text/plain; charset=UTF-8 +Content-Transfer-Encoding: 8bit + +Command `dnf install-n ` does not install only according +to package mame but still search in provides. The patch limits +searrch only to NEVRA forms for install, remove, autoremove, +and repoquery commands. + +Resolves partially: https://issues.redhat.com/browse/RHEL-5747 + +Signed-off-by: Petr Písař +--- + dnf/base.py | 14 ++++++++++++-- + dnf/cli/commands/repoquery.py | 5 +++-- + doc/api_base.rst | 6 +++++- + doc/command_ref.rst | 7 +++++-- + 4 files changed, 25 insertions(+), 7 deletions(-) + +diff --git a/dnf/base.py b/dnf/base.py +index a6b35746..dac3cefd 100644 +--- a/dnf/base.py ++++ b/dnf/base.py +@@ -2064,9 +2064,14 @@ class Base(object): + def install(self, pkg_spec, reponame=None, strict=True, forms=None): + # :api + """Mark package(s) given by pkg_spec and reponame for installation.""" ++ kwargs = {'forms': forms, 'with_src': False} ++ if forms: ++ kwargs['with_nevra'] = True ++ kwargs['with_provides'] = False ++ kwargs['with_filenames'] = False + + subj = dnf.subject.Subject(pkg_spec) +- solution = subj.get_best_solution(self.sack, forms=forms, with_src=False) ++ solution = subj.get_best_solution(self.sack, **kwargs) + + if self.conf.multilib_policy == "all" or subj._is_arch_specified(solution): + q = solution['query'] +@@ -2306,8 +2311,13 @@ class Base(object): + def remove(self, pkg_spec, reponame=None, forms=None): + # :api + """Mark the specified package for removal.""" ++ kwargs = {'forms': forms} ++ if forms: ++ kwargs['with_nevra'] = True ++ kwargs['with_provides'] = False ++ kwargs['with_filenames'] = False + +- matches = dnf.subject.Subject(pkg_spec).get_best_query(self.sack, forms=forms) ++ matches = dnf.subject.Subject(pkg_spec).get_best_query(self.sack, **kwargs) + installed = [ + pkg for pkg in matches.installed() + if reponame is None or +diff --git a/dnf/cli/commands/repoquery.py b/dnf/cli/commands/repoquery.py +index 83b52a8a..41dd688e 100644 +--- a/dnf/cli/commands/repoquery.py ++++ b/dnf/cli/commands/repoquery.py +@@ -461,9 +461,10 @@ class RepoQueryCommand(commands.Command): + if self.opts.key: + remote_packages = self._add_add_remote_packages() + +- kwark = {} ++ kwark = {'with_provides': False} + if self.opts.command in self.nevra_forms: + kwark["forms"] = [self.nevra_forms[self.opts.command]] ++ kwark['with_filenames'] = False + pkgs = [] + query_results = q.filter(empty=True) + +@@ -474,7 +475,7 @@ class RepoQueryCommand(commands.Command): + for key in self.opts.key: + query_results = query_results.union( + dnf.subject.Subject(key, ignore_case=True).get_best_query( +- self.base.sack, with_provides=False, query=q, **kwark)) ++ self.base.sack, query=q, **kwark)) + q = query_results + + if self.opts.recent: +diff --git a/doc/api_base.rst b/doc/api_base.rst +index 389b28ec..95d2d570 100644 +--- a/doc/api_base.rst ++++ b/doc/api_base.rst +@@ -280,7 +280,11 @@ + .. method:: install(pkg_spec, reponame=None, strict=True, forms=None) + + Mark packages matching `pkg_spec` for installation. +- `reponame` can be a name of a repository or a list of repository names. If given, the selection of available packages is limited to packages from these repositories. If strict is set to False, the installation ignores packages with dependency solving problems. Parameter `forms` has the same meaning as in :meth:`dnf.subject.Subject.get_best_query`. ++ `reponame` can be a name of a repository or a list of repository names. If given, the selection of available ++ packages is limited to packages from these repositories. If strict is set to False, the installation ignores ++ packages with dependency solving problems. Parameter `forms` is list of pattern forms from `hawkey`_. ++ Leaving the parameter to ``None`` results in using a reasonable default list of forms. When forms is specified, ++ method will not match `pkg_spec` with provides and file provides. + + .. method:: package_downgrade(pkg, strict=False) + +diff --git a/doc/command_ref.rst b/doc/command_ref.rst +index 25431ca1..5fba2ebf 100644 +--- a/doc/command_ref.rst ++++ b/doc/command_ref.rst +@@ -857,7 +857,8 @@ Install Command + are considered correct, the resulting package is picked simply by lexicographical order. + + There are also a few specific install commands ``install-n``, ``install-na`` and +- ``install-nevra`` that allow the specification of an exact argument in the NEVRA format. ++ ``install-nevra`` that allow the specification of an exact argument in the NEVRA format. As a consequence, ++ will be not matched with provides and file provides. + + See also :ref:`\configuration_files_replacement_policy-label`. + +@@ -1191,7 +1192,8 @@ Remove Command + Removes old installonly packages, keeping only latest versions and version of running kernel. + + There are also a few specific remove commands ``remove-n``, ``remove-na`` and ``remove-nevra`` +- that allow the specification of an exact argument in the NEVRA format. ++ that allow the specification of an exact argument in the NEVRA format. As a consequence, ++ will be not matched with provides and file provides. + + Remove Examples + --------------- +@@ -1255,6 +1257,7 @@ Repoquery Command + + There are also a few specific repoquery commands ``repoquery-n``, ``repoquery-na`` and ``repoquery-nevra`` + that allow the specification of an exact argument in the NEVRA format (does not affect arguments of options like --whatprovides , ...). ++ As a consequence, will be not matched with file provides. + + Select Options + -------------- +-- +2.45.2 + diff --git a/0003-doc-Remove-provide-of-spec-definition-for-repoquery-.patch b/0003-doc-Remove-provide-of-spec-definition-for-repoquery-.patch new file mode 100644 index 0000000..8c36672 --- /dev/null +++ b/0003-doc-Remove-provide-of-spec-definition-for-repoquery-.patch @@ -0,0 +1,31 @@ +From f211e1a41a3d3180481e930836ee1a52a7a32487 Mon Sep 17 00:00:00 2001 +From: Jaroslav Mracek +Date: Thu, 23 May 2024 11:25:29 +0200 +Subject: [PATCH] [doc] Remove provide of spec definition for repoquery command +MIME-Version: 1.0 +Content-Type: text/plain; charset=UTF-8 +Content-Transfer-Encoding: 8bit + +Repoquery command never matched spec with provides. + +Signed-off-by: Petr Písař +--- + doc/command_ref.rst | 2 +- + 1 file changed, 1 insertion(+), 1 deletion(-) + +diff --git a/doc/command_ref.rst b/doc/command_ref.rst +index 5fba2ebf..5684b061 100644 +--- a/doc/command_ref.rst ++++ b/doc/command_ref.rst +@@ -1266,7 +1266,7 @@ Together with ````, control what packages are displayed in th + packages to those matching the specification. All packages are considered if no ```` is specified. + + ```` +- Package specification in the NEVRA format (name[-[epoch:]version[-release]][.arch]), a package provide or a file provide. See :ref:`Specifying Packages ++ Package specification in the NEVRA format (name[-[epoch:]version[-release]][.arch]) or a file provide. See :ref:`Specifying Packages + `. + + ``-a``, ``--all`` +-- +2.45.2 + diff --git a/dnf.spec b/dnf.spec index b15782c..ebde06e 100644 --- a/dnf.spec +++ b/dnf.spec @@ -75,6 +75,8 @@ License: GPL-2.0-or-later AND GPL-1.0-only URL: https://github.com/rpm-software-management/dnf Source0: %{url}/archive/%{version}/%{name}-%{version}.tar.gz Patch1: 0001-man-Improve-upgrade-minimal-command-docs-RHEL-6417.patch +Patch2: 0002-Limit-queries-to-nevra-forms-when-provided-by-comman.patch +Patch3: 0003-doc-Remove-provide-of-spec-definition-for-repoquery-.patch BuildArch: noarch BuildRequires: cmake BuildRequires: gettext @@ -420,6 +422,8 @@ popd * Wed Jun 12 2024 Petr Pisar - 4.20.0-2 - Disable downloading filelists by default (RHEL-12355) - Fix dnf upgrade-minimal documentation in dnf(8) manual (RHEL-39892) +- Fix dnf remove-n, install-n, and autoremove-n commands to only match package + names (RHEL-5747) * Wed Apr 24 2024 Jan Kolarik - 4.20.0-1 - Update to 4.20.0 (RHEL-38831)