docs: agregar conclusiones y soporte de compilación en macOS - #49
Merged
Merged
Conversation
…ejorar manejo de dependencias en Fedora en macOs
Reviewer's GuideEl PR completa las conclusiones de Dayana, actualiza el historial y adapta los scripts de compilación para manejar dependencias específicas de Linux, instalaciones locales en macOS y apertura opcional del PDF; también evita versionar artefactos de LaTeX. Flow diagram for cross-platform report compilationflowchart TD
A["compilar.sh"] --> B{"Argumento"}
B -->|--clean| C["latexmk -c"]
B -->|--open| D["Compilación"]
B -->|sin argumento| D
D --> E{"Sistema operativo"}
E -->|Linux| F["Configurar dependencias Fedora"]
E -->|macOS| G["Usar MacTeX o BasicTeX local"]
F --> H["Verificar latexmk"]
G --> H
H --> I["Regenerar historial y tablas"]
I --> J["latexmk"]
J --> K["build/main.pdf"]
K -->|--open en macOS| L["open build/main.pdf"]
K -->|--open en Linux| M["xdg-open build/main.pdf"]
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
There was a problem hiding this comment.
Hey - I've found 2 issues
Prompt for AI Agents
Please address the comments from this code review:
## Individual Comments
### Comment 1
<location path="tp/informe/pc2/compilar.sh" line_range="17-18" />
<code_context>
+abrir=false
+case "${1:-}" in
+ "") ;;
+ -c|--clean)
+ latexmk -c
+ exit 0
+ ;;
</code_context>
<issue_to_address>
**issue (broader_impact):** La opción `-c/--clean` ejecuta `latexmk -c` antes de configurar `LD_LIBRARY_PATH` y `PERL5LIB` o descargar las dependencias Fedora, por lo que la limpieza falla en Fedora cuando `latexmk` necesita `perl-sigtrap` o `libxcrypt-compat` instalados solamente en `build/deps`.
**Triggers:** Cuando se ejecuta `./compilar.sh -c` en Fedora con las dependencias locales de `build/deps`.
**Suggested fix:** Procesar la configuración de dependencias y verificar `latexmk` antes de ejecutar la rama de limpieza, o aplicar esas variables de entorno también a `latexmk -c`.
</issue_to_address>
### Comment 2
<location path="tp/informe/pc2/compilar.sh" line_range="60-67" />
<code_context>
+}
-./generar-historial.sh
+if ! ./generar-historial.sh; then
+ if [ -s generado/historial.tex ]; then
+ echo "aviso: no se pudo actualizar el historial; se conserva generado/historial.tex" >&2
+ else
+ echo "error: no existe un historial anterior para continuar" >&2
+ exit 1
+ fi
+fi
python3 ../../scripts/tablas_informe.py --salida generado
</code_context>
<issue_to_address>
**issue (bug_risk):** Cualquier fallo de `generar-historial.sh` se convierte en una advertencia y la compilación continúa usando el historial anterior si `generado/historial.tex` no está vacío, de modo que el PDF puede presentar conteos y commits obsoletos sin que el build falle.
**Triggers:** Cuando el historial no puede regenerarse por un tag faltante, una configuración Git inválida, permisos o refs remotas desactualizadas, y ya existe un archivo generado previamente.
**Suggested fix:** Limitar el fallback a errores explícitamente recuperables, como una falla de red con refs válidas, y hacer fallar la compilación ante errores de validación o generación.
```suggestion
./generar-historial.sh
```
</issue_to_address>Sourcery assessment
Approval pending. 2 findings to address first.
Blocking findings: tp/informe/pc2/compilar.sh:18, tp/informe/pc2/compilar.sh:67
Comment on lines
+17
to
+18
| -c|--clean) | ||
| latexmk -c |
There was a problem hiding this comment.
issue (broader_impact): La opción -c/--clean ejecuta latexmk -c antes de configurar LD_LIBRARY_PATH y PERL5LIB o descargar las dependencias Fedora, por lo que la limpieza falla en Fedora cuando latexmk necesita perl-sigtrap o libxcrypt-compat instalados solamente en build/deps.
Triggers: Cuando se ejecuta ./compilar.sh -c en Fedora con las dependencias locales de build/deps.
Suggested fix: Procesar la configuración de dependencias y verificar latexmk antes de ejecutar la rama de limpieza, o aplicar esas variables de entorno también a latexmk -c.
Comment on lines
+60
to
+67
| if ! ./generar-historial.sh; then | ||
| if [ -s generado/historial.tex ]; then | ||
| echo "aviso: no se pudo actualizar el historial; se conserva generado/historial.tex" >&2 | ||
| else | ||
| echo "error: no existe un historial anterior para continuar" >&2 | ||
| exit 1 | ||
| fi | ||
| fi |
There was a problem hiding this comment.
issue (bug_risk): Cualquier fallo de generar-historial.sh se convierte en una advertencia y la compilación continúa usando el historial anterior si generado/historial.tex no está vacío, de modo que el PDF puede presentar conteos y commits obsoletos sin que el build falle.
Triggers: Cuando el historial no puede regenerarse por un tag faltante, una configuración Git inválida, permisos o refs remotas desactualizadas, y ya existe un archivo generado previamente.
Suggested fix: Limitar el fallback a errores explícitamente recuperables, como una falla de red con refs válidas, y hacer fallar la compilación ante errores de validación o generación.
Suggested change
| if ! ./generar-historial.sh; then | |
| if [ -s generado/historial.tex ]; then | |
| echo "aviso: no se pudo actualizar el historial; se conserva generado/historial.tex" >&2 | |
| else | |
| echo "error: no existe un historial anterior para continuar" >&2 | |
| exit 1 | |
| fi | |
| fi | |
| ./generar-historial.sh |
R0SEWT
approved these changes
Oct 2, 2026
R0SEWT
added a commit
that referenced
this pull request
Oct 2, 2026
…nforme del TP Cierra los \pendiente de la tabla de pull requests y de la participación del Entregable 3 con lo que trae cada PR, y regenera el historial de commits.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary by Sourcery
Incorpora las conclusiones de Dayana y facilita la compilación y visualización multiplataforma de los informes.
Enhancements:
Documentation: