Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deprecation and syntax warnings in Python 3.8 #1554

Open
tirkarthi opened this issue May 3, 2020 · 1 comment
Open

Deprecation and syntax warnings in Python 3.8 #1554

tirkarthi opened this issue May 3, 2020 · 1 comment

Comments

@tirkarthi
Copy link

  • 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+)')
@intgr
Copy link

intgr commented Mar 29, 2023

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants