python-zipp

Edit Package python-zipp
No description set
Refresh
Refresh
Source Files (show unmerged sources)
Filename Size Changed
_multibuild 0000000053 53 Bytes
python-zipp.changes 0000009421 9.2 KB
python-zipp.spec 0000002526 2.47 KB
zipp-3.21.0.tar.gz 0000024545 24 KB
Latest Revision
buildservice-autocommit accepted request 1225379 from Dirk Mueller's avatar Dirk Mueller (dirkmueller) (revision 49)
baserev update by copy to link target
Comments 8

Manuel Jacob's avatar

Why is it still built on Leap 15.x although skip_python36 is 1?


Matej Cepl's avatar

The symbol in SLE-15 (and thus Leap 15.*) is python3 not python36.


Manuel Jacob's avatar

With this package, I have the same problem as described here.

I looked at upstream and they already removed some Python 3.6 compatibility code: https://github.com/jaraco/zipp/commit/c0802277a1b9d799118751a4f032b495172ef1f5

Starting from CPython 3.6, dicts are ordered, although it’s part of the language specification only from Python 3.7. PyPy had ordered dicts for some longer time. So in practice, it won’t really make a difference. Of course, if more compatibility code is removed in the future, we’ll run into problems. Although not a perfect solution, we could copy the approach from https://build.opensuse.org/request/show/984778 for now.

What do you think?


Matej Cepl's avatar

If the test suite passes, I would agree, go for it. If necessary, add a conditional BR on python-ordereddict, we should have it somewhere (probably not in Factory)


Manuel Jacob's avatar

python-ordereddict is for Python < 2.7, we will be fine without it. ;)

So I’ve checked that the current version of zipp works on Python 3.6. However, this has to be checked everytime we upgrade it. Would it be sufficient to hope that people check it on every upgrade? Alternatively, it could fix the required python metadata in the package if and only if Python == 3.6 and %version < 3.8.0. Then, on upgrade, build would fail on Python 3.6 until the check is updated, too. Is there a macro to do version comparisons?


Matej Cepl's avatar

That’s why I insist on running test suites whenever we can … then we have at least some automatic checking every time anything changes.


Manuel Jacob's avatar

I agree that testing is important and should be done for each package.

However, the question is whether we consider running the test suite as sufficient or we still want people to manually check that no incompatibility was introduced. I think it wouldn’t be that unusual to rely on some implementation detail of a particular Python version without writing a test for it.


Matej Cepl's avatar

Running the test suite is probably the best (and only) thing we have, so yes, it probably have to be sufficient. If we find a problem, let's move it upstream and get more tests for it.

openSUSE Build Service is sponsored by