Skip to content

Document all download_cache's properly #3176

Description

@maksim-petukhou

I would like to cache downloads of sources.
I look into documentation documentation and see that it mentions core.download:download_cache configuration and describes it as

Define path to a file download cache

Are sources files? Yep. This must be a right fit. And it's not because this conf controls path to a cache of internal files. It would be great to mention that in docs.

Then we have tools.files.download:download_cache configuration and the documentation says that it's a conf for

caching the external/third_party files downloaded by recipes in download() or get() calls.

So I set it and get an error from conan

ERROR: Unknown conf 'tools.files.download:download_cache'.

And... This conf doesn't exist. It turns out that this conf was deleted. Changelogs doesn't mention that fact at all. This conf was replaced (as far as I understand) by core.sources:download_cache which is not documented at all.

Summing up

  • core.download:download_cache conf documentation should say that it's for internal files
  • tools.files.download:download_cache conf documentation should be removed
  • core.sources:download_cache conf should be documented

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions