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

[20.09] ipfs: Add 0.7.0 in addition #100676

Merged

Conversation

Ericson2314
Copy link
Member

Motivation for this change

IPFS has been evolving quite fast lately, so I think tracking new release in stable NixOS is prudent.

CC @Luflosi

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.

Luflosi and others added 4 commits October 15, 2020 22:58
IPFS would complain with warnings like
Configuration file /nix/store/...-ipfs-0.6.0/etc/systemd/system/ipfs.service is marked executable. Please remove executable permission bits. Proceeding anyway.

(cherry picked from commit 1a382e9)
Trying to move a directory into itself will result in a warning:
mv: cannot move 'ipfs-src' to a subdirectory of itself, 'ipfs-src/ipfs-src'

This can be prevented by excluding that directory.

(cherry picked from commit d385065)
IPFS has been evolving quite fast lately, so I think tracking new
release in stable NixOS is prudent.

(cherry picked from commit b75e431)
@Luflosi
Copy link
Contributor

Luflosi commented Oct 16, 2020

Given that the two files for the two versions are identical except for the version and the hash, would it make sense to somehow combine the two?
Otherwise, looks good to me.

@fabianhjr
Copy link
Member

Is there a particular reason to split the IPFS package into two version releases?

@Ericson2314
Copy link
Member Author

@Luflosi I thought about that, but I feel seeing that this is just on a release branch with bounded life, it would be a premature abstraction not with the effort. Also, suppose 0.8 comes out before 21.03 requiring more invasive changes?

[Conversely, I think the more permanent duplication of LLVMs, GCCs, etc. is a problem, and deduplicating those would be a great use of time.]

Otherwise, looks good to me.

OK, will merge.

@fabianhjr I don't want to bump the default major version on a stable branch, that's all.

@Ericson2314 Ericson2314 merged commit 8b56565 into NixOS:release-20.09 Oct 16, 2020
@Ericson2314 Ericson2314 deleted the ipfs_latest-for-stable branch October 16, 2020 22:38
Ericson2314 added a commit to obsidiansystems/nixpkgs that referenced this pull request Jul 2, 2021
IPFS evolves quite fast. The network protocol is compatible, but we
don't want to force migrations on stable, so we add a new version
instead.

See NixOS#100676 for last time we did this.

(Adapted from from commit d96ccfa)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants