From dd68d5e8b74d2ba7d98ef3b0f33b32f5dbe7eb3722e772df99bd059a36473a28 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Mark=C3=A9ta=20Machov=C3=A1?= Date: Mon, 18 Nov 2024 08:02:29 +0000 Subject: [PATCH] Accepting request 1224780 from home:mcepl:branches:devel:languages:python:mailman - %defattr is evil! Rather have a long (and readable) list of %attr in front of every line - one consequence of using %defattr is that we were changing ownership of /srv/www, which any individual web server has no business to do. OBS-URL: https://build.opensuse.org/request/show/1224780 OBS-URL: https://build.opensuse.org/package/show/devel:languages:python:mailman/python-postorius?expand=0&rev=66 --- python-postorius.changes | 9 +++++++++ python-postorius.spec | 27 +++++++++++++-------------- 2 files changed, 22 insertions(+), 14 deletions(-) diff --git a/python-postorius.changes b/python-postorius.changes index 0eb0323..7e4ae74 100644 --- a/python-postorius.changes +++ b/python-postorius.changes @@ -1,3 +1,12 @@ +------------------------------------------------------------------- +Mon Nov 18 07:23:45 UTC 2024 - Matej Cepl + +- %defattr is evil! Rather have a long (and readable) list of + %attr in front of every line +- one consequence of using %defattr is that we were changing + ownership of /srv/www, which any individual web server has no + business to do. + ------------------------------------------------------------------- Thu Nov 14 03:16:20 UTC 2024 - Steve Kowalik diff --git a/python-postorius.spec b/python-postorius.spec index 9101f57..160592d 100644 --- a/python-postorius.spec +++ b/python-postorius.spec @@ -267,22 +267,21 @@ fi %dir %{webapps_dir} %dir %{webapps_dir}/mailman -%defattr(-,root,postorius) %dir %{srv_www_dir} -%dir %{postorius_basedir} -%{postorius_basedir}/__init__.py -%{postorius_basedir}/manage.py -%{postorius_basedir}/settings.py -%{postorius_basedir}/settings_local.py -%{postorius_basedir}/test_settings.py -%{postorius_basedir}/urls.py -%{postorius_basedir}/wsgi.py +%attr(-,root,postorius) %dir %{postorius_basedir} +%attr(-,root,postorius) %{postorius_basedir}/__init__.py +%attr(-,root,postorius) %{postorius_basedir}/manage.py +%attr(-,root,postorius) %{postorius_basedir}/settings.py +%attr(-,root,postorius) %{postorius_basedir}/settings_local.py +%attr(-,root,postorius) %{postorius_basedir}/test_settings.py +%attr(-,root,postorius) %{postorius_basedir}/urls.py +%attr(-,root,postorius) %{postorius_basedir}/wsgi.py -%dir %{postorius_basedir}/static -%{postorius_basedir}/static/admin -%{postorius_basedir}/static/django-mailman3 -%{postorius_basedir}/static/postorius -%{postorius_basedir}/static/debug_toolbar +%attr(-,root,postorius) %dir %{postorius_basedir}/static +%attr(-,root,postorius) %{postorius_basedir}/static/admin +%attr(-,root,postorius) %{postorius_basedir}/static/django-mailman3 +%attr(-,root,postorius) %{postorius_basedir}/static/postorius +%attr(-,root,postorius) %{postorius_basedir}/static/debug_toolbar %attr(750,root,postorius) %dir %{postorius_etcdir} %attr(640,root,postorius) %config(noreplace) %{postorius_etcdir}/settings_local.py