Skip to content
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

Replace %{?systemd_requires} with%{?systemd_ordering} #3036

Merged
merged 1 commit into from
Dec 23, 2024

Conversation

jvoisin
Copy link
Collaborator

@jvoisin jvoisin commented Dec 22, 2024

As said in the documentation:

If the package wants to use systemd tools if they are available, but does not
want to declare a dependency, then the %{?systemd_ordering} macro MAY be used as a weaker form of %{?systemd_requires} that only declares an ordering during an RPM transaction.

See systemd/systemd@2424b6b and https://pagure.io/packaging-committee/issue/644 for more information.

And also use --setopt=install_weak_deps=False to avoid installing useless dependencies.

@jvoisin jvoisin mentioned this pull request Dec 22, 2024
17 tasks
@jvoisin jvoisin marked this pull request as draft December 22, 2024 20:25
As said [in the documentation](https://docs.fedoraproject.org/en-US/packaging-guidelines/Scriptlets/#_dependencies_on_the_systemd_package):

> If the package wants to use systemd tools if they are available, but does not
want to declare a dependency, then the %{?systemd_ordering} macro MAY be used
as a weaker form of %{?systemd_requires} that only declares an ordering during
an RPM transaction.

See systemd/systemd@2424b6b
and https://pagure.io/packaging-committee/issue/644 for more information.

And also use `--setopt=install_weak_deps=False` to avoid installing a lot of
useless dependencies.
@jvoisin jvoisin marked this pull request as ready for review December 22, 2024 21:09
@fguillot fguillot merged commit b93543f into miniflux:main Dec 23, 2024
16 checks passed
@jvoisin jvoisin deleted the systemd_ordering branch December 26, 2024 22:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants