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

C++17 and Python 3.12 #46

Open
sarnold opened this issue Sep 6, 2024 · 4 comments
Open

C++17 and Python 3.12 #46

sarnold opened this issue Sep 6, 2024 · 4 comments

Comments

@sarnold
Copy link

sarnold commented Sep 6, 2024

There's some good news:

  • changing cmake to c++17 and extensions ON seems to work fine
    and some not-so-good news:
  • SyntaxWarnings in py 3.12
=============================== warnings summary ===============================
tests/test_charliterals.txt::test_charliterals.txt
  <doctest test_charliterals.txt[12]>:1: SyntaxWarning: invalid escape sequence '\9'
tests/test_charliterals.txt::test_charliterals.txt
tests/test_charliterals.txt::test_charliterals.txt
  <unknown>:1: SyntaxWarning: invalid escape sequence '\9'
tests/test_charliterals.txt::test_charliterals.txt
  <doctest test_charliterals.txt[19]>:1: SyntaxWarning: invalid escape sequence '\9'
tests/test_search.txt::test_search.txt
  <doctest test_search.txt[4]>:1: SyntaxWarning: invalid escape sequence '\d'
tests/test_search.txt::test_search.txt
  <doctest test_search.txt[5]>:1: SyntaxWarning: invalid escape sequence '\d'
tests/test_sub.txt::test_sub.txt
  <doctest test_sub.txt[7]>:1: SyntaxWarning: invalid escape sequence '\('
-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
======================== 74 passed, 7 warnings in 1.52s ========================
@sarnold
Copy link
Author

sarnold commented Sep 7, 2024

@andreasvc fixes in #47

@tyteen4a03
Copy link

@sarnold FYI I am accepting PRs for my repo: https://github.com/tyteen4a03/pyre2

@andreasvc
Copy link
Owner

@tyteen4a03 this repo is also accepting PRs.

@tyteen4a03
Copy link

@tyteen4a03 this repo is also accepting PRs.

Awesome! Hope to see a new version from this repo soon so I don't have to maintain my fork :D

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

3 participants