Linker.cmake should use target_link_options instead of target_compile_options
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 1/5
- Tiempo estimado
- Menos de una hora
- Aptitud para principiantes
- 58/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- cmake
- Área
- build-system
Línea de trabajo
Abre cmake/Linker.cmake en la línea 29 y compara la llamada actual a target_compile_options con el comando target_link_options descrito en el issue. Se considera terminado cuando los flags específicos del linker se aplican en el momento del enlazado en lugar de en el momento de la compilación, evitando la advertencia de argumento no utilizado informada en las configuraciones afectadas.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
From my understanding target_compile_options can be replaced by the more appropriate target_link_options.
Motivation
Functionally there's no difference, but adding linker-specific flags to the compilation step will raise warnings e.g. clang: error: argument unused during compilation: '-fuse-ld=mold' [-Werror,-Wunused-command-line-argument] in certain setups.
Example from CI in a project of mine. Treating warnings as errors means this harmless warning caused CI to fail.
Using target_link_options resolved it.
Thank you for sharing and maintaining this template project, it is a tremendous help!
- Lenguaje dominante
- CMake
- Estrellas
- 1.8k
- Forks
- 205
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Sin guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de cpp-best-practices/cmake_template
-
Put back `-Wsign-conversion` for gcc in CompilerWarnings.cmakePosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 Menos de una hora Aptitud para principiantes 85/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
-
Enable sanitizers globallyAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
cpp-best-practices/cmake_template#140 · 2 comentarios ·
-
From-scratch clone/build fails on `src/ftxui_sample/CMakeFiles/intro.dir/main.cpp.obj`Posiblemente ocupada @leftibot la tomó hace 195 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 48/100
cpp-best-practices/cmake_template#139 · 2 comentarios ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
cpp-best-practices/cmake_template#91 · 2 comentarios ·
Todos los issues de cpp-best-practices/cmake_template
Issues similares
-
Font inlining cache is not created or reusedPosiblemente ocupada @nagahara-naoki la tomó hoy. Abiertoarea: @angular/build
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
angular/angular-cli#34719 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
osaurus-ai/osaurus#3422 ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
foundation/foundation-emails#2127 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
mw00/project-maya#983 · 1 comentario ·
Los mantenedores suelen responder en 2 días