Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package python-psutil for openSUSE:Factory 
checked in at 2022-08-04 13:23:05
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/python-psutil (Old)
 and      /work/SRC/openSUSE:Factory/.python-psutil.new.1521 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "python-psutil"

Thu Aug  4 13:23:05 2022 rev:67 rq:989376 version:5.9.1

Changes:
--------
--- /work/SRC/openSUSE:Factory/python-psutil/python-psutil.changes      
2022-06-25 10:23:48.390634716 +0200
+++ /work/SRC/openSUSE:Factory/.python-psutil.new.1521/python-psutil.changes    
2022-08-04 13:23:14.652461337 +0200
@@ -1,0 +2,8 @@
+Fri Jul 15 07:59:26 UTC 2022 - Ben Greiner <c...@bnavigator.de>
+
+- Fix tests: setuptools changed the builddir library path and does
+  not find the module from it. Use the installed platlib instead
+  and exclude psutil.tests only later.
+- Refresh skip-obs.patch
+
+-------------------------------------------------------------------

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ python-psutil.spec ++++++
--- /var/tmp/diff_new_pack.ub6mlw/_old  2022-08-04 13:23:15.120462665 +0200
+++ /var/tmp/diff_new_pack.ub6mlw/_new  2022-08-04 13:23:15.132462700 +0200
@@ -78,7 +78,6 @@
 %{python_expand mkdir -p %{buildroot}%{_docdir}/%{$python_prefix}-psutil
 cp -r scripts %{buildroot}%{_docdir}/%{$python_prefix}-psutil/
 find %{buildroot}%{_docdir}/%{$python_prefix}-psutil/scripts/ -type f -name 
"*.py" -exec sed -i "s|#!%{_bindir}/env python.*|#!%{__$python}|" {} \;
-rm -rf %{buildroot}%{$python_sitearch}/psutil/tests
 %fdupes %{buildroot}%{_docdir}/%{$python_prefix}-psutil/
 %fdupes %{buildroot}%{$python_sitearch}
 }
@@ -87,13 +86,12 @@
 %check
 export LANG=en_US.UTF-8
 export PSUTIL_TESTING=1
-export TRAVIS=1
-march=`python3 -c "import platform ; print(platform.machine())"`
-
-# Note test_fetch_all is a bit flaky, occasionally failing
-%{python_expand export PYTHONPATH=build/lib.linux-${march}-%{$python_version}/
-$python -Wa psutil/tests/runner.py
-}
+export PSUTIL_DEBUG=1
+export PYTHONDONTRWRITEBYTECODE=1
+mkdir testd
+pushd testd
+%python_expand PYTHONPATH=%{buildroot}%{$python_sitearch} $python -Wa -m 
psutil.tests
+popd
 %endif
 
 %files %{python_files}
@@ -101,7 +99,7 @@
 %doc CREDITS HISTORY.rst README.rst
 %{_docdir}/%{python_prefix}-psutil/scripts/
 %{python_sitearch}/psutil/
-%{python_sitearch}/psutil/_psutil_*.so
-%{python_sitearch}/psutil-%{version}-py*.egg-info
+%exclude %{python_sitearch}/psutil/tests
+%{python_sitearch}/psutil-%{version}*-info
 
 %changelog

++++++ skip-obs.patch ++++++
--- /var/tmp/diff_new_pack.ub6mlw/_old  2022-08-04 13:23:15.164462790 +0200
+++ /var/tmp/diff_new_pack.ub6mlw/_new  2022-08-04 13:23:15.168462801 +0200
@@ -1,128 +1,136 @@
-Index: psutil-5.9.0/psutil/tests/test_misc.py
+Index: psutil-5.9.1/psutil/tests/test_misc.py
 ===================================================================
---- psutil-5.9.0.orig/psutil/tests/test_misc.py
-+++ psutil-5.9.0/psutil/tests/test_misc.py
-@@ -368,6 +368,7 @@ class TestMisc(PsutilTestCase):
+--- psutil-5.9.1.orig/psutil/tests/test_misc.py
++++ psutil-5.9.1/psutil/tests/test_misc.py
+@@ -252,6 +252,7 @@ class TestMisc(PsutilTestCase):
  
      # XXX: https://github.com/pypa/setuptools/pull/2896
      @unittest.skipIf(APPVEYOR, "temporarily disabled due to setuptools bug")
-+    @unittest.skipIf(True, 'does not work on obs')
++    @unittest.skip("Fails in OBS")
      def test_setup_script(self):
          setup_py = os.path.join(ROOT_DIR, 'setup.py')
          if CI_TESTING and not os.path.exists(setup_py):
-@@ -710,7 +711,7 @@ class TestScripts(PsutilTestCase):
+@@ -745,6 +746,7 @@ class TestScripts(PsutilTestCase):
              src = f.read()
          ast.parse(src)
  
--    def test_coverage(self):
-+    def _test_coverage(self):
++    @unittest.skip("Fails in OBS")
+     def test_coverage(self):
          # make sure all example scripts have a test method defined
          meths = dir(self)
-         for name in os.listdir(SCRIPTS_DIR):
-@@ -729,7 +730,7 @@ class TestScripts(PsutilTestCase):
+@@ -764,6 +766,7 @@ class TestScripts(PsutilTestCase):
                      if not stat.S_IXUSR & os.stat(path)[stat.ST_MODE]:
                          raise self.fail('%r is not executable' % path)
  
--    def test_disk_usage(self):
-+    def _test_disk_usage(self):
++    @unittest.skip("Fails in OBS")
+     def test_disk_usage(self):
          self.assert_stdout('disk_usage.py')
  
-     def test_free(self):
-Index: psutil-5.9.0/psutil/tests/test_linux.py
+Index: psutil-5.9.1/psutil/tests/test_linux.py
 ===================================================================
---- psutil-5.9.0.orig/psutil/tests/test_linux.py
-+++ psutil-5.9.0/psutil/tests/test_linux.py
-@@ -683,7 +683,7 @@ class TestSystemCPUCountLogical(PsutilTe
+--- psutil-5.9.1.orig/psutil/tests/test_linux.py
++++ psutil-5.9.1/psutil/tests/test_linux.py
+@@ -688,6 +688,7 @@ class TestSystemCPUCountLogical(PsutilTe
          self.assertEqual(psutil.cpu_count(logical=True), num)
  
      @unittest.skipIf(not which("lscpu"), "lscpu utility not available")
--    def test_against_lscpu(self):
-+    def _test_against_lscpu(self):
++    @unittest.skip("Fails in OBS")
+     def test_against_lscpu(self):
          out = sh("lscpu -p")
          num = len([x for x in out.split('\n') if not x.startswith('#')])
-         self.assertEqual(psutil.cpu_count(logical=True), num)
-@@ -726,7 +726,7 @@ class TestSystemCPUCountLogical(PsutilTe
+@@ -731,6 +732,7 @@ class TestSystemCPUCountLogical(PsutilTe
  class TestSystemCPUCountCores(PsutilTestCase):
  
      @unittest.skipIf(not which("lscpu"), "lscpu utility not available")
--    def test_against_lscpu(self):
-+    def _test_against_lscpu(self):
++    @unittest.skip("Fails in OBS")
+     def test_against_lscpu(self):
          out = sh("lscpu -p")
          core_ids = set()
-         for line in out.split('\n'):
-@@ -1072,7 +1072,7 @@ class TestSystemDiskPartitions(PsutilTes
+@@ -1077,6 +1079,7 @@ class TestSystemDiskPartitions(PsutilTes
  
      @unittest.skipIf(not hasattr(os, 'statvfs'), "os.statvfs() not available")
      @skip_on_not_implemented()
--    def test_against_df(self):
-+    def _test_against_df(self):
++    @unittest.skip("Fails in OBS")     
+     def test_against_df(self):
          # test psutil.disk_usage() and psutil.disk_partitions()
          # against "df -a"
-         def df(path):
-@@ -1246,7 +1246,7 @@ class TestSystemDiskIoCounters(PsutilTes
+@@ -1251,6 +1254,7 @@ class TestSystemDiskIoCounters(PsutilTes
                  self.assertEqual(ret.read_count, 1)
                  self.assertEqual(ret.write_count, 5)
  
--    def test_emulate_use_sysfs(self):
-+    def _test_emulate_use_sysfs(self):
++    @unittest.skip("Fails in OBS")
+     def test_emulate_use_sysfs(self):
          def exists(path):
              if path == '/proc/diskstats':
-                 return False
-Index: psutil-5.9.0/psutil/tests/test_posix.py
-===================================================================
---- psutil-5.9.0.orig/psutil/tests/test_posix.py
-+++ psutil-5.9.0/psutil/tests/test_posix.py
-@@ -377,7 +377,7 @@ class TestSystemAPIs(PsutilTestCase):
+@@ -1294,6 +1298,7 @@ class TestRootFsDeviceFinder(PsutilTestC
+         finder.ask_sys_class_block()
+ 
+     @unittest.skipIf(GITHUB_ACTIONS, "unsupported on GITHUB_ACTIONS")
++    @unittest.skip("Fails in OBS")
+     def test_comparisons(self):
+         finder = RootFsDeviceFinder()
+         self.assertIsNotNone(finder.find())
+@@ -1321,6 +1326,7 @@ class TestRootFsDeviceFinder(PsutilTestC
+         findmnt_value = sh("findmnt -o SOURCE -rn /")
+         self.assertEqual(psutil_value, findmnt_value)
+ 
++    @unittest.skip("Fails in OBS")
+     def test_disk_partitions_mocked(self):
+         with mock.patch(
+                 'psutil._pslinux.cext.disk_partitions',
+Index: psutil-5.9.1/psutil/tests/test_posix.py
+===================================================================
+--- psutil-5.9.1.orig/psutil/tests/test_posix.py
++++ psutil-5.9.1/psutil/tests/test_posix.py
+@@ -377,6 +377,7 @@ class TestSystemAPIs(PsutilTestCase):
      # AIX can return '-' in df output instead of numbers, e.g. for /proc
      @unittest.skipIf(AIX, "unreliable on AIX")
      @retry_on_failure()
--    def test_disk_usage(self):
-+    def _test_disk_usage(self):
++    @unittest.skip("Fails in OBS")
+     def test_disk_usage(self):
          def df(device):
              out = sh("df -k %s" % device).strip()
-             line = out.split('\n')[1]
-Index: psutil-5.9.0/psutil/tests/test_system.py
+Index: psutil-5.9.1/psutil/tests/test_system.py
 ===================================================================
---- psutil-5.9.0.orig/psutil/tests/test_system.py
-+++ psutil-5.9.0/psutil/tests/test_system.py
-@@ -200,6 +200,7 @@ class TestMiscAPIs(PsutilTestCase):
+--- psutil-5.9.1.orig/psutil/tests/test_system.py
++++ psutil-5.9.1/psutil/tests/test_system.py
+@@ -199,6 +199,7 @@ class TestMiscAPIs(PsutilTestCase):
          self.assertLess(bt, time.time())
  
      @unittest.skipIf(CI_TESTING and not psutil.users(), "unreliable on CI")
-+    @unittest.skipIf(True, "does not work on obs")
++    @unittest.skip("Fails in OBS")
      def test_users(self):
          users = psutil.users()
          self.assertNotEqual(users, [])
-@@ -580,7 +581,7 @@ class TestDiskAPIs(PsutilTestCase):
+@@ -579,6 +580,7 @@ class TestDiskAPIs(PsutilTestCase):
      def test_disk_usage_bytes(self):
          psutil.disk_usage(b'.')
  
--    def test_disk_partitions(self):
-+    def _test_disk_partitions(self):
++    @unittest.skip("Fails in OBS")
+     def test_disk_partitions(self):
          def check_ntuple(nt):
              self.assertIsInstance(nt.device, str)
-             self.assertIsInstance(nt.mountpoint, str)
-Index: psutil-5.9.0/psutil/tests/test_contracts.py
+Index: psutil-5.9.1/psutil/tests/test_contracts.py
 ===================================================================
---- psutil-5.9.0.orig/psutil/tests/test_contracts.py
-+++ psutil-5.9.0/psutil/tests/test_contracts.py
+--- psutil-5.9.1.orig/psutil/tests/test_contracts.py
++++ psutil-5.9.1/psutil/tests/test_contracts.py
 @@ -246,6 +246,7 @@ class TestSystemAPITypes(PsutilTestCase)
              self.assertIsInstance(k, str)
              self.assert_ntuple_of_nums(v, type_=(int, long))
  
-+    @unittest.skipIf(True, "does not work on obs")
++    @unittest.skip("Fails in OBS")
      def test_disk_partitions(self):
          # Duplicate of test_system.py. Keep it anyway.
          for disk in psutil.disk_partitions():
-Index: psutil-5.9.0/psutil/tests/test_process.py
+Index: psutil-5.9.1/psutil/tests/test_process.py
 ===================================================================
---- psutil-5.9.0.orig/psutil/tests/test_process.py
-+++ psutil-5.9.0/psutil/tests/test_process.py
+--- psutil-5.9.1.orig/psutil/tests/test_process.py
++++ psutil-5.9.1/psutil/tests/test_process.py
 @@ -343,6 +343,7 @@ class TestProcess(PsutilTestCase):
  
      @unittest.skipIf(not HAS_IONICE, "not supported")
      @unittest.skipIf(not LINUX, "linux only")
-+    @unittest.skipIf(True, "does not work reliably on obs")
++    @unittest.skip("Unreliable in OBS")
      def test_ionice_linux(self):
          p = psutil.Process()
          if not CI_TESTING:

Reply via email to