toolchain: refactor enum into an opaque struct

Author: pietroalbini

CHANGELOG.md

@@ -7,12 +7,19 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Added
 
+- New struct `toolchain::CiToolchain` containing a CI toolchain's metadata.
+- New struct `toolchain::DistToolchain` containing a dist toolchain's metadata.
 - New method `WorkspaceBuilder::rustup_profile` to configure the rustup profile
   used during builds.
+- New method `Toolchain::as_ci` to get a CI toolchain's metadata.
+- New method `Toolchain::as_dist` to get a dist toolchain's metadata.
+- New method `Toolchain::ci` to create CI toolchains.
+- New method `Toolchain::dist` to create dist toolchains.
 
 ### Changed
 
 - **BREAKING**: The default rustup profile is now `minimal`.
+- **BREAKING**: The `Toolchain` enum is now an opaque struct.
 - The directory `target/` inside local crates won't be copied into the build
   anymore.
 - Symbolic links will be followed instead of copied as links.

examples/docs-builder.rs

@@ -11,9 +11,7 @@ fn main() -> Result<(), Box<dyn Error>> {
         WorkspaceBuilder::new(Path::new(".workspaces/docs-builder"), "rustwide-examples").init()?;
 
     // Run the builds on stable
-    let toolchain = Toolchain::Dist {
-        name: "stable".into(),
-    };
+    let toolchain = Toolchain::dist("stable");
     toolchain.install(&workspace)?;
 
     // Fetch lazy_static from crates.io

src/build.rs

@@ -42,7 +42,7 @@ impl<'a> BuildBuilder<'a> {
     /// # use std::error::Error;
     /// # fn main() -> Result<(), Box<dyn Error>> {
     /// # let workspace = WorkspaceBuilder::new("".as_ref(), "").init()?;
-    /// # let toolchain = Toolchain::Dist { name: "".into() };
+    /// # let toolchain = Toolchain::dist("");
     /// # let krate = Crate::local("".as_ref());
     /// # let sandbox = SandboxBuilder::new();
     /// let mut build_dir = workspace.build_dir("foo");
@@ -76,7 +76,7 @@ impl<'a> BuildBuilder<'a> {
     /// # use std::error::Error;
     /// # fn main() -> Result<(), Box<dyn Error>> {
     /// # let workspace = WorkspaceBuilder::new("".as_ref(), "").init()?;
-    /// # let toolchain = Toolchain::Dist { name: "".into() };
+    /// # let toolchain = Toolchain::dist("");
     /// # let krate = Crate::local("".as_ref());
     /// # let sandbox = SandboxBuilder::new();
     /// let mut build_dir = workspace.build_dir("foo");
@@ -110,7 +110,7 @@ impl BuildDirectory {
     /// # use std::error::Error;
     /// # fn main() -> Result<(), Box<dyn Error>> {
     /// # let workspace = WorkspaceBuilder::new("".as_ref(), "").init()?;
-    /// # let toolchain = Toolchain::Dist { name: "".into() };
+    /// # let toolchain = Toolchain::dist("");
     /// # let krate = Crate::local("".as_ref());
     /// # let sandbox = SandboxBuilder::new();
     /// let mut build_dir = workspace.build_dir("foo");
@@ -208,7 +208,7 @@ impl Build<'_> {
     /// # use std::error::Error;
     /// # fn main() -> Result<(), Box<dyn Error>> {
     /// # let workspace = WorkspaceBuilder::new("".as_ref(), "").init()?;
-    /// # let toolchain = Toolchain::Dist { name: "".into() };
+    /// # let toolchain = Toolchain::dist("");
     /// # let krate = Crate::local("".as_ref());
     /// # let sandbox = SandboxBuilder::new();
     /// let mut build_dir = workspace.build_dir("foo");
@@ -245,7 +245,7 @@ impl Build<'_> {
     /// # use std::error::Error;
     /// # fn main() -> Result<(), Box<dyn Error>> {
     /// # let workspace = WorkspaceBuilder::new("".as_ref(), "").init()?;
-    /// # let toolchain = Toolchain::Dist { name: "".into() };
+    /// # let toolchain = Toolchain::dist("");
     /// # let krate = Crate::local("".as_ref());
     /// # let sandbox = SandboxBuilder::new();
     /// let mut build_dir = workspace.build_dir("foo");

src/lib.rs

@@ -25,7 +25,7 @@ mod inside_docker;
 pub mod logging;
 mod native;
 mod prepare;
-mod toolchain;
+pub mod toolchain;
 mod tools;
 mod utils;
 mod workspace;