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

warning: unused manifest key: package.categories #3607

Closed
sunjay opened this issue Jan 28, 2017 · 5 comments
Closed

warning: unused manifest key: package.categories #3607

sunjay opened this issue Jan 28, 2017 · 5 comments

Comments

@sunjay
Copy link
Member

sunjay commented Jan 28, 2017

This is my Cargo.toml file:

[package]
name = "brain"
version = "0.0.1"
authors = ["Sunjay Varma <varma.sunjay@gmail.com>"]
description = "Compiler for the brain programming language. Compiles brain into optimized brainfuck code. Also includes a brainfuck interpreter."
repository = "https://github.com/sunjay/brain"
readme = "README.md"
keywords = ["brain", "brainfuck", "compiler", "interpreter", "optimizer"]
# crates.io/category_slugs
categories = ["command-line-interface"]
license = "MIT"

[dependencies]
clap = "^2.0"

[dependencies.nom]
version = "^2.0"
features = ["verbose-errors"]

[[bin]]
name = "brain"
doc = false

[[bin]]
name = "brainfuck"

Every time I run cargo run, I get the following output:

$ cargo run --bin brain
warning: unused manifest key: package.categories
   Compiling brain v0.0.1 (file:///~/Documents/projects/brain)
    Finished debug [unoptimized + debuginfo] target(s) in 3.78 secs
     Running `target/debug/brain`
...blah...

The warning probably shouldn't be there because package.categories is a valid key as documented in The Manifest Format.

@sfackler
Copy link
Member

Your version of cargo is presumably too old to have category support.

@sunjay
Copy link
Member Author

sunjay commented Jan 29, 2017

@sfackler Have categories landed in stable? I notice that if I use rustup override set nightly, the warning disappears but then if I unset and use the default (stable), I get the warning again.

$ cargo --version
cargo 0.15.0-nightly (298a012 2016-12-20)
$ cargo build
warning: unused manifest key: package.categories
    Finished debug [unoptimized + debuginfo] target(s) in 0.0 secs

@sfackler
Copy link
Member

The PR adding them merged 11 days ago, so it is not in the stable release: #3301

@sunjay
Copy link
Member Author

sunjay commented Jan 29, 2017

@sfackler Okay. This doesn't seem to be much of an issue then since the warning will go away when #3301 lands in stable. Looks like @briansmith commented in that pull request about this.

@alexcrichton
Copy link
Member

Yes the fields here are supported, just in nightly Rust right now and not yet on stable. I believe there's other bugs about the docs mismatch, but otherwise this is working as intended so I'm going to close. Thanks though for the report!

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

No branches or pull requests

3 participants