Skip to content

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
@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 added a commit to rwgk/pybind11clif that referenced this pull request Dec 2, 2023
rwgk added 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