https://bugzilla.redhat.com/show_bug.cgi?id=2168538
Neal Gompa <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |[email protected] --- Comment #13 from Neal Gompa <[email protected]> --- > # Edit the default config file > sed -i 's/#log_dir=\/var\/log/#log_dir=\/var\/log\/minidlna/' \ > %{name}.conf To make this easier to read, you should consider changing the substitution delimiter, like so: sed -i 's|#log_dir=/var/log|#log_dir=/var/log/minidlna|' %{name}.conf > %pre > getent group minidlna >/dev/null || groupadd -r minidlna > getent passwd minidlna >/dev/null || \ > useradd -r -g minidlna -d /dev/null -s /sbin/nologin \ > -c "minidlna service account" minidlna > exit 0 System users need to use the sysusers.d format: https://fedoraproject.org/wiki/Changes/Adopting_sysusers.d_format Cf. https://docs.fedoraproject.org/en-US/packaging-guidelines/UsersAndGroups/#_dynamic_allocation -- 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=2168538 _______________________________________________ package-review mailing list -- [email protected] To unsubscribe send an email to [email protected] 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/[email protected] Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
