From b2e874e151548930d674a64940506f29a58541b8 Mon Sep 17 00:00:00 2001 From: Matt Brubeck Date: Tue, 10 May 2016 16:03:42 -0700 Subject: [PATCH] Remove unnecessary dependencies and features from top-level Cargo.tomls The features for each crate will be computed based on the union of features specified in the dependency graph. Specifying the same ones again just adds more ways for them to get out of sync. --- components/script/Cargo.toml | 2 +- components/servo/Cargo.lock | 1 - components/servo/Cargo.toml | 7 +++---- components/servo/lib.rs | 1 - ports/cef/Cargo.lock | 7 +------ ports/cef/Cargo.toml | 11 +++-------- ports/cef/lib.rs | 5 ----- ports/geckolib/Cargo.lock | 1 - ports/geckolib/Cargo.toml | 12 ++++++------ ports/gonk/Cargo.lock | 2 +- python/servo/build_commands.py | 3 --- 11 files changed, 15 insertions(+), 37 deletions(-) diff --git a/components/script/Cargo.toml b/components/script/Cargo.toml index 6f9fc934c3c..d5ca90d0f8e 100644 --- a/components/script/Cargo.toml +++ b/components/script/Cargo.toml @@ -63,6 +63,6 @@ smallvec = "0.1" string_cache = {version = "0.2.12", features = ["heap_size", "unstable"]} time = "0.1.12" unicase = "1.0" -url = {version = "1.0.0", features = ["heap_size"]} +url = {version = "1.0.0", features = ["heap_size", "query_encoding"]} uuid = { version = "0.2", features = ["v4"] } websocket = "0.17" diff --git a/components/servo/Cargo.lock b/components/servo/Cargo.lock index 185b7428af1..e285c0f8a44 100644 --- a/components/servo/Cargo.lock +++ b/components/servo/Cargo.lock @@ -19,7 +19,6 @@ dependencies = [ "glutin_app 0.0.1", "image 0.9.1 (registry+https://github.com/rust-lang/crates.io-index)", "ipc-channel 0.2.2 (git+https://github.com/servo/ipc-channel)", - "layers 0.2.5 (git+https://github.com/servo/rust-layers)", "layout 0.0.1", "layout_tests 0.0.1", "libc 0.2.11 (registry+https://github.com/rust-lang/crates.io-index)", diff --git a/components/servo/Cargo.toml b/components/servo/Cargo.toml index 81a46875200..6b9d6b0212f 100644 --- a/components/servo/Cargo.toml +++ b/components/servo/Cargo.toml @@ -65,18 +65,17 @@ devtools = {path = "../devtools"} webdriver_server = {path = "../webdriver_server", optional = true} devtools_traits = {path = "../devtools_traits"} glutin_app = {path = "../../ports/glutin", optional = true} -android_glue = {version = "0.1.3", optional = true} ipc-channel = {git = "https://github.com/servo/ipc-channel"} -layers = {git = "https://github.com/servo/rust-layers", features = ["plugins"]} gleam = "0.2" browserhtml = {git = "https://github.com/browserhtml/browserhtml", branch = "gh-pages"} env_logger = "0.3" -euclid = {version = "0.6.4", features = ["plugins"]} +euclid = "0.6.4" libc = "0.2" -url = {version = "1.0.0", features = ["heap_size", "serde", "query_encoding"]} +url = "1.0.0" [target.'cfg(target_os = "android")'.dependencies] log = "0.3" +android_glue = "0.1.3" [target.'cfg(not(target_os = "windows"))'.dependencies] gaol = {git = "https://github.com/servo/gaol"} diff --git a/components/servo/lib.rs b/components/servo/lib.rs index c8355affa58..648e0ffb85b 100644 --- a/components/servo/lib.rs +++ b/components/servo/lib.rs @@ -30,7 +30,6 @@ pub extern crate devtools_traits; pub extern crate euclid; pub extern crate gfx; pub extern crate ipc_channel; -pub extern crate layers; pub extern crate layout; pub extern crate msg; pub extern crate net; diff --git a/ports/cef/Cargo.lock b/ports/cef/Cargo.lock index 0fee2bea10f..12f172377fc 100644 --- a/ports/cef/Cargo.lock +++ b/ports/cef/Cargo.lock @@ -2,7 +2,6 @@ name = "embedding" version = "0.0.1" dependencies = [ - "azure 0.4.5 (git+https://github.com/servo/rust-azure)", "cgl 0.1.4 (registry+https://github.com/rust-lang/crates.io-index)", "cocoa 0.3.2 (registry+https://github.com/rust-lang/crates.io-index)", "compositing 0.0.1", @@ -11,10 +10,8 @@ dependencies = [ "core-text 1.1.0 (registry+https://github.com/rust-lang/crates.io-index)", "devtools 0.0.1", "euclid 0.6.6 (registry+https://github.com/rust-lang/crates.io-index)", - "gfx 0.0.1", "gleam 0.2.17 (registry+https://github.com/rust-lang/crates.io-index)", "glutin_app 0.0.1", - "js 0.1.2 (git+https://github.com/servo/rust-mozjs)", "layers 0.2.5 (git+https://github.com/servo/rust-layers)", "libc 0.2.11 (registry+https://github.com/rust-lang/crates.io-index)", "log 0.3.6 (registry+https://github.com/rust-lang/crates.io-index)", @@ -22,10 +19,8 @@ dependencies = [ "net_traits 0.0.1", "objc 0.2.1 (registry+https://github.com/rust-lang/crates.io-index)", "plugins 0.0.1", - "script 0.0.1", "script_traits 0.0.1", "servo 0.0.1", - "style 0.0.1", "style_traits 0.0.1", "url 1.1.0 (registry+https://github.com/rust-lang/crates.io-index)", "util 0.0.1", @@ -1836,6 +1831,7 @@ dependencies = [ name = "servo" version = "0.0.1" dependencies = [ + "android_glue 0.1.3 (registry+https://github.com/rust-lang/crates.io-index)", "browserhtml 0.1.5 (git+https://github.com/browserhtml/browserhtml?branch=gh-pages)", "canvas 0.0.1", "canvas_traits 0.0.1", @@ -1849,7 +1845,6 @@ dependencies = [ "gleam 0.2.17 (registry+https://github.com/rust-lang/crates.io-index)", "glutin_app 0.0.1", "ipc-channel 0.2.2 (git+https://github.com/servo/ipc-channel)", - "layers 0.2.5 (git+https://github.com/servo/rust-layers)", "layout 0.0.1", "libc 0.2.11 (registry+https://github.com/rust-lang/crates.io-index)", "log 0.3.6 (registry+https://github.com/rust-lang/crates.io-index)", diff --git a/ports/cef/Cargo.toml b/ports/cef/Cargo.toml index b42ef0b1dc4..17f495dce78 100644 --- a/ports/cef/Cargo.toml +++ b/ports/cef/Cargo.toml @@ -16,27 +16,22 @@ codegen-units = 4 # lto = false [dependencies] -euclid = {version = "0.6.4", features = ["plugins"]} +euclid = "0.6.4" gleam = "0.2.8" libc = "0.2" -url = {version = "1.0.0", features = ["heap_size"]} +url = "1.0.0" servo = {path = "../../components/servo"} glutin_app = {path = "../glutin"} plugins = {path = "../../components/plugins"} compositing = {path = "../../components/compositing"} -gfx = {path = "../../components/gfx"} log = {version = "0.3.5", features = ["release_max_level_info"]} -script = {path = "../../components/script"} script_traits = {path = "../../components/script_traits"} net_traits = {path = "../../components/net_traits"} msg = {path = "../../components/msg"} util = {path = "../../components/util", features = ["non-geckolib"]} -style = {path = "../../components/style"} style_traits = {path = "../../components/style_traits"} devtools = {path = "../../components/devtools"} -azure = {git = "https://github.com/servo/rust-azure", features = ["plugins"]} -js = {git = "https://github.com/servo/rust-mozjs"} -layers = {git = "https://github.com/servo/rust-layers", features = ["plugins"]} +layers = {git = "https://github.com/servo/rust-layers"} [target.'cfg(target_os="macos")'.dependencies] objc = "0.2" diff --git a/ports/cef/lib.rs b/ports/cef/lib.rs index 8f1f67dadda..72f0771bcf8 100644 --- a/ports/cef/lib.rs +++ b/ports/cef/lib.rs @@ -22,21 +22,16 @@ extern crate log; extern crate servo; extern crate compositing; -extern crate azure; extern crate euclid; -extern crate gfx; extern crate gleam; extern crate glutin_app; -extern crate js; extern crate layers; extern crate rustc_unicode; -extern crate script; extern crate script_traits; extern crate net_traits; extern crate msg; extern crate util; -extern crate style; extern crate style_traits; extern crate libc; diff --git a/ports/geckolib/Cargo.lock b/ports/geckolib/Cargo.lock index 617a8a508cd..51d8938dc20 100644 --- a/ports/geckolib/Cargo.lock +++ b/ports/geckolib/Cargo.lock @@ -618,7 +618,6 @@ name = "url" version = "1.1.0" source = "registry+https://github.com/rust-lang/crates.io-index" dependencies = [ - "encoding 0.2.32 (registry+https://github.com/rust-lang/crates.io-index)", "heapsize 0.3.5 (registry+https://github.com/rust-lang/crates.io-index)", "heapsize_plugin 0.1.4 (registry+https://github.com/rust-lang/crates.io-index)", "idna 0.1.0 (registry+https://github.com/rust-lang/crates.io-index)", diff --git a/ports/geckolib/Cargo.toml b/ports/geckolib/Cargo.toml index 2778792afcd..7ef5872c248 100644 --- a/ports/geckolib/Cargo.toml +++ b/ports/geckolib/Cargo.toml @@ -11,18 +11,18 @@ path = "lib.rs" crate-type = ["staticlib"] [dependencies] -app_units = {version = "0.2.3", features = ["plugins"]} -cssparser = {version = "0.5.4", features = ["heap_size", "serde-serialization"]} -euclid = {version = "0.6.4", features = ["plugins"]} +app_units = "0.2.3" +cssparser = "0.5.4" +euclid = "0.6.4" heapsize = "0.3.0" heapsize_plugin = "0.1.2" lazy_static = "0.2" libc = "0.2" num_cpus = "0.2.2" -selectors = {version = "0.5", features = ["heap_size", "unstable"]} +selectors = "0.5" smallvec = "0.1" -string_cache = {version = "0.2.12", features = ["heap_size", "unstable"]} -url = {version = "1.0.0", features = ["heap_size", "query_encoding", "serde"]} +string_cache = "0.2.12" +url = "1.0.0" log = {version = "0.3.5", features = ["release_max_level_info"]} plugins = {path = "../../components/plugins"} util = {path = "../../components/util"} diff --git a/ports/gonk/Cargo.lock b/ports/gonk/Cargo.lock index d821a84bbb7..62006b74837 100644 --- a/ports/gonk/Cargo.lock +++ b/ports/gonk/Cargo.lock @@ -1819,6 +1819,7 @@ dependencies = [ name = "servo" version = "0.0.1" dependencies = [ + "android_glue 0.1.3 (registry+https://github.com/rust-lang/crates.io-index)", "browserhtml 0.1.5 (git+https://github.com/browserhtml/browserhtml?branch=gh-pages)", "canvas 0.0.1", "canvas_traits 0.0.1", @@ -1831,7 +1832,6 @@ dependencies = [ "gfx 0.0.1", "gleam 0.2.17 (registry+https://github.com/rust-lang/crates.io-index)", "ipc-channel 0.2.2 (git+https://github.com/servo/ipc-channel)", - "layers 0.2.5 (git+https://github.com/servo/rust-layers)", "layout 0.0.1", "libc 0.2.11 (registry+https://github.com/rust-lang/crates.io-index)", "log 0.3.6 (registry+https://github.com/rust-lang/crates.io-index)", diff --git a/python/servo/build_commands.py b/python/servo/build_commands.py index feca17976d0..3dca0f3fae6 100644 --- a/python/servo/build_commands.py +++ b/python/servo/build_commands.py @@ -197,9 +197,6 @@ class MachCommands(CommandBase): if debug_mozjs or self.config["build"]["debug-mozjs"]: features += ["script/debugmozjs"] - if android: - features += ["android_glue"] - if features: opts += ["--features", "%s" % ' '.join(features)]