-
Notifications
You must be signed in to change notification settings - Fork 172
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
Environment: VSCode con CMake y Docker #167
Conversation
Lo veo piola, lo unico que note, que no se si deberia pasar es que si levanto el dev container en windows con la imagen de ubuntu me fallan un par de tests |
add_library(commons SHARED | ||
commons/bitarray.c | ||
commons/config.c | ||
commons/error.c | ||
commons/log.c | ||
commons/memory.c | ||
commons/process.c | ||
commons/string.c | ||
commons/temporal.c | ||
commons/txt.c | ||
commons/collections/dictionary.c | ||
commons/collections/list.c | ||
commons/collections/queue.c) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Esto no se puede hacer de manera generica con el directorio commons
y no tener que sumar cada file.c
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Por poder se puede, pero según la docu de CMake no es recomendable hacerlo:
Note We do not recommend using GLOB to collect a list of source files from your source tree. If no CMakeLists.txt file changes when a source is added or removed then the generated build system cannot know when to ask CMake to regenerate.
A mi en Mac no me levanto el devcontainer. Si alguien mas quiere validarrr. |
@gastonprieto Corregido (o eso creo), se ve que el plugin por defecto usa Ninja para buildear en vez de Unix Makefiles, por lo que al hacer
@mesaglio ¿Te aparece algún error al intentar levantar el container? |
…ibrary into feature/cmake_vscode_environment
@RaniAgus ahi me funciono bien :) |
cc: @mesaglio @gastonprieto @mgarciaisaia
Cambié a CMake porque creo que como mantainers nos va a ser más sencillo importar todos los proyectos desde CLion o VSCode, ya que se levantan todos los targets de una para poder compilarlos (adjunto capturas). También incluí un
.devcontainer
para levantar el entorno incluyendo los plugins básicos de CMake.De paso, esto permitiría instalar las commons desde zip (resolves #49) ya que el CMakeLists.txt del proyecto de unit tests lo incluye usando
FetchContent
.Dejé un makefile en el root para que la forma de instalarlas siga siendo igual que siempre y manteniendo los targets
make clean
,make test
ymake valgrind
por si prefieren usar la consola para buildear.