Script 'mail_helper' called by obssrc
Hello community,
here is the log from the commit of package python-pyrad for openSUSE:Factory
checked in at 2024-04-08 17:39:29
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/python-pyrad (Old)
and /work/SRC/openSUSE:Factory/.python-pyrad.new.1905 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "python-pyrad"
Mon Apr 8 17:39:29 2024 rev:7 rq:1166046 version:2.4
Changes:
--------
--- /work/SRC/openSUSE:Factory/python-pyrad/python-pyrad.changes
2021-11-08 17:25:24.372741456 +0100
+++ /work/SRC/openSUSE:Factory/.python-pyrad.new.1905/python-pyrad.changes
2024-04-08 17:51:51.564771807 +0200
@@ -1,0 +2,10 @@
+Mon Apr 8 04:36:12 UTC 2024 - Steve Kowalik <steven.kowa...@suse.com>
+
+- Switch to autosetup and pyproject macros.
+- No more greedy globs in %files.
+- Do not install example directory into site-packages.
+- Add missing Requires on netaddr.
+- Add patch use-correct-assertion-methods.patch:
+ * Use non-removed assertion methods.
+
+-------------------------------------------------------------------
New:
----
use-correct-assertion-methods.patch
BETA DEBUG BEGIN:
New:- Add missing Requires on netaddr.
- Add patch use-correct-assertion-methods.patch:
* Use non-removed assertion methods.
BETA DEBUG END:
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ python-pyrad.spec ++++++
--- /var/tmp/diff_new_pack.vdGW6t/_old 2024-04-08 17:51:51.988787457 +0200
+++ /var/tmp/diff_new_pack.vdGW6t/_new 2024-04-08 17:51:51.992787605 +0200
@@ -1,7 +1,7 @@
#
# spec file for package python-pyrad
#
-# Copyright (c) 2021 SUSE LLC
+# Copyright (c) 2024 SUSE LLC
#
# All modifications and additions to the file contributed by third parties
# remain the property of their copyright owners, unless otherwise agreed
@@ -16,7 +16,6 @@
#
-%{?!python_module:%define python_module() python-%{**} python3-%{**}}
Name: python-pyrad
Version: 2.4
Release: 0
@@ -24,12 +23,17 @@
License: BSD-3-Clause
URL: https://github.com/pyradius/pyrad
Source0: https://github.com/pyradius/pyrad/archive/%{version}.tar.gz
+# PATCH-FIX-UPSTREAM gh#pyradius/pyrad#162
+Patch0: use-correct-assertion-methods.patch
BuildRequires: %{python_module netaddr}
-BuildRequires: %{python_module setuptools}
+BuildRequires: %{python_module pip}
+BuildRequires: %{python_module poetry}
BuildRequires: %{python_module six}
+BuildRequires: %{python_module wheel}
BuildRequires: fdupes
BuildRequires: netcfg
BuildRequires: python-rpm-macros
+Requires: python-netaddr
Requires: python-six
BuildArch: noarch
%python_subpackages
@@ -40,13 +44,15 @@
them and decoding responses.
%prep
-%setup -q -n pyrad-%{version}
+%autosetup -p1 -n pyrad-%{version}
%build
-%python_build
+%pyproject_wheel
%install
-%python_install
+%pyproject_install
+# Do not install the examples into site-packages
+%python_expand rm -r %{buildroot}%{$python_sitelib}/example
%python_expand %fdupes %{buildroot}%{$python_sitelib}
%check
@@ -54,5 +60,6 @@
%files %{python_files}
%license LICENSE.txt
-%{python_sitelib}/*
+%{python_sitelib}/pyrad
+%{python_sitelib}/pyrad-%{version}.dist-info
++++++ use-correct-assertion-methods.patch ++++++
>From 02a196a961be37d74725bd3a162dccb8d56f3f3d Mon Sep 17 00:00:00 2001
From: Karthikeyan Singaravelan <tir.kar...@gmail.com>
Date: Mon, 11 Oct 2021 11:30:45 +0000
Subject: [PATCH] Refactor deprecated unittest aliases for Python 3.11
compatibility.
---
tests/testBidict.py | 4 ++--
tests/testClient.py | 26 +++++++++++++-------------
tests/testHost.py | 20 ++++++++++----------
tests/testProxy.py | 6 +++---
tests/testServer.py | 28 ++++++++++++++--------------
5 files changed, 42 insertions(+), 42 deletions(-)
diff --git a/tests/testBidict.py b/tests/testBidict.py
index ad2f134..951e37b 100644
--- a/tests/testBidict.py
+++ b/tests/testBidict.py
@@ -52,5 +52,5 @@ def testItemAccessor(self):
self.bidict.Add("shake", "vanilla")
self.bidict.Add("pie", "custard")
self.assertRaises(KeyError, operator.getitem, self.bidict, "missing")
- self.assertEquals(self.bidict["shake"], "vanilla")
- self.assertEquals(self.bidict["pie"], "custard")
+ self.assertEqual(self.bidict["shake"], "vanilla")
+ self.assertEqual(self.bidict["pie"], "custard")
diff --git a/tests/testClient.py b/tests/testClient.py
index f609294..0effe08 100644
--- a/tests/testClient.py
+++ b/tests/testClient.py
@@ -22,33 +22,33 @@ def setUp(self):
def testSimpleConstruction(self):
client = Client(self.server)
- self.failUnless(client.server is self.server)
+ self.assertTrue(client.server is self.server)
self.assertEqual(client.authport, 1812)
self.assertEqual(client.acctport, 1813)
self.assertEqual(client.secret, six.b(''))
self.assertEqual(client.retries, 3)
self.assertEqual(client.timeout, 5)
- self.failUnless(client.dict is None)
+ self.assertTrue(client.dict is None)
def testParameterOrder(self):
marker = object()
client = Client(self.server, 123, 456, 789, "secret", marker)
- self.failUnless(client.server is self.server)
+ self.assertTrue(client.server is self.server)
self.assertEqual(client.authport, 123)
self.assertEqual(client.acctport, 456)
self.assertEqual(client.coaport, 789)
self.assertEqual(client.secret, "secret")
- self.failUnless(client.dict is marker)
+ self.assertTrue(client.dict is marker)
def testNamedParameters(self):
marker = object()
client = Client(server=self.server, authport=123, acctport=456,
secret="secret", dict=marker)
- self.failUnless(client.server is self.server)
+ self.assertTrue(client.server is self.server)
self.assertEqual(client.authport, 123)
self.assertEqual(client.acctport, 456)
self.assertEqual(client.secret, "secret")
- self.failUnless(client.dict is marker)
+ self.assertTrue(client.dict is marker)
class SocketTests(unittest.TestCase):
@@ -66,7 +66,7 @@ def testReopen(self):
self.client._SocketOpen()
sock = self.client._socket
self.client._SocketOpen()
- self.failUnless(sock is self.client._socket)
+ self.assertTrue(sock is self.client._socket)
def testBind(self):
self.client.bind((BIND_IP, BIND_PORT))
@@ -123,7 +123,7 @@ def testAuthDelay(self):
self.client.timeout = 1
packet = MockPacket(AccessRequest)
self.assertRaises(Timeout, self.client._SendPacket, packet, 432)
- self.failIf("Acct-Delay-Time" in packet)
+ self.assertFalse("Acct-Delay-Time" in packet)
def testSingleAccountDelay(self):
self.client.retries = 2
@@ -154,7 +154,7 @@ def testValidReply(self):
MockPoll.results = [(1, select.POLLIN)]
packet = MockPacket(AccountingRequest, verify=True)
reply = self.client._SendPacket(packet, 432)
- self.failUnless(reply is packet.reply)
+ self.assertTrue(reply is packet.reply)
def testInvalidReply(self):
self.client.retries = 1
@@ -172,14 +172,14 @@ def setUp(self):
def testCreateAuthPacket(self):
packet = self.client.CreateAuthPacket(id=15)
- self.failUnless(isinstance(packet, AuthPacket))
- self.failUnless(packet.dict is self.client.dict)
+ self.assertTrue(isinstance(packet, AuthPacket))
+ self.assertTrue(packet.dict is self.client.dict)
self.assertEqual(packet.id, 15)
self.assertEqual(packet.secret, six.b('zeer geheim'))
def testCreateAcctPacket(self):
packet = self.client.CreateAcctPacket(id=15)
- self.failUnless(isinstance(packet, AcctPacket))
- self.failUnless(packet.dict is self.client.dict)
+ self.assertTrue(isinstance(packet, AcctPacket))
+ self.assertTrue(packet.dict is self.client.dict)
self.assertEqual(packet.id, 15)
self.assertEqual(packet.secret, six.b('zeer geheim'))
diff --git a/tests/testHost.py b/tests/testHost.py
index ec51deb..2b17240 100644
--- a/tests/testHost.py
+++ b/tests/testHost.py
@@ -32,20 +32,20 @@ def setUp(self):
def testCreatePacket(self):
packet = self.host.CreatePacket(id=15)
- self.failUnless(isinstance(packet, Packet))
- self.failUnless(packet.dict is self.host.dict)
+ self.assertTrue(isinstance(packet, Packet))
+ self.assertTrue(packet.dict is self.host.dict)
self.assertEqual(packet.id, 15)
def testCreateAuthPacket(self):
packet = self.host.CreateAuthPacket(id=15)
- self.failUnless(isinstance(packet, AuthPacket))
- self.failUnless(packet.dict is self.host.dict)
+ self.assertTrue(isinstance(packet, AuthPacket))
+ self.assertTrue(packet.dict is self.host.dict)
self.assertEqual(packet.id, 15)
def testCreateAcctPacket(self):
packet = self.host.CreateAcctPacket(id=15)
- self.failUnless(isinstance(packet, AcctPacket))
- self.failUnless(packet.dict is self.host.dict)
+ self.assertTrue(isinstance(packet, AcctPacket))
+ self.assertTrue(packet.dict is self.host.dict)
self.assertEqual(packet.id, 15)
@@ -78,10 +78,10 @@ def setUp(self):
def testSendPacket(self):
self.host.SendPacket(self.fd, self.packet)
- self.failUnless(self.fd.data is self.packet.packet)
- self.failUnless(self.fd.target is self.packet.source)
+ self.assertTrue(self.fd.data is self.packet.packet)
+ self.assertTrue(self.fd.target is self.packet.source)
def testSendReplyPacket(self):
self.host.SendReplyPacket(self.fd, self.packet)
- self.failUnless(self.fd.data is self.packet.replypacket)
- self.failUnless(self.fd.target is self.packet.source)
+ self.assertTrue(self.fd.data is self.packet.replypacket)
+ self.assertTrue(self.fd.target is self.packet.source)
diff --git a/tests/testProxy.py b/tests/testProxy.py
index a672679..249237a 100644
--- a/tests/testProxy.py
+++ b/tests/testProxy.py
@@ -30,7 +30,7 @@ def tearDown(self):
def testProxyFd(self):
self.proxy._poll = MockPoll()
self.proxy._PrepareSockets()
- self.failUnless(isinstance(self.proxy._proxyfd, MockSocket))
+ self.assertTrue(isinstance(self.proxy._proxyfd, MockSocket))
self.assertEqual(list(self.proxy._fdmap.keys()), [1])
self.assertEqual(self.proxy._poll.registry,
{1: select.POLLIN | select.POLLPRI | select.POLLERR})
@@ -50,7 +50,7 @@ def testHandleProxyPacketUnknownHost(self):
try:
self.proxy._HandleProxyPacket(self.packet)
except ServerPacketError as e:
- self.failUnless('unknown host' in str(e))
+ self.assertTrue('unknown host' in str(e))
else:
self.fail()
@@ -63,7 +63,7 @@ def testHandleProxyPacketHandlesWrongPacket(self):
try:
self.proxy._HandleProxyPacket(self.packet)
except ServerPacketError as e:
- self.failUnless('non-response' in str(e))
+ self.assertTrue('non-response' in str(e))
else:
self.fail()
diff --git a/tests/testServer.py b/tests/testServer.py
index 2571fa3..ac8a940 100644
--- a/tests/testServer.py
+++ b/tests/testServer.py
@@ -110,10 +110,10 @@ def gen(data):
fd = MockFd()
fd.source = object()
pkt = self.server._GrabPacket(gen, fd)
- self.failUnless(isinstance(pkt, TrivialObject))
- self.failUnless(pkt.fd is fd)
- self.failUnless(pkt.source is fd.source)
- self.failUnless(pkt.data is fd.data)
+ self.assertTrue(isinstance(pkt, TrivialObject))
+ self.assertTrue(pkt.fd is fd)
+ self.assertTrue(pkt.source is fd.source)
+ self.assertTrue(pkt.data is fd.data)
def testPrepareSocketNoFds(self):
self.server._poll = MockPoll()
@@ -160,7 +160,7 @@ def testHandleAuthPacketUnknownHost(self):
try:
self.server._HandleAuthPacket(self.packet)
except ServerPacketError as e:
- self.failUnless('unknown host' in str(e))
+ self.assertTrue('unknown host' in str(e))
else:
self.fail()
@@ -169,7 +169,7 @@ def testHandleAuthPacketWrongPort(self):
try:
self.server._HandleAuthPacket(self.packet)
except ServerPacketError as e:
- self.failUnless('port' in str(e))
+ self.assertTrue('port' in str(e))
else:
self.fail()
@@ -180,7 +180,7 @@ def HandleAuthPacket(self, pkt):
Server.HandleAuthPacket = HandleAuthPacket
self.server._HandleAuthPacket(self.packet)
- self.failUnless(self.server.handled is self.packet)
+ self.assertTrue(self.server.handled is self.packet)
Server.HandleAuthPacket = hap
@@ -199,7 +199,7 @@ def testHandleAcctPacketUnknownHost(self):
try:
self.server._HandleAcctPacket(self.packet)
except ServerPacketError as e:
- self.failUnless('unknown host' in str(e))
+ self.assertTrue('unknown host' in str(e))
else:
self.fail()
@@ -208,7 +208,7 @@ def testHandleAcctPacketWrongPort(self):
try:
self.server._HandleAcctPacket(self.packet)
except ServerPacketError as e:
- self.failUnless('port' in str(e))
+ self.assertTrue('port' in str(e))
else:
self.fail()
@@ -219,7 +219,7 @@ def HandleAcctPacket(self, pkt):
Server.HandleAcctPacket = HandleAcctPacket
self.server._HandleAcctPacket(self.packet)
- self.failUnless(self.server.handled is self.packet)
+ self.assertTrue(self.server.handled is self.packet)
Server.HandleAcctPacket = hap
@@ -242,8 +242,8 @@ def CreateReply(self, **kw):
reply = self.server.CreateReplyPacket(TrivialPacket(),
one='one', two='two')
- self.failUnless(isinstance(reply, TrivialObject))
- self.failUnless(reply.source is TrivialPacket.source)
+ self.assertTrue(isinstance(reply, TrivialObject))
+ self.assertTrue(reply.source is TrivialPacket.source)
self.assertEqual(reply.kw, dict(one='one', two='two'))
def testAuthProcessInput(self):
@@ -285,8 +285,8 @@ def testRunInitializes(self):
MockClassMethod(Server, '_PrepareSockets')
self.assertRaises(MockFinished, self.server.Run)
self.assertEqual(self.server.called, [('_PrepareSockets', (), {})])
- self.failUnless(isinstance(self.server._fdmap, dict))
- self.failUnless(isinstance(self.server._poll, MockPoll))
+ self.assertTrue(isinstance(self.server._fdmap, dict))
+ self.assertTrue(isinstance(self.server._poll, MockPoll))
def testRunIgnoresPollErrors(self):
self.server.authfds = [MockFd()]
