docker: use update-toolchain in entrypoint

using update-toolchain instead of add-essential files installs the
toolchain first if it doesn't exist.

This still keeps the `add-essential-files` command in case someone needs
to update the essential files without updating the toolchain version.

Author: jyn514

docker-entrypoint.sh

@@ -36,6 +36,6 @@ if ! [ -d "${CRATESFYI_PREFIX}/crates.io-index/.git" ]; then
     git --git-dir="$CRATESFYI_PREFIX/crates.io-index/.git" branch crates-index-diff_last-seen
 fi
 
-cratesfyi build add-essential-files --only-first-time
+cratesfyi build update-toolchain --only-first-time
 
 cratesfyi "$@"

src/bin/cratesfyi.rs

@@ -65,11 +65,13 @@ pub fn main() {
                     .takes_value(true)
                     .conflicts_with_all(&["CRATE_NAME", "CRATE_VERSION"])
                     .help("Build a crate at a specific path")))
-            .subcommand(SubCommand::with_name("add-essential-files")
-                .about("Adds essential files for rustc")
+            .subcommand(SubCommand::with_name("update-toolchain")
+                .about("update the curretntly installed rustup toolchain")
                 .arg(Arg::with_name("ONLY_FIRST_TIME")
                      .long("only-first-time")
-                     .help("add essential files only if no essential files are present")))
+                     .help("update toolchain only if no toolchain is currently installed")))
+            .subcommand(SubCommand::with_name("add-essential-files")
+                .about("Adds essential files for the installed version of rustc"))
             .subcommand(SubCommand::with_name("lock").about("Locks cratesfyi daemon to stop \
                                                               building new crates"))
             .subcommand(SubCommand::with_name("unlock")
@@ -171,18 +173,20 @@ pub fn main() {
                     matches.value_of("CRATE_NAME").unwrap(), matches.value_of("CRATE_VERSION").unwrap(), None),
             }.expect("Building documentation failed");
             docbuilder.save_cache().expect("Failed to save cache");
-        } else if let Some(m) = matches.subcommand_matches("add-essential-files") {
+        } else if let Some(m) = matches.subcommand_matches("update-toolchain") {
             if m.is_present("ONLY_FIRST_TIME") {
                 let conn = db::connect_db().unwrap();
                 let res = conn.query("SELECT * FROM config WHERE name = 'rustc_version';", &[]).unwrap();
                 if !res.is_empty() {
-                    println!("add-essential files was already called in the past, exiting");
+                    println!("update-toolchain was already called in the past, exiting");
                     return;
                 }
             }
-
             let mut builder = RustwideBuilder::init().unwrap();
-            builder.update_toolchain().expect("failed to add essential files");
+            builder.update_toolchain().expect("failed to update toolchain");
+        } else if matches.subcommand_matches("add-essential-files").is_some() {
+            let mut builder = RustwideBuilder::init().unwrap();
+            builder.add_essential_files().expect("failed to add essential files");
         } else if let Some(_) = matches.subcommand_matches("lock") {
             docbuilder.lock().expect("Failed to lock");
         } else if let Some(_) = matches.subcommand_matches("unlock") {

src/docbuilder/rustwide_builder.rs

@@ -127,7 +127,7 @@ impl RustwideBuilder {
         }
     }
 
-    fn add_essential_files(&mut self) -> Result<()> {
+    pub fn add_essential_files(&mut self) -> Result<()> {
         self.rustc_version = self.detect_rustc_version()?;
         let rustc_version = parse_rustc_version(&self.rustc_version)?;