f5-common-python icon indicating copy to clipboard operation
f5-common-python copied to clipboard

Deprecation and syntax warnings in Python 3.8

Open tirkarthi opened this issue 5 years ago • 1 comments

  • Deprecation warning due to invalid escape sequences. Using raw strings or escaping them again helps in resolving this.
  • Syntax warnings due to comparison of literals using is.

Check https://github.com/asottile/pyupgrade/ for automatic fix of this.

find . -iname '*.py' | grep -Ev 'rdf4|doc|benchmark|tool' | xargs -P4 -I{} python3.8 -Wall -m py_compile {}
./f5/bigip/tm/sys/test/functional/test_log_config.py:356: SyntaxWarning: "is" with a literal. Did you mean "=="?
  assert dest1.port is 80
./f5/bigip/tm/sys/test/functional/test_application.py:234: SyntaxWarning: "is" with a literal. Did you mean "=="?
  assert len(all_services) is 1
./f5/bigip/tm/cm/test/functional/test_traffic_group.py:28: SyntaxWarning: "is not" with a literal. Did you mean "!="?
  if err.response.status_code is not 404:
./f5/bigip/tm/cm/test/functional/test_device.py:29: SyntaxWarning: "is not" with a literal. Did you mean "!="?
  if err.response.status_code is not 404:
./f5/bigip/tm/net/test/functional/test_vlan.py:79: SyntaxWarning: "is" with a literal. Did you mean "=="?
  assert len(ifcs) is 1
./f5/bigip/tm/net/test/functional/test_vlan.py:83: SyntaxWarning: "is" with a literal. Did you mean "=="?
  assert len(ifcs) is 0
./f5/utils/iapp_parser.py:123: SyntaxWarning: "is" with a literal. Did you mean "=="?
  if brace_count is 0:
./f5/utils/iapp_parser.py:126: SyntaxWarning: "is not" with a literal. Did you mean "!="?
  if brace_count is not 0:
./f5-sdk-dist/Docker/redhat/install_test/fetch_and_install_deps.py:52: DeprecationWarning: invalid escape sequence \w
  dep_match_re = re.compile('^([\w\-\d]+)\s?([<=>]+)\s?(\S+)')
./f5-sdk-dist/Docker/redhat/install_test/fetch_and_install_deps.py:53: DeprecationWarning: invalid escape sequence \-
  f5_dependency_re = re.compile('(f5[\-_].+)')
./f5-sdk-dist/Docker/redhat/install_test/fetch_and_install_deps.py:231: DeprecationWarning: invalid escape sequence \d
  re.sub('-\d+', '', self.version)
./f5-sdk-dist/Docker/redhat/install_test/fetch_and_install_deps.py:270: DeprecationWarning: invalid escape sequence \S
  mod_read = re.compile('^([^<=>]+)([<=>]+)(\S+)')
./f5-sdk-dist/Docker/redhat/install_test/fetch_and_install_deps.py:271: DeprecationWarning: invalid escape sequence \s
  skip_re = re.compile('^\s*#')
./f5-sdk-dist/Docker/redhat/install_test/fetch_and_install_deps.py:379: DeprecationWarning: invalid escape sequence \d
  version_re = re.compile('(\d+)\.(\d+)\.(\d+)')
./f5-sdk-dist/Docker/redhat/7/build-rpm.py:103: DeprecationWarning: invalid escape sequence \d
  breakout_re = re.compile('([^<=>]+)([<=>]+)([\d]\S+)')
./f5-sdk-dist/Docker/ubuntu/install_test/fetch_and_install_deps.py:51: DeprecationWarning: invalid escape sequence \s
  dep_match_re = re.compile('^\s*([\w\-]+)\s\(([=<>]+)\s([^\)]+)')
./f5-sdk-dist/Docker/ubuntu/install_test/fetch_and_install_deps.py:52: DeprecationWarning: invalid escape sequence \-
  f5_dependency_re = re.compile('(f5[\-_].+)')
./f5-sdk-dist/Docker/ubuntu/install_test/fetch_and_install_deps.py:230: DeprecationWarning: invalid escape sequence \d
  re.sub('-\d+', '', self.version)
./f5-sdk-dist/Docker/ubuntu/install_test/fetch_and_install_deps.py:372: DeprecationWarning: invalid escape sequence \d
  version_re = re.compile('(\d+)\.(\d+)\.(\d+)')
./f5-sdk-dist/Docker/ubuntu/install_test/fetch_and_install_deps.py:468: DeprecationWarning: invalid escape sequence \S
  mod_read = re.compile('^([^<=>]+)([<=>]+)(\S+)')
./f5-sdk-dist/Docker/ubuntu/install_test/fetch_and_install_deps.py:469: DeprecationWarning: invalid escape sequence \s
  skip_re = re.compile('^\s*#')
./f5-sdk-dist/scripts/install_test.py:181: DeprecationWarning: invalid escape sequence \d
  pkg_re = re.compile('el(\d+)\.noarch')
./f5-sdk-dist/scripts/install_test.py:178: DeprecationWarning: invalid escape sequence \d
  pkg_re = re.compile('_(\d+)_all\.deb')
./f5-sdk-dist/scripts/install_test.py:251: DeprecationWarning: invalid escape sequence \/
  re.sub('.+/([^\/]+-dist/.+)', '/var/wdir/\g<1>', pkg.pkg))
./f5-sdk-dist/scripts/install_test.py:251: DeprecationWarning: invalid escape sequence \g
  re.sub('.+/([^\/]+-dist/.+)', '/var/wdir/\g<1>', pkg.pkg))
./f5-sdk-dist/scripts/construct_setups.py:76: DeprecationWarning: invalid escape sequence \W
  break_re = re.compile('^[^#\W]')
./f5-sdk-dist/scripts/construct_setups.py:77: DeprecationWarning: invalid escape sequence \s
  req_re = re.compile('^Depends:|^requires\s*=\s*')
./f5-sdk-dist/scripts/get-version-release.py:22: DeprecationWarning: invalid escape sequence \d
  releaseTagRe = re.compile("^v(\d+\.\d+\.\d+((a|rc|RC)\d+)?$)")
./f5-sdk-dist/scripts/get-version-release.py:23: DeprecationWarning: invalid escape sequence \d
  gitDescribeRe = re.compile("^v(\d+\.\d+\.\d+(?:a\d+)?)-(\d+)-g(.*)$")
./f5-sdk-dist/build_pkgs.py:303: DeprecationWarning: invalid escape sequence \S
  arg_re = re.compile('([^;]+);(\S+)')

tirkarthi avatar May 03 '20 12:05 tirkarthi

The SyntaxWarnings in non-test code would be fixed in #1612.

intgr avatar Mar 29 '23 12:03 intgr