Page MenuHomeFreeBSD

tests/sys/opencrypto: use python3
ClosedPublic

Authored by asomers on Jul 16 2020, 3:10 AM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 16 2024, 2:34 PM
Unknown Object (File)
Feb 7 2024, 1:15 AM
Unknown Object (File)
Dec 10 2023, 4:39 AM
Unknown Object (File)
Nov 27 2023, 11:59 PM
Unknown Object (File)
Sep 26 2023, 9:48 PM
Unknown Object (File)
Sep 23 2023, 9:56 AM
Unknown Object (File)
Aug 31 2023, 2:45 PM
Unknown Object (File)
Jul 10 2023, 12:03 PM
Subscribers

Details

Summary

tests/sys/opencrypto: use python3

python2 will be EOL soon

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 32354
Build 29835: arc lint + arc unit

Event Timeline

Looks good to me. I'm adding ngie as they did the port to Python 3. I have not run and verified that this works under Python 3, but fully support the move to 3.

It does look like some changes were made in the conversion that need to be addressed. Not all `` were converted to repr's, in the case of some of the unittest.skipIf. '%s' % strvar != '%s' % repr(strvar) in that if strvar has escape chars or other non-printables, they will corrupt the terminal or other damage whereas repr prevents that change.

Test fine from my side.

This revision is now accepted and ready to land.Jul 20 2020, 10:38 AM
This revision was automatically updated to reflect the committed changes.

This should work, for sure.

The removing from __future__ import print_function wasn't absolutely required, but it does clean up some of the move from python 2 to 3.