From 7030ca148d2cf3d63e1cbeb93ccdb26b0b82273eb601ee7fa5ee84aa52817afa Mon Sep 17 00:00:00 2001 From: Steve Kowalik Date: Wed, 13 Nov 2024 01:19:48 +0000 Subject: [PATCH 1/3] - Use correct PDM package to build, it's pdm-backend. OBS-URL: https://build.opensuse.org/package/show/devel:languages:python:mailman/python-postorius?expand=0&rev=64 --- python-postorius.changes | 5 +++++ python-postorius.spec | 2 +- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/python-postorius.changes b/python-postorius.changes index 6e82b5a..7cb83c9 100644 --- a/python-postorius.changes +++ b/python-postorius.changes @@ -1,3 +1,8 @@ +------------------------------------------------------------------- +Wed Nov 13 01:19:25 UTC 2024 - Steve Kowalik + +- Use correct PDM package to build, it's pdm-backend. + ------------------------------------------------------------------- Sat Sep 21 07:30:43 UTC 2024 - Andreas Schneider diff --git a/python-postorius.spec b/python-postorius.spec index 8ac939f..cf6db6f 100644 --- a/python-postorius.spec +++ b/python-postorius.spec @@ -68,7 +68,7 @@ Source20: README.SUSE.md Patch0: postorius-settings.patch # BuildRequires: %{python_module pip} -BuildRequires: %{python_module pdm} +BuildRequires: %{python_module pdm-backend} BuildRequires: %{python_module setuptools} BuildRequires: %{python_module wheel} BuildRequires: acl From e4efffb4ec04e626e54ce2f9489e1ac0f7f7b6f19aded590fc0b64a658da9acd Mon Sep 17 00:00:00 2001 From: Steve Kowalik Date: Thu, 14 Nov 2024 03:17:09 +0000 Subject: [PATCH 2/3] - Due to bsc#1231027, add /srv/www to %files. OBS-URL: https://build.opensuse.org/package/show/devel:languages:python:mailman/python-postorius?expand=0&rev=65 --- python-postorius.changes | 3 ++- python-postorius.spec | 4 +++- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/python-postorius.changes b/python-postorius.changes index 7cb83c9..0eb0323 100644 --- a/python-postorius.changes +++ b/python-postorius.changes @@ -1,7 +1,8 @@ ------------------------------------------------------------------- -Wed Nov 13 01:19:25 UTC 2024 - Steve Kowalik +Thu Nov 14 03:16:20 UTC 2024 - Steve Kowalik - Use correct PDM package to build, it's pdm-backend. +- Due to bsc#1231027, add /srv/www to %files. ------------------------------------------------------------------- Sat Sep 21 07:30:43 UTC 2024 - Andreas Schneider diff --git a/python-postorius.spec b/python-postorius.spec index cf6db6f..9101f57 100644 --- a/python-postorius.spec +++ b/python-postorius.spec @@ -24,7 +24,8 @@ %global django_max_version 5.1 %global mailmanclient_min_version 3.3.3 -%global webapps_dir /srv/www/webapps +%global srv_www_dir /srv/www +%global webapps_dir %{srv_www_dir}/webapps %global postorius_pkgname postorius @@ -267,6 +268,7 @@ fi %dir %{webapps_dir}/mailman %defattr(-,root,postorius) +%dir %{srv_www_dir} %dir %{postorius_basedir} %{postorius_basedir}/__init__.py %{postorius_basedir}/manage.py From 3b3b47553ea87dafce03792a272a8a6395cd09cf6193005019ca232ccf676a70 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 3/3] 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