Closed ManuelBlanc closed 8 years ago
gjulianm
commented
8 years ago Yo lo veo perfecto, sobre todo lo de no tener que instalar los paquetes :) Los scripts de generación de Dropbox no se rompen, utiliza un comando distinto y no los .sh del repositorio (como referencia, el autocompilador está en este otro repo, un poco chapucero).
Si al resto le parece bien (o no responde, que tomaremos como que les parece bien), hacemos merge.
EduardoMiravalls
commented
8 years ago Tiene buena pinta, pero veo algunas cosillas que opino que se deberían arreglar antes de aceptarla.
He puesto comentarios en el commit.
ManuelBlanc
commented
8 years ago @EduardoMiravalls: Primero de todo, gracias por revisar el codigo! :-) Los problemas que has encontrado deberian estar resueltos en el commit 1bbb930.
Respondiendo a las preguntas:
¿Qué version de coreutils usas?
Suelo usar OSX, que trae coreutils derivados de los de FreeBSD. Para la proxima, intentare asegurarme de que tambien funciona con las herramientas de GNU.
¿Has probado con asignaturas con espacios?
Si, parte de las pruebas que hice fueron con asignaturas con espacios.
Mirando el mensaje de error mas a fondo, parece que se queja cuando $TEXINPUTS no esta definida.
He añadido una comprobacion, lo he probado y deberia estar solucionado el error que te daba.
De todos modos, si se me ha escapado algo, comentalo y lo arreglare en cuanto pueda.
EduardoMiravalls
commented
8 years ago Genial, muchas gracias. Ahora si funciona muy bien. Lo único que veo, es que te has dejado un warning. Se ve que el find de GNU es un poco quisquilloso con el orden de sus argumentos y me imprime un mensaje:
$ ./gestor.sh limpiar EDP/
find: warning: you have specified the -maxdepth option after a non-option argument -iname, but
options are not positional (-maxdepth affects tests specified before it as well as those specified
after it). Please specify options before other arguments.Es el find de la línea 49, se ve que tienes que invertir el orden de "-iname" y "-maxdepth".
gjulianm
commented
8 years ago He arreglado el find que decía Edu y hecho el merge, muchas gracias a los dos :)
Los cambios introducidos en estos commits:
compilar.sh,create_subjectypackengestor.shlatexmkrc.pl)El fichero de opciones cambia (entre otras) las siguientes variables de entorno:
TEXINPUTS: Donde busca LaTeX los paquetes. Hace que no haga falta instalar los paquetes.max_print_line: Para que LaTeX imprima mas caracteres por linea.Me imagino que estos cambios romperan los scripts que generan los PDFs de Dropbox. Puedo ayudar a cambiar los scripts para hacerlo compatible y que se pueda aceptar este pull request.
Un saludo.