Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Sep 21, 2020
Merged

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

merged 1 commit into from
Sep 21, 2020

Conversation

mondeja
Copy link
Contributor

@mondeja mondeja commented Sep 11, 2020 •
edited
Loading

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
Copy link
Contributor

@eamanu eamanu left a comment

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
Copy link
Contributor Author

mondeja commented Sep 18, 2020 •
edited
Loading

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
Copy link
Contributor

eamanu commented Sep 19, 2020

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
Copy link
Collaborator

humitos commented Sep 21, 2020

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

@eamanu eamanu eamanu approved these changes

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