Split requirements/pip.txt

[dojutsu-user]

Fixes #3757

[humitos]

Good approach! I think we are close to have it ready for merging.

I'd like to be more aggressive on the split and have it properly separated.

Example: If I downloaded the git repo and want to build the Read the Docs documentation, I should only do pip install requirements/docs.txt and then make html should work and produce the desired docs _without installing anything no-needed` (django or whatever).

This will probably ends up on having repeated packages on the requirements, but that's fine to me because we want to be able to fine tune it: "use Sphinx 2.0 for our docs, but not for building user's docs"

[humitos]

I suppose that these two packages are for building our own docs and not for building user's documentation. So, these should probably go into a different reqs file maybe.

[dojutsu-user]

Wasn't aware of that these are only for RTD docs.

these should probably go into a different reqs file maybe.

Since these are RTD-only requirements, we can put it back in the pip.txt

[humitos]

Well, they are not required to "run Read the Read software" but to build the docs of it.

So, basically, we should use this requirements/docs.txt file in our own projects under readthedocs.io (https://readthedocs.org/projects/docs/)

[dojutsu-user]

Okay.
I have separated them in docs.txt and also add -r local-docs-build.txt to install the other dependencies.

[dojutsu-user]

I put this here because it was required to make the lint tests.
This gets repeated here as it was already moved to the local_docs_build.txt

[dojutsu-user]

@humitos

Example: If I downloaded the git repo and want to build the Read the Docs documentation, I should only do pip install requirements/docs.txt and then make html should work and produce the desired docs _without installing anything no-needed` (django or whatever).

I tried to do exactly the same thing. But in the conf.py, but we are importing django and other stuff, so make html doesn't work with the virtual environment created from the local_docs_build.txt

[humitos]

But in the conf.py, but we are importing django

Do you know why we are doing that? Is it to document our django settings?

On the other hand, in case that it's strictly needed, does the docs build just installing django without installing the whole RTD dependencies?

[dojutsu-user]

@humitos

Do you know why we are doing that? Is it to document our django settings?

These lines.

https://github.com/rtfd/readthedocs.org/blob/b07f4550345b90e07fd2504b984c3361a05a97c6/docs/conf.py#L15-L19

• I didn't find any use for settings.
• But timezone is being used down in the file.
• And django.setup()

On the other hand, in case that it's strictly needed, does the docs build just installing django without installing the whole RTD dependencies?

I tried this.

• Installed django (raised error for future)
• Install future (raised error for celery)

I think that RTD docs needs the RTD-specific requirements (pip.txt)

[humitos]

We are using the extension djangodocs which allow us to use :djangosetting: in our .rst files.

I think that RTD docs needs the RTD-specific requirements (pip.txt)

OK, we can open another issue to research about this later and try to have the minimal dependencies for our docs.

[humitos]

Please, revert the creation of docs.txt, open a new issue to track that path and we will be ready to merge this PR from my point of view.

[humitos]

Install future (raised error for celery)

This is because we are importing from celery.schedules import crontab in our settings. I'm almost sure that we can split this to a docs.txt and install only the needed dependencies.

[dojutsu-user]

@humitos
I have moved Docs requirements back to local_docs_build.txt

This is because we are importing from celery.schedules import crontab in our settings. I'm almost sure that we can split this to a docs.txt and install only the needed dependencies.

It would be great. Will open a new issue to track this.

[ericholscher]

Won't this cause our docs build to fail, unless we update our .readthedocs.yml to point at a new config file: https://github.com/rtfd/readthedocs.org/blob/master/.readthedocs.yml#L6

[dojutsu-user]

@ericholscher
I forget to do this.
I have updated tha yaml file.

[ericholscher]

This will go out with next deploy, so we have time to debug 👍