#50908 Issue 50758 - Only Recommend bash-completion, not Require
Closed 3 years ago by spichugi. Opened 4 years ago by mhonek.

file modified
+1 -1
@@ -303,7 +303,7 @@ 

  Requires: python%{python3_pkgversion}-argcomplete

  Requires: python%{python3_pkgversion}-libselinux

  Requires: python%{python3_pkgversion}-setuptools

- Requires: bash-completion

+ Recommends: bash-completion

  %{?python_provide:%python_provide python%{python3_pkgversion}-lib389}

  

  %description -n python%{python3_pkgversion}-lib389

Bug Description:
bash-completion package is only useful for interactive use. E.g. in case of a
container deployment this is unnecessary.

Fix Description:
Change Requires to Recommends, which allows the dnf/yum ran with
install_weak_deps=False to not install this package; however, this option is by
default True, therefore a regular installation will install the package.

Relates https://pagure.io/389-ds-base/issue/50758

Author: Matus Honek mhonek@redhat.com

Review by: ???

rebased onto 6fd09c4

4 years ago

Pull-Request has been merged by mreynolds

4 years ago

LGTM too, merging for Matus :-)

389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.

This pull request has been cloned to Github as issue and is available here:
- https://github.com/389ds/389-ds-base/issues/3961

If you want to continue to work on the PR, please navigate to the github issue,
download the patch from the attachments and file a new pull request.

Thank you for understanding. We apologize for all inconvenience.

Pull-Request has been closed by spichugi

3 years ago
Metadata