Skip to content
This repository was archived by the owner on Nov 6, 2020. It is now read-only.

dev feature enabled when compiling without --release #627

Merged
merged 9 commits into from
Mar 10, 2016
Merged

Conversation

tomusdrw
Copy link
Collaborator

@tomusdrw tomusdrw commented Mar 7, 2016

No description provided.

@tomusdrw tomusdrw added the A0-pleasereview 🤓 Pull request needs code review. label Mar 7, 2016
ethsync = { path = "sync" }
ethcore-rpc = { path = "rpc", optional = true }
fdlimit = { path = "util/fdlimit" }
daemonize = "0.2"
ethcore-devtools = { path = "devtools" }
Copy link
Collaborator

Choose a reason for hiding this comment

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

@NikVolf Is this a dev dependency as well?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I think that it isn't. It should still be used when you run cargo test --release

@tomusdrw
Copy link
Collaborator Author

tomusdrw commented Mar 7, 2016

This is the best I came up with.

  1. Clippy dependency is optional (and has to be because it's not possible to compile clippy on stable/beta).
  2. When compiling on nightly for development (without --release flag) the build will fail because of missing clippy create.
  3. You have to explicitly invoke cargo test --features dev-clippy when building without --release on nightly.
  4. It's not possible to add dependency or to enable feature from build.rs

@gavofyork gavofyork removed the A0-pleasereview 🤓 Pull request needs code review. label Mar 9, 2016
travis-beta = ["ethcore/json-tests"]
travis-nightly = ["ethcore/json-tests", "dev"]
travis-nightly = ["ethcore/json-tests", "clippy", "dev"]
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think, it should be dev-clippy instead of clippy.

@gavofyork gavofyork added the A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. label Mar 9, 2016
@gavofyork
Copy link
Contributor

does this address the problem properly?

@NikVolf NikVolf added A0-pleasereview 🤓 Pull request needs code review. and removed A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. labels Mar 9, 2016
@tomusdrw
Copy link
Collaborator Author

tomusdrw commented Mar 9, 2016

It just forces you to use --feature dev-clippy when compiling on nightly without --release. I'm not sure if that's the correct way to address the problem.

There is also a convenience script included to run something like ./cargo.sh test -p ethcore (it adds the feature for you).

@NikVolf NikVolf added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Mar 9, 2016
gavofyork added a commit that referenced this pull request Mar 10, 2016
`dev` feature enabled when compiling without `--release`
@gavofyork gavofyork merged commit 249a89f into master Mar 10, 2016
@gavofyork gavofyork deleted the clippy-dev branch March 10, 2016 10:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants