google/googletest

[Bug]: CMake FetchContent_* does not work with GTest as dependency name on Windows

Open

#4 384 ouverte le 3 oct. 2023

Voir sur GitHub
 (7 commentaires) (0 réactions) (0 assignés)C++ (10 772 forks)batch import
help wanted

Métriques du dépôt

Stars
 (38 603 stars)
Métriques de merge PR
 (Merge moyen 7j 8h) (1 PR mergée en 30 j)

Description

Describe the issue

CMake FetchContent_* does not work with GTest as dependency name. If the dependency name is changed to googletest or to foobar everything works as expected. This is very unfortunate because GTets is used in the find_package command of CMake and if this fails the FetchContent mechanism can be used as fallback. But not on windows as you can see here:

https://github.com/ObiWahn/googletest-CMake-FetchContent/pull/1

Steps to reproduce the problem

https://github.com/ObiWahn/googletest-CMake-FetchContent/pull/1/files

What version of GoogleTest are you using?

v.1.14.0

What operating system and version are you using?

windows

What compiler and version are you using?

msvc / clang

What build system are you using?

cmake 3.27.6

Additional context

No response

Guide contributeur

Direction de recherche
Examinez le comportement de CMake FetchContent lorsque le nom de la dependance est 'GTest' sur Windows ; probablement un probleme d'insensibilite a la casse ou de nom reserve. Verifiez le code source de CMake ou le CMakeLists.txt de GoogleTest pour tout traitement special.
Stack technique
Aucun
Domaine
build system
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitCMake
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70