Skip to content
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

Closed
wants to merge 16 commits into from

Conversation

RaniAgus
Copy link
Contributor

@RaniAgus RaniAgus commented Sep 25, 2022

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 y make valgrind por si prefieren usar la consola para buildear.

image

image

@RaniAgus RaniAgus added the build label Sep 25, 2022
@gastonprieto
Copy link
Contributor

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

Comment on lines +1 to +13
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)
Copy link
Contributor

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?

Copy link
Contributor Author

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.

Ref: https://cmake.org/cmake/help/v3.7/command/file.html

@mesaglio
Copy link
Contributor

A mi en Mac no me levanto el devcontainer. Si alguien mas quiere validarrr.

@RaniAgus
Copy link
Contributor Author

RaniAgus commented Jan 24, 2023

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

@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 make test tiraba error. Igual no debería fallar ningún test.

A mi en Mac no me levanto el devcontainer. Si alguien mas quiere validarrr.

@mesaglio ¿Te aparece algún error al intentar levantar el container?

@RaniAgus RaniAgus requested a review from gastonprieto January 24, 2023 17:00
@gastonprieto
Copy link
Contributor

gastonprieto commented Jan 26, 2023

@RaniAgus ahi me funciono bien :)

@RaniAgus RaniAgus closed this Dec 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Soportar la instalación desde el zip
3 participants