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

Build failed at git clean stage #4728

Closed
spinicist opened this issue Oct 4, 2018 · 13 comments
Closed

Build failed at git clean stage #4728

spinicist opened this issue Oct 4, 2018 · 13 comments
Labels
Support Support question

Comments

@spinicist
Copy link

Details

Hello,
I am trying to swap from Github Pages (compiled with mkdoc) to ReadTheDocs (and Sphinx) for my C++ project. I wanted to do this in a separate branch before merging to master, but that appears to be failing because the git clean step fails (there is a directory in the master branch that no longer exists in the Sphinx branch).

Expected Result

ReadTheDocs completes the build

Actual Result

The build fails at the git clean stage.

@spinicist
Copy link
Author

I ended up merging my Sphinx branch to master, but it was still failing. I then tried deleting the project on readthedocs and re-creating it, but it is still failing, now at the git branch step and I cannot see an error message https://readthedocs.org/projects/quit/builds/7889821/

(Sorry for deleting the project before anyone had a chance to look at the previous failure).

@stsewd stsewd added the Support Support question label Oct 4, 2018
@stsewd
Copy link
Member

stsewd commented Oct 4, 2018

The issue is because you have an invalid submodule https://github.com/spinicist/QUIT/blob/0d81abb965d9c1be0ccbd071d9ee453daad6e4db/.gitmodules#L10 and rtd fails when checking for submodules

@stsewd
Copy link
Member

stsewd commented Oct 4, 2018

And thanks for using rtd!

@humitos
Copy link
Member

humitos commented Oct 4, 2018

@spinicist please, remove the submodule that @stsewd mentioned and try building again. It should work after that. If that's the case, please close this issue.

In case you find any different problem on your building after removing that submodule, please open a new issue.

Thanks and welcome home ;)

@spinicist
Copy link
Author

Thanks at @stsewd and @humitos! That invalid submodule must have been hanging around for months 😳 I've excised it and it does now work.

Any idea why rtd didn't report the failing submodule step? When I looked at the raw logs, the last step kept saying it exited with status 0 which was confusing.

@stsewd
Copy link
Member

stsewd commented Oct 4, 2018

That's another issue #4288

@spinicist
Copy link
Author

Great - hopefully that will get resolved in the future.

Final question - is there a way to get rtd to skip the submodule checkout stage? For my project, I don't need the submodules to build the documentation.

Thanks for your fast responses, very much appreciated.

@stsewd
Copy link
Member

stsewd commented Oct 4, 2018

We have that feature in our next version of the configuration file #4493 (hope we have an opt-in or open beta soon), stay tuned!

@spinicist
Copy link
Author

Good news, thanks again.

@humitos
Copy link
Member

humitos commented Oct 5, 2018

Happy that you fixed your issue!

@spinicist spinicist reopened this Oct 5, 2018
@spinicist
Copy link
Author

Um, it fixed one issue but now I have another related one (sorry).

I have two branches - master and sphinx - in my github repo. When I pushed my updates to the sphinx branch and triggered a build, it all worked fine (https://quit.readthedocs.io/en/sphinx/). However, when I merged those changes to master and pushed, which is picked up by rtd as latest, I don't get my documentation, but the documentation for one of the submodules (ceres, https://quit.readthedocs.io/en/latest/). For some reason on latest rtd decides to pick up the conf.py for Ceres (https://readthedocs.org/projects/quit/builds/7896020/)

So I could use that beta version ASAP 😉

@stsewd
Copy link
Member

stsewd commented Oct 5, 2018

You can select a conf.py file from the web admin of your project Admin -> Advanced Settings

@spinicist
Copy link
Author

I missed that setting. Hooray! It all works now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Support Support question
Projects
None yet
Development

No branches or pull requests

3 participants