[Bug 1769843] Review Request: low-memory-monitor - Monitors low-memory conditions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1769843



--- Comment #11 from Bastien Nocera <bnocera@xxxxxxxxxx> ---
(In reply to Michael Catanzaro from comment #10)
> Everything else looks fine, just check that BR once more before uploading.
> The packaging guidelines indicate systemd-rpm-macros should be used.

I built with "systemd-rpm-macros" here:
https://koji.fedoraproject.org/koji/taskinfo?taskID=39019014

and without it here:
https://koji.fedoraproject.org/koji/taskinfo?taskID=39019120

In both cases the systemd-rpm-macros are correctly expanded. I don't think a
dependency on systemd for systemctl is needed as the macros already account
for systemd not being installed yet, and will pick up the .service file
we install when it, itself, gets installed.

If I read correctly, the BR isn't needed, and scratch builds work as expected.

In the worst case, I'll get a bug about it, but it won't make the package any
less
functional in the short term.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux