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

manual: Add missing section on tarball-ttl #2978

Closed
wants to merge 1 commit into from

Conversation

nh2
Copy link
Contributor

@nh2 nh2 commented Jul 1, 2019

No description provided.

@edolstra
Copy link
Member

edolstra commented Jul 1, 2019

See also #2882.

<para>This is most relevant for fetching tarballs that change remotely
via <envar>NIX_PATH</envar> or <function>fetchGit</function>.
Using <option>--option tarball-ttl 0</option> forces nix to re-fetch
these tarballs.</para>
Copy link
Member

Choose a reason for hiding this comment

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

This is not enitirely correct. TTL=0 doesn't force a re-fetch, it forces Nix to check whether the tarball is still up-to-date using the ETag.

Copy link
Member

Choose a reason for hiding this comment

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

I incorporated much of this text, and this feedback in https://github.com/NixOS/nix/pull/2882/files

grahamc added a commit to grahamc/nix that referenced this pull request Jul 3, 2019
Incorporates text from Niklas Hambüchen in NixOS#2978

Closes NixOS#1115
grahamc added a commit to grahamc/nix that referenced this pull request Jul 5, 2019
Incorporates text from Niklas Hambüchen in NixOS#2978

Closes NixOS#1115
grahamc added a commit to grahamc/nix that referenced this pull request Jul 5, 2019
Incorporates text from Niklas Hambüchen in NixOS#2978

Closes NixOS#1115
grahamc added a commit to grahamc/nix that referenced this pull request Jul 5, 2019
Incorporates text from Niklas Hambüchen in NixOS#2978

Closes NixOS#1115
grahamc added a commit to grahamc/nix that referenced this pull request Jul 5, 2019
Incorporates text from Niklas Hambüchen in NixOS#2978

Closes NixOS#1115
@teto
Copy link
Member

teto commented Jul 6, 2019

can be closed I guess since other PR got merged

@edolstra edolstra closed this Jul 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants