Soportar la instalación desde el zip

[mgarciaisaia]

Actualmente el make rompe si el proyecto se bajó usando Download as zip porque no tiene información del submódulo git de las cspec.

Deberíamos solucionar ese problema no-se-cómo :)

[mdumrauf]

Qué tal un if preguntando por la cantidad de elementos de (ls -1 | wc -l) haga un wget para bajar cspec?

[mgarciaisaia]

En realidad deberíamos listar cspec como una dependencia, y ya. Ningún proyecto te dice cómo instalar sus dependencias, se asume que sabés hacerlo.

Lo que sí habría que hacer es armar un método muy simple de instalar cspec (buildear .debs, quizá; hacer una ppa...), y para SO incluir cspec ya instalado en las VMs de la cátedra (que justo hicimos unas nuevas este cuatrimestre y no lo pusimos, shame on me).

Y hacer que por default la instalación de las commons no dependa 100% de los tests. Que si puede los corra, pero si no encuentra cspec que tire un warning y nada más. Para que haya la menor fricción posible, digo.

[mdumrauf]

Me resultó raro que la incluyeran como un submodule, pero si habían elegido esa manera imaginé que tocar el Makefile era la más simple.
@gastonprieto en algún momento había empezado esto https://github.com/gastonprieto/caven, pero no prosperó :P

[RaniAgus]

Encontré que se puede solucionar si agregamos CMake a la compilación de los tests. Tiene un submódulo que se encarga de clonar el repo y buildearlo:

so-commons-library/tests/unit-tests/CMakeLists.txt

Lines 1 to 6 in 9776ea2

	include(FetchContent)
	FetchContent_Declare(cspecs GIT_REPOSITORY https://github.com/mumuki/cspec)
	FetchContent_MakeAvailable(cspecs)
	include_directories(${cspecs_SOURCE_DIR})
	link_libraries(cspecs)