sisoputnfrba / so-commons-library

TADs de uso comun en aplicaciones desarrolladas en C
http://sisoputnfrba.github.io/so-commons-library/
GNU General Public License v3.0
106 stars 175 forks source link

Environment: VSCode con CMake y Docker #167

Closed RaniAgus closed 9 months ago

RaniAgus commented 1 year ago

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

gastonprieto commented 1 year ago

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

mesaglio commented 1 year ago

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

RaniAgus commented 1 year ago

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?

gastonprieto commented 1 year ago

@RaniAgus ahi me funciono bien :)