Skip to content

Corrección de directorio de construcción en Makefile#751

Merged
humitos merged 1 commit into
python:3.8from
mondeja:fix-make-serve
Sep 21, 2020
Merged

Corrección de directorio de construcción en Makefile#751
humitos merged 1 commit into
python:3.8from
mondeja:fix-make-serve

Conversation

@mondeja

@mondeja mondeja commented Sep 11, 2020 •
edited
Loading

Copy link
Copy Markdown
Contributor

Las variables OUTPUT_DOCTREE y OUTPUT_HTML del Makefile están apuntando al directorio /Doc/_build de cpython como directorio donde se construye la documentación, sin embargo el directorio de construcción de la documentación de cpython fue reemplazado por /Doc/build en un commit hace 3 años, así que el comando make serve ejecutado dentro del repositorio cpython espera que exista el directorio /Doc/build y nosotros estamos construyendo en /Doc/_build, por lo que nuestro comando make serve falla.

Simplemente he actualizado las variables para que se construya en $(CPYTHON_WORKDIR)/Doc/build/, tal y como lo hace actualmente cpython.

@cmaureir cmaureir requested a review from humitos September 11, 2020 13:29
eamanu
eamanu approved these changes Sep 18, 2020
View reviewed changes

@eamanu eamanu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Me parece correcto el cambio. Inclusos se puede observar el CI o el .gitignore, para ver que cpython espera la generación de la documentación en Doc/build.

Aún así no puedo reproducir algún error, sin el cambio.

@mondeja

mondeja commented Sep 18, 2020 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Aún así no puedo reproducir algún error, sin el cambio

Sin el cambio, después de ejecutar rm -rf cpython/Doc/build cpython/Doc/_build && make build && make serve, ve a localhost:8000, ¿eres capaz de ver la documentación? Yo sólo veo una pantalla en blanco donde aparece not found. Luego ejecuto mv cpython/Doc/_build cpython/Doc/build && make serve, voy a localhost:8000 y soy capaz de verla. A eso me refiero con que make serve falla, a que no sirve la documentación.

@eamanu

eamanu commented Sep 19, 2020

Copy link
Copy Markdown
Contributor

Aún así no puedo reproducir algún error, sin el cambio

Sin el cambio, después de ejecutar rm -rf cpython/Doc/build cpython/Doc/_build && make build && make serve, ve a localhost:8000, ¿eres capaz de ver la documentación? Yo sólo veo una pantalla en blanco donde aparece not found. Luego ejecuto mv cpython/Doc/_build cpython/Doc/build && make serve, voy a localhost:8000 y soy capaz de verla. A eso me refiero con que make serve falla, a que no sirve la documentación.

Ahí lo vi. Genial. +1 por el PR

@humitos humitos merged commit 557b236 into python:3.8 Sep 21, 2020
@humitos

humitos commented Sep 21, 2020

Copy link
Copy Markdown
Collaborator

Gracias!

@mondeja mondeja deleted the fix-make-serve branch September 26, 2020 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@humitos humitos humitos approved these changes

+1 more reviewer

@eamanu eamanu eamanu approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mondeja @eamanu @humitos