search

remicollet / remirepo

Issue tracker for
https://rpms.remirepo.net/
277 stars 53 forks source link

unit shouldn't require systemd #245

Closed mu1f407 closed 10 months ago

mu1f407 commented 10 months ago

Hi Remi,

I think that hard systemd dependency in the unit package is not necessary. Unit works well without it. It is mostly noticeable in container images. In my use case, in an image based on ubi9-minimal, the resulting image size is 109 MB (3 deps installed) without systemd (using fakeprovide package instead) vs 142 MB (25 deps installed) with systemd.

Probably the %{?systemd_requires} in spec file could be replaced with %{?systemd_ordering} as written in docs?

Thank you!

remicollet commented 10 months ago

Will take a look

Notice: ubi*-minimal have lof ot issue, with PHP, especially missing tzdata, I usually don't recommend it

remicollet commented 10 months ago

Fixed by https://git.remirepo.net/cgit/rpms/unit.git/commit/?id=bcbc4b9e504eaf36078cefda483d513a33cfb125

No change on EL-7 (cannot verify if RPM have expected flags)

remicollet commented 10 months ago

New build (1.31.1-2) is now available in the repository

mu1f407 commented 10 months ago

Thank you for quick update! Even though systemd is still pulled in through logrotate as dependency. My bad, I didn't checked carefully all packages which requires systemd before, sorry. :man_facepalming:

Regarding the ubi-minimal issues, yes, I also faced the tzdata bug, but found a simple workaround to run microdnf reinstall tzdata which resolves it. Otherwise everything runs fine, including oci8. But it can be little hacky sometimes.

I'm closing this issue. Thank you for your work! :slightly_smiling_face: