PR#4927: spec: Have the log directory owned by the main package - pagure

pagure

#4927 spec: Have the log directory owned by the main package

Merged 3 years ago by pingou. Opened 3 years ago by ngompa.

ngompa/pagure packaging-logdir into master

spec: Have the log directory owned by the main package

Neal Gompa • 3 years ago

6afd134

files/pagure.spec

file modified

+3 -2

		`@@ -493,6 +493,7 @@`
		`%{_unitdir}/pagure_api_key_expire_mail.timer`
		`%{_unitdir}/pagure_mirror_project_in.service`
		`%{_unitdir}/pagure_mirror_project_in.timer`
		`+ %dir %{_localstatedir}/log/pagure`


		`%files web-apache-httpd`
		`@@ -506,8 +507,8 @@`
		`%config(noreplace) %{_sysconfdir}/nginx/conf.d/pagure.conf`
		`%{_unitdir}/pagure_web.service`
		`%{_unitdir}/pagure_docs_web.service`
		`- %dir %{_localstatedir}/log/pagure`
		`- %ghost %{_localstatedir}/log/pagure/*.log`
		`+ %ghost %{_localstatedir}/log/pagure/access_*.log`
		`+ %ghost %{_localstatedir}/log/pagure/error_*.log`


		`%files theme-pagureio`

ngompa commented 3 years ago

It's now entirely reasonable for users to configure Pagure for
extra logging, and Pagure would write log files to /var/log/pagure
by default. Thus, we should just have it owned by the main package.

Signed-off-by: Neal Gompa ngompa13@gmail.com