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

[smart_holder] type_caster_odr_guard MSVC 193732825 C++17 windows-2022 is failing for unknown reasons. #4918

Merged
merged 2 commits into from
Nov 3, 2023

Conversation

rwgk
Copy link
Collaborator

@rwgk rwgk commented Nov 3, 2023

Description

The condition added here is based on close inspection of the GHA logs for #4914.

$ python3 what_fails.py *.txt | grep 'MSVC 193732825 C++17'
MSVC 193732825 C++17 False 0___3.10___windows-latest___clang-latest.txt
MSVC 193732825 C++17 True 10___3.10___windows-2022___x64.txt
MSVC 193732825 C++17 True 11___3.11___windows-2022___x64.txt
MSVC 193732825 C++17 True 12___3.12___windows-2022___x64.txt
MSVC 193732825 C++17 True 14___pypy-3.9___windows-2022___x64.txt
$ python3 what_fails.py *.txt | grep -v 'MSVC 193732825 C++17' | grep MSVC
MSVC 192930152 C++14 False 0___3.6___MSVC_2019___x86.txt
MSVC 192930152 C++17 False 0___3.8___MSVC_2019__Debug____x86_-DCMAKE_CXX_STANDARD=17.txt
MSVC 193732825 C++20 False 0___3.9___MSVC_2022_C++20___x64.txt
MSVC 193732825 C++20 False 0___3.9___MSVC_2022_C++20___x64.txt
MSVC 193732825 C++14 False 10___3.10___windows-2022___x64.txt
MSVC 193732825 C++14 False 11___3.11___windows-2022___x64.txt
MSVC 193732825 C++14 False 12___3.12___windows-2022___x64.txt
MSVC 192930152 C++14 False 1___3.7___MSVC_2019___x86_-DCMAKE_CXX_STANDARD=14.txt
MSVC 192930152 C++20 False 1___3.9___MSVC_2019__Debug____x86_-DCMAKE_CXX_STANDARD=20.txt
MSVC 193532217 C++14 False 13___pypy-3.8___windows-2022___x64.txt
MSVC 193532217 C++17 False 13___pypy-3.8___windows-2022___x64.txt
MSVC 193532217 C++17 False 13___pypy-3.8___windows-2022___x64.txt
MSVC 193732825 C++14 False 14___pypy-3.9___windows-2022___x64.txt
MSVC 193532217 C++14 False 15___pypy-3.10___windows-2022___x64.txt
MSVC 193532217 C++17 False 15___pypy-3.10___windows-2022___x64.txt
MSVC 193532217 C++17 False 15___pypy-3.10___windows-2022___x64.txt
MSVC 192930152 C++17 False 2___3.8___MSVC_2019___x86_-DCMAKE_CXX_STANDARD=17.txt
MSVC 192930152 C++14 False 24___3.6___windows-2019___x64_-DPYBIND11_FINDPYTHON=ON.txt
MSVC 192930152 C++17 False 24___3.6___windows-2019___x64_-DPYBIND11_FINDPYTHON=ON.txt
MSVC 192930152 C++17 False 24___3.6___windows-2019___x64_-DPYBIND11_FINDPYTHON=ON.txt
MSVC 192930152 C++14 False 25___3.9___windows-2019___x64.txt
MSVC 192930152 C++17 False 25___3.9___windows-2019___x64.txt
MSVC 192930152 C++17 False 25___3.9___windows-2019___x64.txt
MSVC 192930152 C++20 False 3___3.9___MSVC_2019___x86_-DCMAKE_CXX_STANDARD=20.txt
MSVC 193532217 C++14 False 8___3.6___windows-2022___x64.txt
MSVC 193532217 C++17 False 8___3.6___windows-2022___x64.txt
MSVC 193532217 C++17 False 8___3.6___windows-2022___x64.txt
MSVC 193532217 C++14 False 9___3.9___windows-2022___x64.txt

what_fails.py:

"""."""

import sys


def pin_point(filename):
  """."""
  last_cxx_std = None
  found_failed = None
  def transition():
    if last_cxx_std is None:
      assert found_failed is None
    else:
      assert found_failed is not None
      print(last_cxx_std, found_failed, filename)
  for line in open(filename).read().splitlines():
    pat = " C++ Info: "
    ix = line.find(pat)
    if ix >= 0:
      jx = line.find(" __pybind11_internals_")
      assert jx > ix
      transition()
      last_cxx_std = line[ix + len(pat) : jx].strip()
      found_failed = False
      continue
    ix = line.find("FAILED")
    jx = line.find("test_type_caster_odr_guard_")
    if ix >= 0 and jx > ix:
      found_failed = True
      continue
  transition()


def run(args):
  for filename in args:
    pin_point(filename)


if __name__ == "__main__":
  run(args=sys.argv[1:])

Suggested changelog entry:

@rwgk rwgk changed the title [smart_holder] type_caster_odr_guard MSVC 193732825 C++17 windows-2020 is failing for unknown reasons. [smart_holder] type_caster_odr_guard MSVC 193732825 C++17 windows-2022 is failing for unknown reasons. Nov 3, 2023
@rwgk rwgk force-pushed the type_caster_odr_guard_msvc_193732825 branch from 10b9e21 to d715b50 Compare November 3, 2023 05:19
@rwgk rwgk marked this pull request as ready for review November 3, 2023 06:03
@rwgk rwgk merged commit edfaaed into pybind:smart_holder Nov 3, 2023
152 checks passed
@rwgk rwgk deleted the type_caster_odr_guard_msvc_193732825 branch November 3, 2023 06:03
@github-actions github-actions bot added the needs changelog Possibly needs a changelog entry label Nov 3, 2023
@henryiii henryiii removed the needs changelog Possibly needs a changelog entry label Nov 15, 2023
rwgk pushed a commit to rwgk/pybind11clif that referenced this pull request Dec 2, 2023
rwgk pushed a commit to rwgk/pybind11clif that referenced this pull request Dec 2, 2023
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

Successfully merging this pull request may close these issues.

2 participants