Skip to content
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.

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

Always show the raw compilation for debugging purpose #7847

Closed
wants to merge 1 commit into from

Conversation

alkino
Copy link

@alkino alkino commented Jan 20, 2021

Let show the compilation log, it's easier to find information inside.

@stsewd stsewd added the Needed: design decision A core team decision is required label Jan 20, 2021
Copy link
Member

@stsewd stsewd left a comment

Choose a reason for hiding this comment

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

Not sure about this, the full log being shown at the end makes sense to me. Having it show early could be misleading that the build finished.

@alkino
Copy link
Author

alkino commented Jan 20, 2021

My problem today is my builds looking stuck on cat conf.py for 10 minutes!
https://readthedocs.org/projects/coreneuron/builds/12808875/

In fact it's running something behind the scene.
And showing the raw log doesn't help because it shows only finished commands.

I think the log should be better to help user making things.

@alkino
Copy link
Author

alkino commented Jan 20, 2021

Wait for #7005

@alkino alkino closed this Jan 20, 2021
@alkino alkino deleted the always-show-raw branch January 20, 2021 15:24
@stsewd
Copy link
Member

stsewd commented Jan 20, 2021

My problem today is my builds looking stuck on cat conf.py for 10 minutes!

That's because we show the log only when the current command is finished, in this case, the command being executed that lasted 10 min was the sphinx command, the raw log would have shown the same step as you see in the normal log. See #4288

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needed: design decision A core team decision is required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants