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

buildRustPackage: fix deprecated use of registry.index config key #22987

Merged
merged 1 commit into from
Feb 19, 2017

Conversation

anderspapitto
Copy link
Contributor

See
rust-lang/cargo@8214bb9
for the cargo commit which deprecated the registry.index key, and
implements this as a replacement. This gets rid of the error message

warning: custom registry support via the registry.index configuration is being removed, this functionality will not work in the future

I tested by faking a depsSha256 for a package using buildRustPackage, rebuilding, and verifying that

  1. the warning goes away
  2. it still doesn't talk to crates.io (i.e. I didn't break it)

See
rust-lang/cargo@8214bb9
for the cargo commit which deprecated the registry.index key, and
implements this as a replacement. This gets rid of the error message

warning: custom registry support via the `registry.index` configuration is being removed, this functionality will not work in the future
@mention-bot
Copy link

@anderspapitto, thanks for your PR! By analyzing the history of the files in this pull request, we identified @wizeman, @madjar and @abbradar to be potential reviewers.

@zimbatm zimbatm merged commit 6dbb3ab into NixOS:master Feb 19, 2017
@grahamc
Copy link
Member

grahamc commented Feb 27, 2017

Awesome, thank you!

@grahamc
Copy link
Member

grahamc commented Mar 1, 2017

Hi @anderspapitto, it looks like this change has broken building some packages. Can you help? #23282

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