catchorg/Catch2

Incorrect trap code for iOS fails compilation

Open

#1870 aperta il 19 feb 2020

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)C++ (3228 fork)batch import
BugHelp wanted

Metriche repository

Star
 (20.381 star)
Metriche merge PR
 (Merge medio 4g 16h) (2 PR mergiate in 30 g)

Descrizione

Describe the bug v2.11.1 introduced support for breaking into the debugger on iOS and while it works with XCode 10 and 11, it fails with 8 and 9.

You can reproduce the issue by cloning sjson-cpp at this branch. Make sure to clone submodules as Catch2 is used as a submodule.

python make.py -clean -build -compiler ios -config Debug

The issue also happens with Release config. See here for example build output.

Expected behavior I know that these are older xcode versions but I am wondering if we could support them as 2.11.0 worked fine previously.

Reproduction steps Simply compiling Catch2 v2.11.1 on XCode 8 or 9 will fail.

Platform information:

  • OS: OS X 10.13.6
  • Compiler+version: Apple LLVM version 9.1.0 (clang-902.0.39.2)
  • Catch version: v2.11.1

Additional context

Guida contributor

Direzione di ricerca
Indaga sul codice di trap introdotto nella PR #1817. Controlla i controlli della versione del compilatore (ad es. clang major ) e assicura la compatibilità con Xcode 8/9 (clang basato su LLVM 9.1.0). Il codice che fallisce è probabilmente in un file di supporto del debugger specifico della piattaforma, come catch debugger.hpp. Considera di aggiungere la compilazione condizionale per le versioni più vecchie di clang.
Tech stack
cpp
Dominio
testing
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++preprocessor macros
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40