https://bugzilla.redhat.com/show_bug.cgi?id=2007918 --- Comment #2 from Frank Crawford <frank@xxxxxxxxxxxxxxxxxx> --- Jakub, Thanks for looking at this, and taking it on. I'll go through your comments sometime in the next few days and either adopt them or discuss them with you further. At first glance, they look to be sensible. As for the line > su -s /bin/sh _tuptime -c "(umask 0022 && /usr/bin/tuptime -x)" that essentially creates and initialises the tuptime DB with its first entry. Doing it here ensures that the permissions, etc, are controlled, rather than being in a more random state. -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=2007918 _______________________________________________ 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 Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
